300.longest increasing subsequence by tom4649 · Pull Request #29 · tom4649/Coding

tom4649 · 2026-03-27T03:08:07Z

https://leetcode.com/problems/longest-increasing-subsequence/submissions/1960476790/

5ky7 · 2026-03-27T17:19:20Z

300/sol1.py

+class Solution:
+    def lengthOfLIS(self, nums: List[int]) -> int:
+        length_of_is = [1] * len(nums)
+        for i, n in enumerate(nums):


好みの問題かとは思いますが，nよりnumの方が好みです．nは意味が広く，例えばlen(nums)にも使われうると感じるためコードが長くなると勘違いしうる一方で，numならnumsの要素だと特定できる気がするからです．
この程度のスコープならnで良いという意見ももっともだとは思います．

5ky7 · 2026-03-27T17:20:21Z

300/sol1.py

@@ -0,0 +1,9 @@
+class Solution:
+    def lengthOfLIS(self, nums: List[int]) -> int:
+        length_of_is = [1] * len(nums)


length_of_isよりもlength_of_increasing_sequenceと省略しない流儀の方が現代では一般的なようです（cf）．
lenの方は省略しても（比較的よくみるので）良いかもしれません：len_increasing_subsequence

関数名にLISが疲れているから良いかと思ったのですが、isだと伝わらないかもしれませんね。
省略は極力避けた方が良さそうです。

個人的には len を省略するなら、len_is まで省略しても許容です。それならそれで is が何なのかコメントで補足してもらえると有り難いです。

長いので、len_is を採用しました。コメントも追加しました

5ky7 · 2026-03-27T17:26:35Z

300/sol1.py

+                if n > nums[j]:
+                    length_of_is[i] = max(length_of_is[i], length_of_is[j] + 1)
+
+        return max(length_of_is)


maxだと線形時間かかるので，length_of_is[-1]でも良さそうですね．
もっともループが多項式(2乗)時間なので線形時間ならボトルネックにはならず，大した改善にはならなさそうですが．

いえ、ここはlength_of_is[-1]ではなくmaxを取る必要があります

あくまでnが最後尾にくる部分列の最大長がlength_of_isに入っているので，おっしゃる通りmaxを取る必要がありますね．
失礼いたしました．

5ky7 · 2026-03-27T17:36:35Z

300/sol2.py

+    def lengthOfLIS(self, nums: List[int]) -> int:
+        tails = []
+        for n in nums:
+            lis_if_tail_is_n = bisect_left(tails, n) + 1


lisという文字からは，整数型ではなく配列（リスト）を想像しました．
len_lis_with_tail_nとするのはいかがでしょうか．

個人的にはpos_insert_nにして+ 1を外すのが好みです．

Suggested change

lis_if_tail_is_n = bisect_left(tails, n) + 1

pos_insert_n = bisect_left(tails, n)

if pos_insert_n == len(tails):

tails.append(n)

continue

tails[pos_insert_n] = n

そのまま採用させていただきました

nodchip · 2026-03-28T09:55:25Z

300/sol4.py

+            return bisect_left(coords, x) + 1
+
+        bit = BITMax(len(coords))
+        ans = 0


ans は設問に対する回答というニュアンスを感じました。変数に格納される値の意味や役割に寄せた命名のほうが良いと思います。 max_length はいかがでしょうか？

Geminiに伝えておきます
いや、読んで修正しなかった自分が気をつけようと思います

nodchip · 2026-03-28T09:58:50Z

300/sol3.py

+        def num_to_rank(n):
+            return bisect_left(nums_sorted, n) + 1
+
+        seg_tree = SegTree(len(nums))


こちらのコメントをご参照ください。
hemispherium/LeetCode_Arai60#10 (comment)

segment_treeの方が良さそうですね。極力省略しない、なかなか身につかないので気をつけます

mamo3gr · 2026-03-28T13:54:47Z

300/sol2.py

+
+class Solution:
+    def lengthOfLIS(self, nums: List[int]) -> int:
+        tails = []


3か月くらい後にこのコードを読むと、tails が何を表していて、なぜ問題が解けるのか読み解くのに苦労しそうです。何らかコメントでサポートできると良いと思います。

# tails[i] is the smallest ending element of all increasing subsequences of length (i + 1)
とコメントを追加しました

mamo3gr · 2026-03-28T14:01:47Z

300/sol3.py

+    def _to_leaf_index(self, index):
+        return index + self.offset - 1
+
+    def update(self, rank, value):


rank という概念は本問側のもので、それがこっちに染み出しているように見えます。

今回の問題に依存した変数名なのは改善すべきですね。
indexに変更しました。

tom4649 added 2 commits March 27, 2026 08:53

Add first step

76dd68a

Add BIT

247c00f

5ky7 reviewed Mar 27, 2026

View reviewed changes

Add suggested changes

7c0dc0c

nodchip reviewed Mar 28, 2026

View reviewed changes

mamo3gr reviewed Mar 28, 2026

View reviewed changes

Apply suggested changes

d2bf962

-            lis_if_tail_is_n = bisect_left(tails, n) + 1
+            pos_insert_n = bisect_left(tails, n)
+            if pos_insert_n == len(tails):
+                tails.append(n)
+                continue
+            tails[pos_insert_n] = n

Conversation

tom4649 commented Mar 27, 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!

5ky7 Mar 28, 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!

tom4649 Mar 28, 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

5ky7 Mar 28, 2026 •

edited

Loading

tom4649 Mar 28, 2026 •

edited

Loading