FC2ブログ

リファクタリング:制御フラグの削除

一連の論理型の式に対して制御フラグとして機能する1つの変数がある場合、
代わりにbreakかreturnを使います。

bool found = false;

for (int i = 0; i < characters.Length; i++)
{
if (!found)
{
if (characters[i].Job == Jobs.Hero)
{
Debug.WriteLine("発見しました。");
found = true;
}
}
}

for (int i = 0; i < characters.Length; i++)
{
if (characters[i].Job == Jobs.Hero)
{
Debug.WriteLine("発見しました。");
break;
}
}
リファクタリング―プログラムの体質改善テクニック (Object Technology Series)

Comment

Comment Form
公開設定

Trackback


→ この記事にトラックバックする(FC2ブログユーザー)