Remove duplicates from sorted list ii

[ask-1407]

問題：https://leetcode.com/problems/remove-duplicates-from-sorted-list-ii/

考えたこと：https://github.com/ask-1407/leetcode/blob/remove_duplicates_from_sorted_list_ii/Arai60/LinkedList/answer.md

次に解く予定：https://leetcode.com/problems/add-two-numbers/

[hroc135]

実際はO(n)でできているのですが、whileが二重になっているのでぱっと見O(n^2)になっていることがなんとなく気になります。二重whileを使わない方法としては、duplicatedなどのフラグを用意してやるとできると思います。

[nodchip]

sentinel という単語を使うのも良いと思います。
hroc135/leetcode#4 (comment)

[nodchip]

if 文の条件が True のときと False のときのどちらの時に値が重複しているのか、直感的でないように感じました。また、変数を previous_node と current_node の 2 つ使うのではなく、 1 つだけにしたほうがシンプルだと感じました。以下のようなコードはいかがでしょうか？

node = dummy
while node and node.next and node.next.next:
    if node.next.val != node.next.next.val:
        # 重複していない場合
        # early return する。
        node = node.next
        continue

    # 重複している場合
    value = node.next.val
    while node.next and node.next.val == value:
        node.next = node.next.next

[ask-1407]

たしかにご提示のコードのほうが直感的だなと感じました。

[sasanquaneuf]

currentの初期化時にcurrent = previous = dummyとしておくと、walrus演算子を使ってcurrent := current.nextと書くことができ、ループ変数を進める処理をわかりやすくできます（python3.8以降）
好みもありますが参考まで。

[ask-1407]

walrus演算子で変数への代入と変数の使用を同時に行えるのですね。適切に使えるとコードをすっきりできそうだなと思いました。ありがとうございます。

[sasanquaneuf]

Step1の指摘と類似ですが、ここの分岐は読み飛ばしが発生したかしていないかなので、もう少しわかりやすくできると嬉しいですね。前のwhileの条件と逆になるのが見にくいのかも？と思いました。
代入回数が増えてしまいますが、elseを省略することはできます。

            while current.next and current.val == current.next.val:
                current = current.next
                previous.next = current.next
            if previous.next == current:
                previous = previous.next

[ask-1407]

ブロックがすくないこちらのほうが私も見やすいと感じました。

[Mike0121]

最適解ではありませんが、自分はHashMapでカウントしていく解法でも解きました。
Mike0121/LeetCode@5e139d5

[ask-1407]

ありがとうございます。自分では思いついていなかったのでこちらも読み込んでみます。