if文の良くないネストとか

gistfile1.java

/*
 * 例１
 */
if(hoge != null) {
	hogeList.add(hoge);
}

// 例１の全然アリな例①
if(hoge != null)
	hogeList.add(hoge);

// 例１の全然アリな例②
if(hoge != null) hogeList.add(hoge);

// 例１の絶対許せない例
if(hoge != null)
hogeList.add(hoge);


/*
 * 例２
 */
if(hoge != null) {
	if(fuga != null) {
		hogeList.add(hoge+fuga);
	}
}

// 例２のギリギリ許せる例
if(hoge != null)
	if(fuga != null)
		hogeList.add(hoge+fuga);

// 例２のギリギリ許せないこともない例
if(hoge != null)
	if(fuga != null) hogeList.add(hoge+fuga);

// 例２の絶対に許せない例①
if(hoge != null)
if(fuga != null)
hogeList.add(hoge+fuga);

// 例２の絶対に許せない例②
if(hoge != null) if(fuga != null)
hogeList.add(hoge+fuga);

// 例２の改善案①（条件をまとめてネストを減らす）
if(hoge != null && fuga != null) {
	hogeList.add(hoge+fuga);
}

// 例２の改善案②（↑ならワンライナーしてもいいんじゃね感）
if(hoge != null && fuga != null) hogeList.add(hoge+fuga);

/*
 * 例３
 */
if(hoge != null) {
	if(fuga != null) {
		hogeList.add(hoge+fuga);
	} else {
		hogeList.add(hoge);
	}
}

// 例３のギリギリ許せる例
if(hoge != null)
	if(fuga != null)
		hogeList.add(hoge+fuga);
	else
		hogeList.add(hoge);
		
// 例３のギリギリ許せないこともない例
if(hoge != null)
	if(fuga != null) hogeList.add(hoge+fuga);
	else hogeList.add(hoge);

// 例３の絶対に許せない例①
if(hoge != null)
if(fuga != null)
hogeList.add(hoge+fuga);
else
hogeList.add(hoge);

// 例３の絶対に許せない例②
if(hoge != null)
if(fuga != null) hogeList.add(hoge+fuga);
else hogeList.add(hoge);

// 例３の改善案①（条件をまとめてネストを減らす）
if(hoge != null && fuga != null) {
	hogeList.add(hoge+fuga);
} else if (hoge != null) {
	hogeList.add(hoge);
}

// 例３の改善案②（if-else文なら三項演算子でまとめられる場合も）
if(hoge != null) {
	hogeList.add((fuga != null) ? hoge+fuga : hoge);
}