Skip to content

Commit e6d6b0e

Browse files
committed
LeetCode 206: Add feedback
1 parent 51276ff commit e6d6b0e

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

leetcode/206/memo.md

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,4 +185,12 @@ def factorial(n: int, accumulated: int = 1) -> int:
185185

186186
## Feedback
187187

188-
TODO: Add feedback from reviewers here
188+
- **再帰 <-> ループ間変形**
189+
- 末尾再帰
190+
- 変数の末尾にコメントを書くとフォーマットが大変なので、素直にdocstringに書いた方がいいかも
191+
- 意味ではなく操作の順番からの命名 (previous) に気を付ける、パズルを作らないように
192+
- 四行くらいしかないなら空行を入れても読みやすさは変わらないかも
193+
- どうせ空行を入れるなら、コメントを積極的に入れてもいいかも
194+
- `reversing`, `appending`という命名
195+
- PEP8の (sparingly) というニュアンスに気づかなかった "Extra blank lines may be used (sparingly) to separate groups of related functions."
196+
- `rest*`, `*_tails`

0 commit comments

Comments
 (0)