【ゲーム開発のためのC#入門講座・おまけ編】ガード節を使ってネストを減らそう【#1】

/*
    ネストというのはスペースを使ってコードの記述を入れ子にすること。
    例えば下記のコードは二つのif文で成り立っているが、
    ふたつ目のif文は先頭にスペースがあることで、
    どこからどこまでがふたつ目のif文の範囲なのかがわかりやすい！
*/
if (text == "hoge")
{
    if (result == true)
    {
        ……処理……
    }
}
​
// ↓こんな風にネストがないと、条件の範囲が分かり辛いのだ！
if (text == "hoge")
{
if (result == true)
{
……処理……
}
}

public void CreateNewLine()
{
    if (character != "、")
    {
        if (character != "。")
        {
            if (character != "ゃ")
            {
                if (character != "ゅ")
                {
                    if (character != "ょ")
                    {
                        if (character != "っ")
                        {
                            if (character != "ぁ")
                            {
                                if (character != "」")
                                {
                                    ...改行処理...
                                }
                            }
                        }
                    }
                }
            }
        }
    }
}

public void CreateNewLine()
{
    if (character != "、")
    {
        if (character != "。")
        {
            // ↓実はここだけ「==」
            if (character == "ゃ")
            {
                // ↓何か毛色の違う条件
                if (result == true)
                {
                    // ↓何か毛色の違う条件2
                    if (partyName != "data")
                    {
                        if (character != "っ")
                        {
                            if (character != "ぁ")
                            {
                                if (character != "」")
                                {
                                    ...改行処理...
                                }
                            }
                        }
                    }
                    else
                    {
                        // ↓ここだけelse処理
                        ……異なる処理……
                    }
                }
            }
        }
    }
}

if (条件A)
{
    if (条件B)
    {
        ……処理C……
    }
}

// 1. 条件Aじゃなかったら処理Cは実行しない
if (!条件A)
{
    return;
}
​
// 2. 条件Bじゃなかったら処理Cを実行しない
if (!条件B)
{
    return;
}
​
// 3. 1と2以外（＝条件AかつB）だったら処理Cを実行する
……処理C……

public void CreateNewLine()
{
    if (character == "、")
    {
        return;
    }
    
    if (character == "。")
    {
        return;        
    }
    
    if (character == "ゃ")
    {
        return;            
    }
    
    if (character == "ゅ")
    {
        return;               
    }
    
    if (character == "ょ")
    {
        return;                 
    }
    
    if (character == "っ")
    {
        return;             
    }
    
    if (character == "ぁ")
    {
        return;                            
    }
    
    if (character == "」")
    {
        return;
    }
    
    ...改行処理...
}

// 次の文字が「、」じゃなくて、
if (character != "、")
{
    // 次の文字が「。」でもなくて、
    if (character != "。")
    {
        // 次の文字が「ゃ」でもなくて……
        if (character != "ゃ")
        {

// 次の文字が「、」だったら対象外。次！
if (character == "、")
{
    return;
}
    
// 次の文字が「。」だったら対象外。次！
if (character == "。")
{
    return;        
}
    
// 次の文字が「ゃ」だったら対象外。次！
if (character == "ゃ")
{
    return;            
}
​
// と、条件毎に考えを区切ることができるので、情報の整理が楽！

// ↓これも悪いコードではないけれど……
if (!条件A)
{
    return false;
}
​
if (!条件B)
{
    return false;
}
​
return true;
​
// ↓素直にこう書いた方がわかりやすいことも！
if (条件A && 条件B)
{
    return true;
}
{
    return false;
}

for (int i = 1; i < 100; i++)
{
    if (i % 3 != 0) continue;
    if (i % 10 != 0) continue;
    Console.WriteLine(i);
}

public void CreateNewLine()
{
    if (character == "、") return;
    if (character == "。") return; 
    if (character == "ゃ") return; 
    if (character == "ゅ") return; 
    if (character == "ょ") return; 
    if (character == "っ") return; 
    if (character == "ぁ") return; 
    if (character == "」") return; 
    
    ...改行処理...
}

if (条件) 条件を満たす場合に実行する1行のみの処理;

public void DammyMethod()
{
    // ↓混在していると読み辛い……
    if (character == "、") return;
    Console.WriteLine("hoge");
    if (isCompleted) Console.WriteLine("fuga");
    Console.WriteLine("hogehoge");
    if (isCanceled)
    {
        Console.WriteLine("Cancelされました");
        return;
    }
    ...改行処理...
}