From f5b72daaf5e8841d4d1a4e95335025c53e1d04ea Mon Sep 17 00:00:00 2001 From: canisterism Date: Sun, 21 Jul 2024 15:27:34 +0900 Subject: [PATCH 1/4] done --- arai60/141_linked_list_cycle.md | 155 ++++++++++++++++++++++++++++++++ 1 file changed, 155 insertions(+) create mode 100644 arai60/141_linked_list_cycle.md diff --git a/arai60/141_linked_list_cycle.md b/arai60/141_linked_list_cycle.md new file mode 100644 index 0000000..0cb1fd6 --- /dev/null +++ b/arai60/141_linked_list_cycle.md @@ -0,0 +1,155 @@ +https://leetcode.com/problems/linked-list-cycle/ + +# Step1 + +かかった時間：40min + +```ruby +# Definition for singly-linked list. +# class ListNode +# attr_accessor :val, :next +# def initialize(val) +# @val = val +# @next = nil +# end +# end + +# @param {ListNode} head +# @return {Boolean} +def hasCycle(head) + return false if head.nil? || head.next.nil? + slow, fast = head, head + + while !fast.nil? && !fast.next.nil? + slow = slow.next + fast = fast.next.next + return true if slow.val == fast.val + end + false +end +``` +思考ログ： +- 解説見たことあったので解法はなんとなく頭に入ってるつもり + - slowとfastのポインタを用意してfastが追い越したらcycle + +何がわからなかったか + - `@param {ListNode} head` に入ってくるデータの構造がよくわかってなかった + - こんな感じのコードを書いて`fast`がnilになったりして困っていた +```ruby +def hasCycle(head) + slow = head + fast = head.next + + loop do + if slow.val == fast.val + return true + elsif fast.next.nil? + return false + end + slow = slow.next + fast = fast.next.next + end +end +``` +- >`@param {ListNode} head` に入ってくるデータの構造がよくわかってなかった + - これはデータ構造が分かってないんじゃなくて、ノードの長さが1だけのときのケースについてケアできてないだけだった +- 答えを見てから書いたコード +```ruby +def hasCycle(head) + return false if head.nil? || head.val.nil? + slow, fast = head, head + + while !fast.nil? && !fast.next.nil? # <- fastがnilで落ちる + slow = slow.next + fast = fast.next.next + + return true if slow.val == fast.val + end + false +end +``` +- パスしなかったのでもう一回消して答えを見てから書いた（一番上のコード）。パスしたのでStep1終わり + +何を考えて解いていたか +- アルゴリズムの仕組みとしては分かってたもののコードでどう書いたらその通りに動くのかが掴めなくて時間を使った + - そもそも事前に解説を見たことがあるという半分ズルなので初見だとまず解き方を思いついてなかったと思う + +正解してから気づいたこと +- 頭からnilのチェックをするケースを書き始めると混乱するのでやりたいメジャーケース（今回だとwhileの中）から書き始めて、あとから細かいケースについて詰めていくと自分の考えてることと書いてるコードが一致してて書きやすいかもと思った +- off topic気味だけどleetcodeのエディタにダイレクトで書いてるのはいいのかな（IDE使わないようにしてればOKだと思うが） + +# Step2 + +参考にした過去ログなど： +- https://github.com/sendahuang14/leetcode/pull/1/files +- https://github.com/kagetora0924/leetcode-grind/pull/1/files + +思考ログ： +- Step1はwhileの条件が読みにくいかも？改善ポイントってこういうのでいいのかな？ +- わからなかったので他の人のPRを探す + - https://github.com/sendahuang14/leetcode/pull/1/files + - HashMapでたどったノードをメモする解法。チラッと考えたけど前に見たことある解法で解くイメージが先行して試してなかった。 + - 辞書に書き込むのと配列にpushするのだと辞書に書き込んだ方がO(1)？でアクセスできるので良さそう？ +- そういえば計算量について全然意識できてなかったことに気づく + - 今のアルゴリズムだとどうなるんだ...？ + - 時間計算量は最大でO(N)？ + - 空間計算量はO(1)でいいのかな +- 「より自然な」コードだとたどったノードをメモっていくのが直感的な感じがするので辞書にたどったノードを書いてみる + - と思ったけど辞書のキーがnode.valだったとして、キーに対応する値に何をいれるべきなのか迷った + - 値にboolean入れる想像をしたけど、結局アクセスする時に返ってくる値がややこしくなるのでやめた + - のでとりあえず配列に詰めることにした +- 配列に重複があることをどう判定する？ + - `Array#uniq` は重複を排除した配列を返すので、`visited_nodes.length == visited_nodes.uniq.length`で判定はできるけど冗長な感じがする +- RubyにSetがあったことを思い出す + - https://docs.ruby-lang.org/ja/latest/class/Set.html#I_--3C--3C + - >Set#add? は、集合に要素が追加された場合には self を、変化がなかった場合には nil を返します。 + - これの返り値で判定できそう +- とりあえずパスするコードは書けた +```ruby +def hasCycle(head) + return false if head.nil? || head.next.nil? + + visited_nodes = Set.new + visited_nodes.add(head.val) + current = head.next + + while !current.nil? do + has_visited = visited_nodes.add?(current.val).nil? + return true if has_visited + + current = current.next + end + false +end +``` +- セルフツッコミしてみる + - もうちょっと最初の5行ぐらいをシュッと書けるかも + - 先にhead.valをaddするんじゃなくて、そのまま全部whileの中に入れられないかな + - `has_visited = visited_nodes.add?(current.val).nil?`が読みづらい気がする + - `Set#add?`が要素を追加できなかったらnilを返すことを知ってなきゃシュッと読めない + +# Step3 + +かかった時間：15min + +```ruby +def hasCycle(head) + visited_nodes = Set.new + current = head + + while !current.nil? do + has_visited = visited_nodes.add?(current.val).nil? + return true if has_visited + + current = current.next + end + false +end +``` +思考ログ： +- 書きながらどうやったら直感的になるか考えた + - 何も考えずに最初から「headをcurrentとして、valを詰める→ポインタが示す先に移動する→ノードがあるか確認する」を繰り返すだけで行けるかも + - 書けた。だいぶ直感的になった気がする。 + - このぐらいシュッとしてるなら`visited_nodes.add?(current.val).nil?`の部分については許容できるレベルだと思った + - 変数名については振り返ってなかったけど、見直してもあんまり改善ポイントが浮かばなかった + - 1発で3回連続パスした！終わり。 From fd7ebed3b283d79cd44599b42f2245d8725b00d6 Mon Sep 17 00:00:00 2001 From: canisterism Date: Mon, 22 Jul 2024 23:26:23 +0900 Subject: [PATCH 2/4] =?UTF-8?q?val=E3=81=AF=E3=83=A6=E3=83=8B=E3=83=BC?= =?UTF-8?q?=E3=82=AF=E3=81=A7=E3=81=AA=E3=81=84=E3=81=AE=E3=81=A7hash?= =?UTF-8?q?=E3=81=94=E3=81=A8set=E3=81=AB=E5=85=A5=E3=82=8C=E3=82=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- arai60/141_linked_list_cycle.md | 46 ++++++++++++++++++++++++++++++++- 1 file changed, 45 insertions(+), 1 deletion(-) diff --git a/arai60/141_linked_list_cycle.md b/arai60/141_linked_list_cycle.md index 0cb1fd6..8c2067d 100644 --- a/arai60/141_linked_list_cycle.md +++ b/arai60/141_linked_list_cycle.md @@ -138,7 +138,7 @@ def hasCycle(head) current = head while !current.nil? do - has_visited = visited_nodes.add?(current.val).nil? + has_visited = visited_nodes.add?(current).nil? return true if has_visited current = current.next @@ -153,3 +153,47 @@ end - このぐらいシュッとしてるなら`visited_nodes.add?(current.val).nil?`の部分については許容できるレベルだと思った - 変数名については振り返ってなかったけど、見直してもあんまり改善ポイントが浮かばなかった - 1発で3回連続パスした！終わり。 +```ruby +def hasCycle(head) + visited_nodes = Set.new + current = head + + while !current.nil? do + has_visited = visited_nodes.add?(current.val).nil? + return true if has_visited + + current = current.next + end + false +end +``` +- 以下Odaさんのレビューを受けた時のログ + - valはすべてのノードでuniqueでないので、Hash自体を詰めないと同じvalが出てくると正しい結果にならない。↓ + - `has_visited = visited_nodes.add?(current.val).nil?` +- SetにHashを入れた時の等価の判定ってどうなるんだっけ。多分key:valueで見ると思うけど... + - https://docs.ruby-lang.org/ja/latest/library/set.html + - >Set は内部記憶として Hash を使うため、集合要素の等価性は Object#eql? と Object#hash を用いて判断されます。 +```ruby +{'a': 1}.hash +=> 575188353908168896 +{'a': 2}.hash +=> 1910308263874936072 +{'a': 1}.eql?({'a': 2}) +=> false +{'a': 1}.eql?({'a': 1}) +=> true +``` + +```ruby +set = Set.new +=> # +set.add({'a': 1}) +=> #1}}> +set.add({'a': 1}) +=> #1}}> +set.add({'a': 2}) +=> #1}, {:a=>2}}> +``` +- 合ってそう。 + - key:valueでハッシュ値を作ってるぽいけどそれで合ってるのかソースを見に行こうとしたがどうやらCで実装してるぽくて読めなかった + - https://github.com/ruby/ruby/blob/master/hash.c これかな？ From 1d34e99b7f8bfcd269e36b5fa2080191a040cbfb Mon Sep 17 00:00:00 2001 From: canisterism Date: Mon, 22 Jul 2024 23:49:52 +0900 Subject: [PATCH 3/4] =?UTF-8?q?=E3=83=95=E3=83=AD=E3=82=A4=E3=83=89?= =?UTF-8?q?=E3=81=AE=E5=BE=AA=E7=92=B0=E6=A4=9C=E5=87=BA=E6=B3=95=E3=82=92?= =?UTF-8?q?loop+if=E3=81=A7=E6=9B=B8=E3=81=8F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- arai60/141_linked_list_cycle.md | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/arai60/141_linked_list_cycle.md b/arai60/141_linked_list_cycle.md index 8c2067d..7f6125b 100644 --- a/arai60/141_linked_list_cycle.md +++ b/arai60/141_linked_list_cycle.md @@ -197,3 +197,20 @@ set.add({'a': 2}) - 合ってそう。 - key:valueでハッシュ値を作ってるぽいけどそれで合ってるのかソースを見に行こうとしたがどうやらCで実装してるぽくて読めなかった - https://github.com/ruby/ruby/blob/master/hash.c これかな？ +- slow, fastのアルゴリズムをloop + if で書き直してみた。 +```ruby +def hasCycle(head) + slow = head + fast = head + + loop do + return false if fast.nil? || fast.next.nil? + + slow = slow.next + fast = fast.next.next + return true if slow == fast + end +end +``` +- 「次のノードまで見る→進んでもOKだったのでポインタを動かす→一致してるか判定」の流れで書いたら一発でこれが出てきて自分でビックリした。 + - slow, fastのアルゴリズム（フロイドの循環検出法と言うらしい）苦手だと思ってたけど思考の流れが整理されてないから分からないだけだったのかと思うなど From 00a8a97f0c11eedfa2d5f111c46e3731e247c061 Mon Sep 17 00:00:00 2001 From: canisterism Date: Mon, 22 Jul 2024 23:54:21 +0900 Subject: [PATCH 4/4] =?UTF-8?q?=E3=83=AC=E3=83=93=E3=83=A5=E3=83=BC?= =?UTF-8?q?=E3=81=AE=E6=8C=87=E6=91=98=E3=81=A7=E3=81=AE=E4=BF=AE=E6=AD=A3?= =?UTF-8?q?=E3=81=AF=E6=96=B0=E3=81=97=E3=81=84Step=E3=81=AB=E8=BF=BD?= =?UTF-8?q?=E8=A8=98=E3=81=99=E3=82=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- arai60/141_linked_list_cycle.md | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/arai60/141_linked_list_cycle.md b/arai60/141_linked_list_cycle.md index 7f6125b..09cdab9 100644 --- a/arai60/141_linked_list_cycle.md +++ b/arai60/141_linked_list_cycle.md @@ -138,7 +138,7 @@ def hasCycle(head) current = head while !current.nil? do - has_visited = visited_nodes.add?(current).nil? + has_visited = visited_nodes.add?(current.val).nil? return true if has_visited current = current.next @@ -153,13 +153,21 @@ end - このぐらいシュッとしてるなら`visited_nodes.add?(current.val).nil?`の部分については許容できるレベルだと思った - 変数名については振り返ってなかったけど、見直してもあんまり改善ポイントが浮かばなかった - 1発で3回連続パスした！終わり。 + +# Step4 + +レビューを受けて修正する。 + - >これ、val が unique であるという保証あるんでしたっけ? Set は Object 入れられたと思いますので、そちらのほうが素直では。 + - https://github.com/canisterism/leetcode/pull/2/files#r1686085757 + - valはすべてのノードでuniqueでないので、~Hash~ Node自体を詰めないと同じvalが出てくると正しい結果にならない。 + - 修正するとこうなる ```ruby def hasCycle(head) visited_nodes = Set.new current = head while !current.nil? do - has_visited = visited_nodes.add?(current.val).nil? + has_visited = visited_nodes.add?(current).nil? return true if has_visited current = current.next @@ -167,9 +175,6 @@ def hasCycle(head) false end ``` -- 以下Odaさんのレビューを受けた時のログ - - valはすべてのノードでuniqueでないので、Hash自体を詰めないと同じvalが出てくると正しい結果にならない。↓ - - `has_visited = visited_nodes.add?(current.val).nil?` - SetにHashを入れた時の等価の判定ってどうなるんだっけ。多分key:valueで見ると思うけど... - https://docs.ruby-lang.org/ja/latest/library/set.html - >Set は内部記憶として Hash を使うため、集合要素の等価性は Object#eql? と Object#hash を用いて判断されます。 @@ -197,7 +202,11 @@ set.add({'a': 2}) - 合ってそう。 - key:valueでハッシュ値を作ってるぽいけどそれで合ってるのかソースを見に行こうとしたがどうやらCで実装してるぽくて読めなかった - https://github.com/ruby/ruby/blob/master/hash.c これかな？ -- slow, fastのアルゴリズムをloop + if で書き直してみた。 + +>loop do + if return に分解してみて欲しいです。 +https://github.com/canisterism/leetcode/pull/2/files#r1685762007 +slow, fastのアルゴリズムをloop + if で書き直してみた。 + ```ruby def hasCycle(head) slow = head