142. linked list cycle II

[tNita]

This: https://leetcode.com/problems/linked-list-cycle-ii/description/
Next: https://leetcode.com/problems/remove-duplicates-from-sorted-list/description/

[oda]

コメント集に整え方を書いておいたと思うのですが、フラグを立てたくなったら何か整え方があるくらいに思っておくといいでしょう。

[TakayaShirai]

コメント集の整え方のリンクこちらです。
https://docs.google.com/document/d/11HV35ADPo9QxJOpJQ24FcZvtvioli770WWdZZDaLOfg/edit?tab=t.0#heading=h.9kpbwslvv3yv

こちらも参考になると思います。
nanae772/leetcode-arai60#3 (comment)

[t9a-dev]

全体的に読みやすかったです。

[TakayaShirai]

コメント集の整え方のリンクこちらです。
https://docs.google.com/document/d/11HV35ADPo9QxJOpJQ24FcZvtvioli770WWdZZDaLOfg/edit?tab=t.0#heading=h.9kpbwslvv3yv

こちらも参考になると思います。
nanae772/leetcode-arai60#3 (comment)

[TakayaShirai]

無限ループはバグの原因になりかねないため、個人的には避けたい気持ちです。今回は確かに無限ループから抜けることは確定しているので安全ですが、実務の場合は、こちらのコードに今後他の人が手を加える可能性もあり、意図せず無限ループが起きてしまうリスクがあるように感じます。以下のようなコードはどうでしょうか。

while from_start is not from_join_point:
    from_start = from_start.next
    from_join_point = from_join_point.next
return from_start

[tNita]

ご指摘ありがどうございます！

また、例までありがとうございます。参考にさせていただきます。

Discordの中もあさってみたところ、いくつか過去にも同様な議論がありました。

• 絶対ダメというものではない
• whileの条件式にループの条件をまとめておけば、ループの「主役」や終了条件がわかりやすい（→無限ループも起きにくくなる）

と自分は理解しました〜

huyfififi/coding-challenges#31 (comment)

[TakayaShirai]

リンクありがとうございます！オープンソースでも無限ループけっこう使用されているのですね。バグの温床のイメージがあったため結構驚きでした。チームや状況に応じて使い分けができると良さそうですね。

[nodchip]

ローカル変数は、snake_case で命名することをお勧めします。

参考として、関連するスタイルガイドを共有いたします。

https://peps.python.org/pep-0008/#function-and-variable-names

Function names should be lowercase, with words separated by underscores as necessary to improve readability.
Variable names follow the same convention as function names.

https://google.github.io/styleguide/pyguide.html#316-naming

local_var_name

なお、これらのスタイルガイドは唯一の正解というわけではなく、複数あるガイドラインの一つに過ぎません。
所属するチームによって好まれる書き方が異なることもありますので、ご自身の中で基準を持ちつつ、最終的にはチームの一般的な書き方に寄せることをお勧めします。