Conversation
|
Pull Request のタイトルに問題番号と問題名を、 Description に問題へのリンクを張っておくと、レビューワーにとってレビューの助けになると思います。 |
|
|
||
| visited = set() | ||
|
|
||
| current_address = head |
There was a problem hiding this comment.
Python でアドレスといった場合、 Python のインタープリターの内部でオブジェクトを格納しているメモリアドレスを指すように思います。 Python のコードからそれらを直接参照することはほとんどないため、変数名に現れるのは違和感があります。
また current という単語は、情報量が少ないように感じました。
自分であれば node といった、どのような値が格納されているかがより明確に伝わる名前を付けると思います。
ただ、previous／next と対比させたい意図がある場合には current を付けるのも適切だと思います。このあたりは文脈に応じて、より意味が伝わる名前を選ぶと良いと思います。
There was a problem hiding this comment.
Linked Listのメモリアドレスを意識しての記述ですが、仰る通り変数にするのは読み手にとっては不親切ですね。
ご指摘ありがとうざいます。
|
読みやすく、よく理解されていて良いと思います！ |
| current_address = current_address.next | ||
|
|
||
| # NoneになればCycleを抜けたことになるのでFalseを返す | ||
| return False |
There was a problem hiding this comment.
行末は改行を入れたほうがいいと思います。仮にこのコードをプログラムで処理するときに一貫性がなくなってしまうのと、diffなどを行ったときに新しいコードの部分だけでなく改行を入れた部分も変更として捉えられてしまって読みにくくなってしまうことが起こり得ると思います。
https://qiita.com/hamacccccchan/items/11c17c7412a5aeb2ad74
https://qiita.com/ko1nksm/items/3aea7fb8ac7e0396c10f
There was a problem hiding this comment.
Insert Final NewlineをONにしました。ご指摘ありがとうございます！
4 participants
https://leetcode.com/problems/linked-list-cycle/