【必修】読みやすいソースコードのポイント紹介【エンジニア】

[@keny4472]

>冗長な処理は排除する。
冗長処理は排除は、ご説、ごもっとも思いますが、本件では反って可読性が下がったと思います。　(^^;
原式(0)で、(1),(2)にするほうがelse式の場合はよいように思います。
---------------------------------------------------(0)　原式
String str = "";
Random random = new Random();
int randomValue = random.nextInt(100);
if(randomValue < 50) {
str = "50未満";
System.out.println(str);
} else {
str = "50以上" ;
System.out.println(str);
}
---------------------------------------------------(1)　コードの対称性
String str = "";
Random random = new Random();
int randomValue = random.nextInt(100);
if(randomValue < 50) {
str = "50未満";
}else {
str = "50以上" ;
}
System.out.println(str);
---------------------------------------------------(2)　宣言と利用の近傍化
Random random = new Random();
int randomValue = random.nextInt(100);
String str = "50以上" ;
if(randomValue < 50) {
str = "50未満";
}
System.out.println(str);

[@higuma08]

コメントありがとうございます。
ご指摘の通り、str変数の定義と内容設定は近づけた方が読みやすいですね…
以後気を付けます！ご指摘ありがとうございます。

[@keny4472]

@@higuma08
難しい分野に思いますが、頑張って下さい！ ( ´ ▽ ` )ﾉ