83. Remove Duplicates from Sorted List

[xbam326]

今回解いた問題

https://leetcode.com/problems/remove-duplicates-from-sorted-list/description/

次回解く予定の問題

Remove Duplicates from Sorted List II

[nodchip]

プルリクエストのタイトルに問題タイトルと問題番号、 Description に問題へのリンクを記入すると、レビューワーにとってどの問題に対するコードか分かりやすくなったり、問題を参照しやすくなったりすると思います。

[nodchip]

ほかの方のソースコードを読んだ際、そのコードのリンクと読んだときの所感をプルリクエストに添えると、レビューワーにとって有益な情報となると思います。

[xbam326]

ありがとうございます。
step2で他の方のソースコードを確認するので次回以降、読んだソースコードと所感をmemo欄に付け加えるようにいたします。

[oda]

うーん。current は私がかなり嫌なんですね。

名前からなんらかの役割やコンテキストが推測できて欲しいのに、current は「自分が今注目しているもの」という程度の意味合いしかここではないので、まったく読む人にとってありがたくないのです。意味を乗せる気がないならば一文字変数のほうがましです。意味を乗せる気がない捨てていい変数だという意味が乗るからです。

逆に、current をプロダクションレベルのコードで使うときには、「現在の設定やスレッド」などのようにグローバルな状態に近い意味で使うことが多いです。

[xbam326]

プルリクエストのタイトルに問題タイトルと問題番号、 Description に問題へのリンクを記入すると、レビューワーにとってどの問題に対するコードか分かりやすくなったり、問題を参照しやすくなったりすると思います。

titleとdescriptionの記載が漏れていました。
次回以降titleとdescriptionが漏れないようにPRを作成するスクリプトを書いておこうと思います

[mamo3gr]

特に違和感ありませんでした。

（このPRだけの問題ではないのですが）命名はもうひと詰めできそうな気がしています。
current は情報が少ないですが node も同じようなものだと思います（そもそもノードを走査する処理なのでほぼ自明）。#5 (comment) でいう「意味を乗せる気がない捨てていい変数」のつもりで node を使う、が現状ベターなんですかね。