141. Linked List Cycle

[tarinaihitori]

https://leetcode.com/problems/linked-list-cycle/

[oda]

current is not Noneと書く方法もあります。これはよしあしあってどちらでもいいですかね。

Google Style Guide はis not None派。
https://google.github.io/styleguide/pyguide.html#2144-decision

PEP8 はどちらでもよさそう。
https://peps.python.org/pep-0008/#other-recommendations:~:text=Also%2C%20beware%20of%20writing%20if%20x%20when%20you%20really%20mean%20if%20x%20is%20not%20None%20%E2%80%93%20e.g.%20when%20testing%20whether%20a%20variable%20or%20argument%20that%20defaults%20to%20None%20was%20set%20to%20some%20other%20value.%20The%20other%20value%20might%20have%20a%20type%20(such%20as%20a%20container)%20that%20could%20be%20false%20in%20a%20boolean%20context!

[tarinaihitori]

Google Style Guide はis not None派。

not None は二重否定っぽくてあまり好みではなかったのですが、Google Style Guide では is not None が推奨されているんですね。None と他の値（例えば 0 や空文字列）を区別するために、明示的に None との比較を行うことで意図が明確になるという点は納得できました。今後は is not None も意識して使ってみようと思います。

[rihib]

関数は分けて実装したほうが読みやすいかなと思います

[tarinaihitori]

ありがとうございます。関数を分けて実装するのは頭になかったです。
関数を分けて実装するメリットとして、再利用性が上がったり、テストのしやすさ、読みやすさなどがありそうですね。
逆にinner functionとすることで、_has_cycleの範囲が限定されて管理しやすいというメリットもあると思います。
選択肢をもつ意味でも、step4では関数を分けて書いてみます。