if分の最後のelseの中身

Question

class At{
	public static void main(String[] args){
		
		//1～20までループ
		for(int i = 1; i <= 20; i++){
		
			//!と?を入れるために初期化
			String mark = "";

			//2の倍数の時！を入れる
			if(i % 2 == 0){
				mark += "!";

			//4倍のとき？を入れる
			}else if(i % 4 == 0){
				mark += "?";
			
			}else{
				
			}

			System.out.println(i + mark);
		}
	}
}

if文の最後の文はelseで終わらなければならないのは承知なのですか、中身に何も入らない場合は、この書き方でいいのでしょうか？

Answer 1

最後のelseは必要なければ書かなくていいです。

else{
}

Comments

[Tutti]

よかったです！ありがとうございます！！！！！

Answer 2

文法上はTuttiさんの書き方で問題ないですし、kopioさんが指摘したとおりでも問題ないです。

しかし、ソースコードレビューでは指摘が入ると思います。なぜなら、「何もやらないから意図的に空である」のか、「うっかりミスで空になっている」のか、後から読んだ人にとっては区別がつかない。

そこで、そこには何らかのコメントを記載しておくべきです。以下のように:

} else {
    // Do Nothing
}

Answer 3

最後必ずelseで終わらないといけないというのははじめて聞きましたが・・・そのコーディング規約ではどのような理由なんでしょうかね。
審美的な理由ですかね。
都市伝説的理由でしょうか。
真面目に考えると、elseのケースもちゃんとカバレッジされているされていることを確認するためでしょうか。

奇数のときはエラーにする、などの処理があるのであればelseは必要でしょうが、何もやることがないならelseは必要ないと思います。

if文がネストしていくと、必ず空の

} else {
  // do nothing
}

などと記述しているとコードが冗長になって見にくくなるなるだけだと思います。