141. Linked List Cycle

[hroc135]

141. Linked List Cycle https://leetcode.com/problems/linked-list-cycle/

[hayashi-ay]

訪れたノードをHash Tableで管理する方法でも解いてみると良いと思います。まずはこっちの解法が思いつくのかなと個人的には思います。

[hroc135]

そうですね、実装してみました。

/**
 * Definition for singly-linked list.
 * type ListNode struct {
 *     Val int
 *     Next *ListNode
 * }
 */
func hasCycle(head *ListNode) bool {
    // hash table
    visited := make(map[*ListNode]bool)
    for head != nil {
        _, exist := visited[head]
        if exist {
            return true
        }
        visited[head] = true
        head = head.Next
    }
    return false
}

hash tableを使った実装だとメモリ使用量がO(n)になるのに対して、tortoise and hareだとO(1)ですね。ただ、hash tableの方が自然な発想という感じがしますね

[hayashi-ay]

良いと思います。

[Mike0121]

一応、チーム内で合意がある場合を除いて変数名はフルで書いた方が良いとの指摘が以前ありました。
https://discord.com/channels/1084280443945353267/1196472827457589338/1249723841228439603
https://discord.com/channels/1084280443945353267/1201211204547383386/1222123409295675394

[hroc135]

ご指摘ありがとうございます。go言語はミニマリスト的な思想が強く、命名はなるべく短くすることが好まれるのですが、それによって失われるわかりやすさにも気をつけるようにしたいと思います

[oda]

https://google.github.io/styleguide/go/decisions#naming
こういうところをまず引き、照らし合わせて論じましょう。

[oda]

私は、map に追加していくのが想定解と思うのと、あと Discord の中を漁って欲しいですね。こう、読む癖をつけるのがかなり大きな目的なので。