Same Tree #45
Open
Same Tree #45
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
liquo-rice
reviewed
Oct 16, 2024
| @@ -0,0 +1,29 @@ | |||
| //lint:file-ignore U1000 Ignore all unused code | |||
There was a problem hiding this comment.
ある木が木の集合中に存在するかを調べたいという問題だったらどうしますか？このクエリは複数回発行されるものとします。
There was a problem hiding this comment.
検索を効率化するために例えば下記のように木全体をハッシュ化するなどはいかがでしょうか。
- 木の追加時にノードの値を文字列に変換して、pre-orderでそれらの文字列を結合していき、最終的に得られた文字列を衝突耐性の高いハッシュ関数（SHA256など）でハッシュ化
- ノードがない箇所については"\nil"のような文字列を入れることで表現する。入力に\が含まれているときは\\にするなどしてエスケープ処理する
There was a problem hiding this comment.
この問題とは関係ないですが、木の全ての部分木もハッシュ化したいならどうしますか？全ての部分木に対してpre-order traversalをするのは効率が悪そうです。
There was a problem hiding this comment.
post-orderで下から順に各ノードでハッシュを計算し、親ノードは左右子ノードのハッシュ値を使ってハッシュを計算するという形にするのが良さそうな気がしました。
There was a problem hiding this comment.
There was a problem hiding this comment.
ありがとうございます。全く考えてなかったことなので勉強になりました。
liquo-rice
reviewed
Oct 16, 2024
| /* | ||
| リファクタした。 | ||
| */ | ||
| func isSameTreeStep2(p *TreeNode, q *TreeNode) bool { |
There was a problem hiding this comment.
再帰をループに直すのは、練習としてわりと重要度高いと個人的には思っています。機会があれば、手の運動くらいの気持ちで。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Same Treeを解きました。レビューをお願い致します。
問題：https://leetcode.com/problems/same-tree/
言語：Go
すでに解いている方々：
none