112. Path Sum #31

nittoco · 2024-07-30T07:12:02Z

https://leetcode.com/problems/path-sum/description/
Given the root of a binary tree and an integer targetSum, return true if the tree has a root-to-leaf path such that adding up all the values along the path equals targetSum.

A leaf is a node with no children.

https://leetcode.com/problems/path-sum/description/ Given the root of a binary tree and an integer targetSum, return true if the tree has a root-to-leaf path such that adding up all the values along the path equals targetSum. A leaf is a node with no children.

TORUS0818 · 2024-07-30T08:06:38Z

112. Path Sum.md

+                search_target_sum(sum_so_far, node.left)
+            if node.right:
+                search_target_sum(sum_so_far, node.right)
+            sum_so_far -= node.val


sum_so_far -= node.valこれ必要でしょうか。

不要でしょう。

TORUS0818 · 2024-07-30T08:15:28Z

112. Path Sum.md

+                path_sums.add(sum_so_far)
+            caluculate_path_sums(sum_so_far, node.left, path_sums)
+            caluculate_path_sums(sum_so_far, node.right, path_sums)
+            sum_so_far -= node.val


sum_so_far -= node.valこれ必要でしょうか。

手作業でbacktrackingするなら戻る時に引くなあと思って入れたんですが、よく考えたら引いた後に関数内でなんか操作している訳ではないのでいらなかったですね。
Pythonでは変数はその関数内でローカルなので、関数抜けたら戻りますし。
ありがとうございます。

sum_so_farをcall stackに積まないで、nonlocalとして扱う書き方もありますね。その場合は、一歩戻るときに sum_so_far -= node.val を入れてあげないとですね。

確かに、自分が頭でシミュレーションしてたのをコードに書くならそれになりますね。
スッキリしました、ありがとうございます。

TORUS0818 · 2024-07-30T08:23:58Z

112. Path Sum.md

+```
+
+- 手でやるならbacktrackingのように足し引きしながら計算する気がするので、Step1を綺麗にしたやつを書いてみる
+    - [True]みたいに要素数1のリストを持たせるのも微妙かと思い、path_sumsをsetで持たせることにした


全ての根から葉までの和を保持しておくのはやや贅沢な気がします。
少し設定を変えてtargetSumに合致するようなパスがいくつあるか数える時、とかならこういう実装も良さそうですね。

fhiyo · 2024-07-31T09:11:52Z

112. Path Sum.md

+
+- 手でやるならbacktrackingのように足し引きしながら計算する気がするので、Step1を綺麗にしたやつを書いてみる
+    - [True]みたいに要素数1のリストを持たせるのも微妙かと思い、path_sumsをsetで持たせることにした
+    - returnに()をつけるか迷った(つけないと分かりにくいか、不要か)


個人的には不要かなと思いました。分かりやすくなっていない気がするので。

不要でしょう。

Yoshiki-Iwasa · 2024-08-01T02:07:52Z

112. Path Sum.md

+                return sum_so_far == targetSum
+            return search_target_sum(sum_so_far, node.left) or search_target_sum(sum_so_far, node.right)
+
+        return search_target_sum(0, root)


ここにtargetSumを渡して、関数内で引いていく方式にすればスコープ外の変数を使わなくて済むのでそれも手だと思います

付随した質問: よくPythonコードでスコープ外の変数を使っている人を見かけるのですが、これってPythonでは一般的なんでしょうか？
個人的には変数のスコープは明確であってほしいと思ってしまいます

「スコープ外の変数を使う」とは今回の場合、69行目において、targetSumが外側にある、ということですよね。それをしたくない理由は、挙動が追いにくいからでしょうか。（言われれば確かにそう感じる理由もわかる気がします）
自分は大規模コードの開発経験がないのでいまいち一般的な感覚がわからないのですが、どうなんでしょう…？

特に問題ないと思います。

liquo-rice · 2024-08-01T11:12:59Z

112. Path Sum.md

+### Step1
+
+- 本当は部下に仕事を任せてreturnしてもらう実装が好きだが、値の候補がたくさんreturnされてしまうのがどうかと思い、backtrackingで書いた
+- nonlocalでもいいが、mutableなリストで書いた


nonlocalの方が意図が明確だと思います。

liquo-rice · 2024-08-01T11:13:19Z

112. Path Sum.md

+                search_target_sum(sum_so_far, node.left)
+            if node.right:
+                search_target_sum(sum_so_far, node.right)
+            sum_so_far -= node.val


不要でしょう。

liquo-rice · 2024-08-01T11:14:05Z

112. Path Sum.md

+    def hasPathSum(self, root: Optional[TreeNode], targetSum: int) -> bool:
+        def search_target_sum(sum_so_far, node: Optional[TreeNode]):
+            sum_so_far += node.val
+            if not (node.left or node.right):


not node.left and not node.rightの方がわかりやすく感じます。

liquo-rice · 2024-08-01T11:14:53Z

112. Path Sum.md

+            sum_so_far += node.val
+            if not (node.left or node.right):
+                if sum_so_far == targetSum:
+                    has_target[0] = True


ここでreturnしたら良いでしょう。

liquo-rice · 2024-08-01T11:16:18Z

112. Path Sum.md

@@ -0,0 +1,96 @@
+### Step1
+
+- 本当は部下に仕事を任せてreturnしてもらう実装が好きだが、値の候補がたくさんreturnされてしまうのがどうかと思い、backtrackingで書いた


答えを一つ見つけたら、そこで打ち切った方が良いでしょう。

liquo-rice · 2024-08-01T11:17:21Z

112. Path Sum.md

+- [参考]
+  - (https://github.com/TORUS0818/leetcode/pull/27/files)
+  - （https://github.com/kazukiii/leetcode/pull/26/files）
+- 再帰をするなら、単にFalse, Trueを返す関数を作ればよかった。(なぜか変にsumを全部返さなきゃとか思ってしまった)


Step 1でも、sumは返していないようです。

liquo-rice · 2024-08-01T11:17:57Z

112. Path Sum.md

+                return True
+            return False
+
+        if not root:


search_target_sumの中に入れたほうが、コードの重複が減りますね。

liquo-rice · 2024-08-01T11:20:14Z

112. Path Sum.md

+                return sum_so_far == targetSum
+            return search_target_sum(sum_so_far, node.left) or search_target_sum(sum_so_far, node.right)
+
+        return search_target_sum(0, root)


特に問題ないと思います。

liquo-rice · 2024-08-01T11:21:04Z

112. Path Sum.md

+
+- 手でやるならbacktrackingのように足し引きしながら計算する気がするので、Step1を綺麗にしたやつを書いてみる
+    - [True]みたいに要素数1のリストを持たせるのも微妙かと思い、path_sumsをsetで持たせることにした
+    - returnに()をつけるか迷った(つけないと分かりにくいか、不要か)


不要でしょう。

liquo-rice · 2024-08-01T11:25:32Z

112. Path Sum.md

+
+class Solution:
+    def hasPathSum(self, root: Optional[TreeNode], targetSum: int) -> bool:
+        def caluculate_path_sums(sum_so_far, node: Optional[TreeNode], path_sums: set):


path_sumsを引数として渡す必要はなさそうです。

liquo-rice · 2024-08-02T03:57:18Z

112. Path Sum.md

+## Step3(レビュー反映復習)
+```python
+
+class Solution:


