初心者が書くコードのパターン

Java

コードレビュー。

boolean flag = this.check();
if (flag == true) {
    ....
}

うーん、どうしてこう書けないのかな。

if (!isValid()) {
    ....
}

ポイントは五つ。

無意味な変数を定義しちゃう

その後使用する予定がない変数は定義しないで欲しいな。

不明確な変数名前

とにかく boolean なら何でも flag。flag2 とか flag3 とか。

this の乱用

"this." って打つと、後は IDE がコードアシストしてくれるから、何でもかんでも this. が付いてる。

不明確なメソッド名

check() というメソッドは、チェックしてどうだったらどんな動作をするのかまったくわからない。

boolean に対する == 判定

boolean と boolean を比較して boolean の結果を得るのは意味が無いんだけど、初心者レベルでは可読性が上がるのだろうか・・・？