変数は、見ただけで中身が分かる名前にすべきです。
よく、resultとかvalueとかdataとかreturnValueという変数名を見かけますが、これはあまり良くないネーミングです。
名前というのは、それ自体が読み手に対して重要な情報を伝える要素なのですが、どうもクラス名やメソッド名に比べて変数名は適当につけられがちな気がします。
プログラムを綺麗に書くコツやリファクタリングのノウハウなど
変数は、見ただけで中身が分かる名前にすべきです。
よく、resultとかvalueとかdataとかreturnValueという変数名を見かけますが、これはあまり良くないネーミングです。
名前というのは、それ自体が読み手に対して重要な情報を伝える要素なのですが、どうもクラス名やメソッド名に比べて変数名は適当につけられがちな気がします。
コメントが書かれている箇所と関係ない場所に修正が入ることによって、情報が勝手に古くなってしまうようなコメントは無意味です。
例えば、メソッドの呼び元のクラス名やメソッド名が書かれていることがあります。
正常に動きさえすればいい、というレベルの汚いソースコードをよく見かけます。
趣味で個人的にプログラムを書いているのなら、それでも構いません。
が、仕事として対価を受け取って書いているプログラムがそれでいいのでしょうか?
三項演算子は使いどころ次第で、可読性を大幅に上げることもあれば下げることもある諸刃の剣です。
たまに、「三項演算子は使うべきではない」という意見も見かけますが、私は一概に禁止すべきではなくケースバイケースだと考えています。
基本的に、不要になった処理をコメントアウトして残しておく必要性はありません。
一ヵ月後とか二ヵ月後とかの近い将来、確実に復活するのなら別ですが、そうでければ邪魔なだけなので潔く消してしまいましょう。