3. Longest Substring Without Repeating Characters #50
Conversation
| class Solution: | ||
| def lengthOfLongestSubstring(self, s: str) -> int: | ||
| start = 0 | ||
| stop = 1 |
There was a problem hiding this comment.
stop より last の方が実態を表していてより適切と書籍（リーダブルコード）で読んだことがあります。
| for left in range(s_len): | ||
| for right in range(left, s_len): |
There was a problem hiding this comment.
自分なら文字ではなくインデックスであることを強調するために left_i 、 right_i としますね。
There was a problem hiding this comment.
#35 (comment)
このようなコメントをいただいた事があり、以降消すようにしています。
| return max_len | ||
| ``` | ||
| 思考ログ： | ||
| - ややテクニカルだが効率はいいなと感じた |
There was a problem hiding this comment.
文字とインデックスの辞書を作成してそれを参照する方法なので、個人的にはそこまでテクニカルではなくオーソドックスな方法だと思いました。
There was a problem hiding this comment.
自分には、最後に見たインデックス情報を活用するこの方法がすぐ出てきませんでした。。
現在のleftより前に戻らないようにする部分も初手でハンドリングできていたか怪しいです。
精進致します。
| substring_chars = set() | ||
| longest_substring_len = 0 | ||
| while stop <= len(s): | ||
| c = s[stop - 1] |
There was a problem hiding this comment.
好みの範疇かもしれませんが、stop (last) のほうが、それを含まないインデックスであるというのは少し違和感があります。
二分探索だと右端を含むか含まないかというのはむしろ自由に書き換えられたほうがよいとは思うのですが、ここは文字列のそのインデックスにアクセスしているので、s[stop] になるように書くのが自然な印象です (そうすると長さを求める部分は stop - start + 1 になりますね)。
| for left in range(s_len): | ||
| for right in range(left, s_len): |
| longest_substring_len = 0 | ||
| for left in range(s_len): | ||
| for right in range(left, s_len): | ||
| sub_s = s[left:right + 1] |
| ```python | ||
| class Solution: | ||
| def lengthOfLongestSubstring(self, s: str) -> int: | ||
| letter_to_count = {c: 0 for c in s} |
| class Solution: | ||
| def lengthOfLongestSubstring(self, s: str) -> int: | ||
| letter_to_count = {c: 0 for c in s} | ||
|
|
| ``` | ||
| 思考ログ： | ||
| - ややテクニカルだが効率はいいなと感じた | ||
| - leftを更新する際に現在のleftより前に戻らないようにすることを注意しないといけない |
|
|
||
| # Step3 | ||
|
|
||
| かかった時間：4min |
There was a problem hiding this comment.
これ 3 回やって 4 min ですかね？突っ込むほどのこともないですが、もうちょっと早くなる気もしており、いや別に早くすることが目的ではないんですが、なにか迷いがあるのかなあ、と思いました。
There was a problem hiding this comment.
確かにと思い、解き直してみましたが、2分程かかりました。
割と解きながら他の選択肢について思い出していたりすることがあるので、それで余計な時間を使ったのかもしれません。
| letter_to_count[s[right]] += 1 | ||
| max_len = max(max_len, right - left + 1) |
There was a problem hiding this comment.
細かいですが、自分ならこの２つの行逆にします。(for文の最後でrightが進むことに関係する部分は動かしたいため)
There was a problem hiding this comment.
確かにそうですね。
忘れそうだと思って先に書いてしまったんですかね。
| if s[right] in letter_to_last_index: | ||
| left = max(left, letter_to_last_index[s[right]] + 1) | ||
|
|
||
| max_len = max(max_len, right - left + 1) |
There was a problem hiding this comment.
最大の長さであることはは読んでいれば割とすぐわかる情報で、むしろresultの方が何についての最大の長さなのか関数名を見ればわかるので(関数名がはっきりしていたら)、個人的には好みです
| for left in range(s_len): | ||
| for right in range(left, s_len): |
|
|
||
| 文字列の長さをNとして | ||
| - 時間計算量：O(N) | ||
| - 空間計算量：O(N) |
There was a problem hiding this comment.
かなり前のPRへのコメントで恐縮ですが、この問題の制約に
sconsists of English letters, digits, symbols and spaces.
とあるので、入力の文字列がめちゃめちゃ大きくなったとしても、substringはあるサイズで頭打ちになる -> 空間計算量はO(1)な気がします。
https://leetcode.com/problems/longest-substring-without-repeating-characters/description/