Java初心者時代にコードレビューで指摘された悪しき習慣 - Qiita

Java初心者だった新入社員の頃、先輩にコードレビューで指摘された事を思い出してまとめてみた。 追記：本記事に関しては賛否含め、多くの有益なコメントを頂いています。本記事をお読みになる際は、是非コメント欄も併せてご覧下さい。 2018/04/26 コメントを参考に「何でも定数にしようとする」の見出し・本文を修正しました。@kagilinn さん、ありがとうございました。 2018/04/30 サンプルコードの判定バグってたので修正しました。@y_miz さん、ご指摘ありがとうございました。 コメントの誤記、用語誤りを修正しました。@scivola さん、編集リクエストありがとうございました。 不要なインスタンス変数を作ってしまう インスタンス変数は状態が保持されるので、バグを作り込みやすい。 「これローカル変数でよくない？」ってよく指摘された。 インスタンス変数を作る前に、ローカル変数で実

[nobeans]

いいぞもっとやれ、と思ったらコメントで「変数先頭宣言派」「最後にreturn派」の人が居て言葉を失う。 未だそういう派閥が現存してしかもこういったところでコメントする程度にはアクティブだというところに驚愕

[ryotkn]

コメントの議論が秀逸

[knjname]

この記事自体まともだと思うので、コメント欄にある、一種の意見は聞かなくていいと思う。

[slkby]

簡素な言語でコモンセンスを身に着けないとコードが汚い状態を認識できなくなるから初心者にjavaをオススメしちゃだめだぞ (宣戦布告)

[nkgr]

宗教戦争にならない程度にまとまってていいと思う。この手の話題はコメント欄が活発だ。

[yuki_2021]

Javaの話だけど他言語でもリファクタリングで使えそうだ。

[y___u]

某SIerさんの設計書を元にコーディングしてる部隊にも熟読して欲しいやつだ

[perl48]

Java独習なう、な私にとってはヨダレが止まらない情報。特にコメント欄がイイ。

[y_yuki]

昔、ZERO=0 (20まで続く)みたいな定数が大量にあるコードを見せられたことを思い出した

[rAdio]

『新人プログラマが良いコードを書けるようになる為には、本人の努力はもちろんだが、優秀なプログラマからコードレビューを受けられるかどうかが重要』…これが最も高難易度。

[hundret]

Java初心者時代にコードレビューで指摘された悪しき習慣 on @Qiita

[darupants]

[java]

[hitsujiniku-sugita]

“優秀なプログラマからコードレビューを受けられるかどうかが重要”