【書き方】早期リターンの勘違いコード
言語は C# です。タイトルの通りですがif 文のネストが深くなると読みづらいので早期リターンを心がけましょうとアナウンスされ出現したコードこちら。
void hoge()
{
if(condition)
{
// 満たした時の処理
foo()
return;
}
// 満たしていない時の処理
fuga()
}
これは!…。早くリターンすれば良いだけと勘違いしているパターン。処理内容が分断されていて混乱する。この場合はreturn しない方が良いですよね。
void hoge()
{
if(condition)
{
// 満たした時の処理
foo()
}
else
{
// 満たしていない時の処理
fuga()
}
}