l62の方が良かったと思います。

確かに自分もどれを選ぶかを選ぶならl62のやつで書くかもです。(今回はStep1の書き直しという感じで書いてみました)
やり直しとかも見てくださりありがとうございます🙇

kazukiii

見ました。コメントのあるところ以外は良さそうに思いました。

kazukiii · 2024-08-03T00:11:14Z

112. Path Sum.md

+                path_sums.add(sum_so_far)
+            caluculate_path_sums(sum_so_far, node.left, path_sums)
+            caluculate_path_sums(sum_so_far, node.right, path_sums)
+            sum_so_far -= node.val


sum_so_farをcall stackに積まないで、nonlocalとして扱う書き方もありますね。その場合は、一歩戻るときに sum_so_far -= node.val を入れてあげないとですね。

oda · 2024-08-06T17:46:59Z

112. Path Sum.md

+        def search_target_sum(sum_so_far, node: Optional[TreeNode]):
+            nonlocal has_target
+            if not node:
+                return


if has_target: return
で枝狩りしておくと、ツリー全体を舐めなくて済みますね。

あ、書いた時頭が混乱していましたが、これはL111のreturnとは別で、探索をかなりスルーできますね。(逆にL111はその後すぐNoneのノードが来るのでreturnする意味そんなにないですね）
ありがとうございます

112. Path Sum

022ee4f

https://leetcode.com/problems/path-sum/description/ Given the root of a binary tree and an integer targetSum, return true if the tree has a root-to-leaf path such that adding up all the values along the path equals targetSum. A leaf is a node with no children.

TORUS0818 reviewed Jul 30, 2024

View reviewed changes

fhiyo reviewed Jul 31, 2024

View reviewed changes

Yoshiki-Iwasa reviewed Aug 1, 2024

View reviewed changes

liquo-rice reviewed Aug 1, 2024

View reviewed changes

112. Path Sum　レビューを反映

d5de116

liquo-rice reviewed Aug 2, 2024

View reviewed changes

kazukiii reviewed Aug 3, 2024

View reviewed changes

oda reviewed Aug 6, 2024

View reviewed changes

colorbox mentioned this pull request Feb 10, 2025

112. Path Sum colorbox/leetcode#39

Open

		@@ -0,0 +1,96 @@
		### Step1

		- 本当は部下に仕事を任せてreturnしてもらう実装が好きだが、値の候補がたくさんreturnされてしまうのがどうかと思い、backtrackingで書いた

112. Path Sum #31

Are you sure you want to change the base?

112. Path Sum #31

Uh oh!

Conversation

nittoco commented Jul 30, 2024

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!

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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kazukiii left a comment

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!

8 participants