Conversation
| fast = head | ||
| slow = head | ||
|
|
||
| while fast and fast.next: |
There was a problem hiding this comment.
fastとfast.nextの条件文は明確にNoneだけを否定できるis not Noneを使った方が良いと思います！今の状態だと()とかも判定に含まれているので
| return True | ||
|
|
||
| visited.add(val) | ||
| head = head.next |
There was a problem hiding this comment.
head は連結リスト先頭のノードを表す単語です。 head が移動していくのは違和感があります。 node 等、一度別の変数に代入し、そちらを動かしていったほうが自然に感じます。
There was a problem hiding this comment.
自分も野田さんと同様に、変数nodeに代入してからのほうが良いと思いました。
| fast = fast.next.next | ||
| slow = slow.next | ||
|
|
||
| if fast == slow: |
There was a problem hiding this comment.
比較には ==、 is などのどれを使うべきか意識しよう、というコメントが多々ありましたので一例を添付いたします。ご参考までに。
参考：
https://github.com/SakaiYuzuki/LeetCode_arai60/pull/1/files#r2444784687
There was a problem hiding this comment.
問題には関係ないのですが気になったので一応コメントします。
こちらのファイルですが、create-pr.shによって書き換えられたテンプレートがコミットされたものでしょうか？(最新のスクリプトではコミットしないようになっていることは確認できました。)
実際の実務を想定するならば、マージ対象でないファイルはプルリクエストから除外したほうがよいかなと思います。
また、create-pr.shの処理についてですが、ファイルを直接上書きするとgit上で変更のあるファイルとして認識されてしまい、コミット時に除外する手間があるのかなと思うので、テンプレートの内容をsedなどで置換し、gh pr createコマンドの--bodyオプションに置換したテキストの内容を渡してプルリクを作成するファイルの変更がでなくてよいかと思いました。
参考: https://cli.github.com/manual/gh_pr_create
-b, --body
Body for the pull request
5 participants
解く問題
Linked List Cycle
次に解く問題
Linked List Cycle Ii