ループのコーディング
こういう書き方見かけるけど僕は嫌いだなー
どこが?といわれると、breakの位置。
for ( int i = 0; i < array.Length; i++ )
{
if ( array[i].no == no )
{
item = array[i].no; break;
}
}
処理を増やしたときにbreakに気づかないとこうなってしまう。
こうなるとa=b;の処理は行われません。
for ( int i = 0; i < array.Length; i++ )
{
if ( array[i].no == no )
{
item = array[i].no; break;
a=b;
}
}
って警告はでますけど・・・
warning CS0162: 到達できないコードが検出されました。
上の例は簡単だから気づくけど変数名が長くなって折り返してしまうと気づかない(warningもなぜwarningか気づきにくい)
と一行一処理を心がけましょう。
for ( int i = 0; i < array.Length; i++ )
{
if ( array[i].no == no )
{
item = array[i].no;
break;
}
}