Maximum Depth of Binary Tree #41
Conversation
| type entry struct { | ||
| node *TreeNode | ||
| depth int | ||
| } |
There was a problem hiding this comment.
慣例かは知らないですが、確かに最初に定義するのが一般的ですね
| if root == nil { | ||
| return 0 | ||
| } | ||
| maximum := 0 |
| if e.node == nil { | ||
| continue | ||
| } | ||
| maximum = max(maximum, e.depth) |
| return 0 | ||
| } | ||
| maximum := 0 | ||
| stack := []entry{{root, 1}} |
There was a problem hiding this comment.
私は、比較的、stack や queue という名前は状況次第で許容側です。
何が入っているかが分かって、その後でどのように使われるか(後ろで出し入れする)のヒントになっています。
そうすると、その後を読む流れをあまり阻害しないからです。
ただ、どういう物が入っているか、説明するのが複雑であるならば、何が入っている stack なのか説明が欲しくなります。
一方で、関数名はだいたい詳細に説明して欲しいです。たとえば、dfs() とだけあると、何が返ってくるかが分からないので、dfs の中を読みにいかないといけなくなります。
|
多くの解法を試していて参考になりました #15 (comment) の議論だと再帰はあまり望ましくないとありましたが、入力サイズが小さいので再帰でやってしまうか、それ以外だとmaxDepthIterativeDFSになるのかなと思いました |
| queue := []entry{{root, 1}} | ||
| for len(queue) > 0 { | ||
| e := queue[0] | ||
| queue = queue[1:] |
There was a problem hiding this comment.
自分は Go 言語に詳しくないため質問させていただきたいのですが、このように書いた場合、配列の後半のコピーが作成されるのでしょうか？
There was a problem hiding this comment.
インデックス1以降の要素が含まれた動的配列が代入されます。
https://go.dev/ref/spec#Slice_expressions
There was a problem hiding this comment.
コピーは作られず、 インデックスとポインタの対応関係が変わるような動作になります。
Slicing does not copy the slice’s data. It creates a new slice value that points to the original array. This makes slice operations as efficient as manipulating array indices. Therefore, modifying the elements (not the slice itself) of a re-slice modifies the elements of the original slice:
d := []byte{'r', 'o', 'a', 'd'}
e := d[2:]
// e == []byte{'a', 'd'}
e[1] = 'm'
// e == []byte{'a', 'm'}
// d == []byte{'r', 'o', 'a', 'm'}
| 一番オーソドックスなやり方。 | ||
|
|
||
| １フレームの大きさは、引数8 + 返り値8 + ローカル変数(8 + 8) + FP + 戻りアドレス = 48Bぐらい。最大ノード数が10^4なのでスタックサイズは、48 * 10^4 = 480KBぐらいになると予想できる。 | ||
| Linuxのデフォルトのスタックの大きさが8MBなのでスタックオーバーフローの危険性は低い。 |
There was a problem hiding this comment.
自分ここら辺の理解が浅いので質問させてください！
ここでLinuxのスタックサイズを考慮するのはなぜですか？
There was a problem hiding this comment.
Goはランタイムがよしなにやってくれているだけで、OSスレッド上でプログラムが動作していることには変わりがないので、、
There was a problem hiding this comment.
前に教えていただいたGoのスタックサイズの上限をこの時は知らず、調べても出てこなかったので、とりあえずはLinuxのスレッドに割り当てられるスタックのデフォルトサイズを基準に考えたという感じですね
There was a problem hiding this comment.
GoがLinuxのスタックサイズを超えてスタックを割り当てられるのは、Goランタイムが適宜ヒープからメモリ領域を割り当てて、スタックサイズを拡張している感じだった気がします（今パッとソースが出てこないのですが、確かそうだったはず）
Maximum Depth of Binary Treeを解きました。レビューをお願い致します。
問題：https://leetcode.com/problems/maximum-depth-of-binary-tree/
言語：Go
すでに解いている方々：
hayashi-ay/leetcode#22
Mike0121/LeetCode#6
sakupan102/arai60-practice#21
nittoco/leetcode#14
fhiyo/leetcode#23
SuperHotDogCat/coding-interview#34
Yoshiki-Iwasa/Arai60#23
NobukiFukui/Grind75-ProgrammingTraining#38
kazukiii/leetcode#22
Ryotaro25/leetcode_first60#23
goto-untrapped/Arai60#45
seal-azarashi/leetcode#20
hroc135/leetcode#20