112.path sum by nicah4o · Pull Request #24 · nicah4o/arai60

nicah4o · 2026-06-02T04:45:44Z

This problem: https://leetcode.com/problems/path-sum/
Next problem: https://leetcode.com/problems/binary-tree-level-order-traversal/description/

Manato110 · 2026-06-02T04:59:36Z

+- https://github.com/attractal/leetcode/pull/37/changes
+bfsなどの方法
+
+自分でもstack bfsをやってみた。


stackでLIFOに探索をしているので、深さ優先探索(DFS)ですね。
ノードに子があったら、子から優先的に探索しているので。

レビューありがとうございます。
気づきませんでしたが確かにそうでした。ありがとうございます。

幅優先探索に変形してみるとよいと思います

@h-masder

class Solution { public: bool hasPathSum(TreeNode* root, int targetSum) { if (!root) return false; queue<pair<TreeNode*, int>> nodes_and_sums; nodes_and_sums.push({root, root->val}); while (!nodes_and_sums.empty()) { auto [node, sum] = nodes_and_sums.front(); nodes_and_sums.pop(); if (!node->left && !node->right) { if (sum == targetSum) return true; } if (node->left) { nodes_and_sums.push({node->left, sum + node->left->val}); } if (node->right) { nodes_and_sums.push({node->right, sum + node->right->val}); } } return false; } };

queueにすることで同じレベルの左右nodeを見てその下に向かう形になりますね。
自分が最初bfsといっていたものはある最深部まで行ってからその一つ上のレベルに戻ります。

h-masder · 2026-06-02T08:55:24Z

+        return hasPathSum(root->left, targetSum) ||
+               hasPathSum(root->right, targetSum);
+    }
+};


経路が見つかったあとも探索するので、プログラムの動作に若干違和感ありますね。

見つかった時点でTrueを返したほうがわかりやすいかもしれません

下の深さ優先のコードのイメージですね

レビューありがとうございます。
リターンの条件を

if (!node->left && !node->right && sum == targetSum)

とすればよかったですが、思いつきませんでした。

経路が見つかったあとも探索するので、プログラムの動作に若干違和感ありますね。

return hasPathSum(root->left, targetSum) || hasPathSum(root->right, targetSum);

この部分のことですかね？||は、短絡評価されるので、条件を満たすpathが見つかった時点で打ち切られると思います。

if (!node->left && !node->right && sum == targetSum)

こちらについては、変更の意図がよく分かりませんでした。

はい、あげていただいた箇所の話です。

条件を満たすpathが見つかった時点で打ち切られると思います。

ご指摘いただきありがとうございます。

bool left_value = hasPathSum(root->left, targetSum); bool right_value = hasPathSum(root->right, targetSum); if (left_value || right_value) { return true; } else { return false; }

を清書したものだと思い込んでいました。

こちらについては、変更の意図がよく分かりませんでした。

私もわかりかねますが、
とはいえ、まずはこちらが何の話をしているのかをちゃんと提示すべきでしたね。すみません。

@rikugame @h-masder
レビューとご指摘ありがとうございます。
最初のh-masderさんのコメントでコメント箇所より下のDFSについての指摘であるということだったので、そこでの変数resultの扱いにおける話だと理解して変更しました。
resultにtrueが入った時点でループが終わらず、全走査をしているからです。

最初のh-masderさんのコメントでコメント箇所より下のDFSについての指摘である

これは勘違いではないでしょうか。「下の深さ優先のコードのイメージですね」というのは、step3を指していて、早期リターンの例示かと思います。

@h-masder さんの最初のコメントは、一番上のコードの以下の部分に対して正当なコメントだと思います。

bool left_value = hasPathSum(root->left, targetSum); bool right_value = hasPathSum(root->right, targetSum); if (left_value || right_value) { return true; } else { return false; }

@nicah4o さん
一方で、Copilotの出したコードが短絡評価を活用したものになっていることについては、大丈夫そうでしょうか？

@rikugame
ありがとうございます。文脈を読めていませんでした。
短絡評価というのは左辺の評価だけで判断できるというものですね。
そちらの変更に目を向けていなかったのでありがたい議論でした。

nodchip · 2026-06-04T09:40:46Z

+        }
+        bool left_value = hasPathSum(root->left, targetSum);
+        bool right_value = hasPathSum(root->right, targetSum);
+        if (left_value || right_value) {


return left_value || right_value;

と書いたほうがシンプルだと思いました。

ありがとうございます。
式の値を返すという考え方がまだ身についていないので積極的に取り入れます。

nodchip · 2026-06-04T09:41:24Z

+        if (targetSum == 0 && !root->left && !root->right) {
+            return true;
+        }
+        bool left_value = hasPathSum(root->left, targetSum);


left_value という変数名からは、左側の子ノード以下に関する何らかの値というニュアンスを感じました。

そうですね。valueはこの場合boolの値以外もあるので良くなかったです。
has_Path_leftなどがよかったかもしれません。

nodchip · 2026-06-04T09:43:40Z

+                result = (sum == targetSum);
+            }
+            if (node->left) {
+                nodes_and_sums.push({node->left, sum});


nodes_and_sums.emplace(node->left, sum);

と書くとシンプルになると思います。

レビューありがとうございます。
pushとの違いが分かりました。emplaceは直接構築できるのですね。

112.path sum

50cf6a3

Manato110 reviewed Jun 2, 2026

View reviewed changes

h-masder reviewed Jun 2, 2026

View reviewed changes

nodchip reviewed Jun 4, 2026

View reviewed changes

Conversation

nicah4o commented Jun 2, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

h-masder Jun 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

h-masder Jun 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

h-masder Jun 3, 2026 •

edited

Loading

h-masder Jun 3, 2026 •

edited

Loading