【書き方】早期リターンの勘違いコード

C#, DEVELOP

言語は C# です。タイトルの通りですがif 文のネストが深くなると読みづらいので早期リターンを心がけましょうとアナウンスされ出現したコードこちら。

void hoge()
{
    if(condition)
    {
        // 満たした時の処理
        foo()
        return;
    } 
    // 満たしていない時の処理
    fuga()
}

これは!…。早くリターンすれば良いだけと勘違いしているパターン。処理内容が分断されていて混乱する。この場合はreturn しない方が良いですよね。

void hoge()
{
    if(condition)
    {
        // 満たした時の処理
        foo()
    }
    else
    {
        // 満たしていない時の処理
        fuga()
    }
}

Posted by kazupon