下記のようなネストされたif文がある場合、深さ1にするうまいやりかたはありますでしょうか?
if(a == null){
if(b == "あああ"){
実行文
}
if(c == "いいい"){
実行文
}
}
スタック・オーバーフローはプログラマーとプログラミングに熱心な人のためのQ&Aサイトです。すぐ登録できます。
登録してこのコミュニティに参加する後続の処理がなければ、単純に以下のようにreturnしてしまいます
if (a != null) {
return;
}
if (b == "あああ") {
実行文
}
if (c == "いいい") {
実行文
}
後続の処理がある場合は、do-while文のbreakを使うことでif文のネストを減らせることがあります。
do {
if (a != null) {
break;
}
if (b == "あああ") {
実行文
}
if (c == "いいい") {
実行文
}
} while (false);
こんにちは、はじめまして。
ご提示していただいた例はわかりやすいコードだと思います。
私見ですが可読性のためには直さないほうがいいです。
ただ、Javaでの文字列の比較は、例外なくequalsを使ってください。
if(a != null){
if(b.equals("あああ")){
実行文
}
if(c.equals("いいい")){
実行文
}
}
&& でつなげて (a!=null && b.equals("あああ")) のようにすれば一文になりますがあとで読みたくなくなりますのでお勧めはしません。
また、ヌルチェックはどうしても避けたい、変数 a が関数の引数などである場合は @NotNull アノテーションを用いることでコンパイルチェックで確認できるようになります。確かに最近はこの方向性ですね。
public void yourCoolFunction(@NotNull final String a ) {
if(b.equals("あああ")){
//Implemetation
}
if(c.equals("いいい")){
//Implemetation
}
}