if文をネスト（入れ子）させずにコーディングする方法

2020年4月29日2023年9月24日

当ページのリンクには広告が含まれています。

if文をネスト（入れ子）させずにコーディングする1つの方法です。
私が好んで使うパターンを紹介します。

基本的なパターン

1．デフォルト値を設定。
2．if文は、デフォルト値と異なる場合を判定して保持する。
3．処理は最後の1回だけとする。

ネスト（入れ子）した例

// 変数(Before, Afterで共通)
String sex = "man";
int age = 20;
String nationality = "Japan";

// Before : ifがネストした（入れ子状態になった）ケース
if(nationality != "Japan") {
    if(age < 20) {
        System.out.println("外国籍で未成年");
    }
}else {
    if(age >= 20) {
        if(sex.equals("man")) {
            System.out.println("日本国籍で成年男性");
        }else {
            System.out.println("日本国籍で成年女性");
        }
    }
}

// 変数(Before, Afterで共通)

String sex = "man";

int age = 20;

String nationality = "Japan";

// Before : ifがネストした（入れ子状態になった）ケース

if(nationality != "Japan") {

if(age < 20) {

System.out.println("外国籍で未成年");

}

}else {

if(age >= 20) {

if(sex.equals("man")) {

System.out.println("日本国籍で成年男性");

}else {

System.out.println("日本国籍で成年女性");

}

＜問題点＞
・条件判定がわかりづらい。
・表示の処理が判定ケース毎にあり、修正箇所が分散している。

ネストしていない例

// 変数(Before, Afterで共通)
String sex = "man";
int age = 20;
String nationality = "Japan";

// After : ネストしないケース
//// 初期値
String nationalityStr = "日本";
String ageStr = "成年";
String sexStr = "男性";
//// 判定
if(nationality != "Japan") nationalityStr = "外";
if(age < 20) ageStr = "未成年";
if(sex.equals("woman")) sexStr = "女性";
//// 表示
System.out.println(String.format("%s国籍で%s%s", nationalityStr, ageStr, sexStr));

// 変数(Before, Afterで共通)

String sex = "man";

int age = 20;

String nationality = "Japan";

// After : ネストしないケース

//// 初期値

String nationalityStr = "日本";

String ageStr = "成年";

String sexStr = "男性";

//// 判定

if(nationality != "Japan") nationalityStr = "外";

if(age < 20) ageStr = "未成年";

if(sex.equals("woman")) sexStr = "女性";

//// 表示

System.out.println(String.format("%s国籍で%s%s", nationalityStr, ageStr, sexStr));

＜メリット＞
・デフォルト値が必ず設定される。
・条件判定（if文）で設定値を個別に設定するので、単純になる。
・処理をまとめたことで修正が局所化された。

まとめ

・デフォルト値で処理する値を初期化する。
・設定値が変わる場合のみを、if文で個別に判定する。
・処理を1つにすることで、余分な記述を省き、修正箇所の局所化を実現できる。

サンプル

public class NotNesting {
    public static void main(String...strings) {

        // 変数(Before, Afterで共通)
        String sex = "man";
        int age = 20;
        String nationality = "Japan";

        // Before : ifがネストした（入れ子状態になった）ケース
        if(nationality != "Japan") {
            if(age < 20) {
                System.out.println("外国籍で未成年");
            }
        }else {
            if(age >= 20) {
                if(sex.equals("man")) {
                    System.out.println("日本国籍で成年男性");
                }else {
                    System.out.println("日本国籍で成年女性");
                }
            }
        }

        // After : ネストしないケース
        //// 初期値
        String nationalityStr = "日本";
        String ageStr = "成年";
        String sexStr = "男性";
        //// 判定
        if(nationality != "Japan") nationalityStr = "外";
        if(age < 20) ageStr = "未成年";
        if(sex.equals("woman")) sexStr = "女性";
        //// 表示
        System.out.println(String.format("%s国籍で%s%s", nationalityStr, ageStr, sexStr));

    }
}

public class NotNesting {

public static void main(String...strings) {

// 変数(Before, Afterで共通)

String sex = "man";

int age = 20;

String nationality = "Japan";

// Before : ifがネストした（入れ子状態になった）ケース

if(nationality != "Japan") {

if(age < 20) {

System.out.println("外国籍で未成年");

}

}else {

if(age >= 20) {

if(sex.equals("man")) {

System.out.println("日本国籍で成年男性");

}else {

System.out.println("日本国籍で成年女性");

}

// After : ネストしないケース

//// 初期値

String nationalityStr = "日本";

String ageStr = "成年";

String sexStr = "男性";

//// 判定

if(nationality != "Japan") nationalityStr = "外";

if(age < 20) ageStr = "未成年";

if(sex.equals("woman")) sexStr = "女性";

//// 表示

System.out.println(String.format("%s国籍で%s%s", nationalityStr, ageStr, sexStr));

}

この記事を書いた人

ログログ

リーマンショックの影響で26歳の時にIT業界から離れ、紆余曲折を経て34歳でエンジニアに復帰しました。
復帰前は開発未経験でしたが、独学した知識と面接時のコミュニケーション力で見事開発エンジニアとして復帰しました！
今はフリーランスエンジニアとして仕事をしています。

■保有資格
・Java Gold SE 11