直感的なCコードを書こう

$Id: c.rd 13 2004-09-29 02:40:07Z $

え?なになに?これなにー? と思わせるようなコードは、良いCのコードとは言えません。 ぱっと見で、(他人が)直感的に理解できるコードを書けるようになりましょう。

線形サーチ

配列から空き状態にあるオブジェクトを探すコードで、以下のようなコードを見る事があります。

for (i = 0; i < arraysize; i++) {
  if (array[i].stat == FREE) {
    break;
  }
}
if (i < arraysize) {
  /* 見つかった */
} else {
  /* 見つからなかった */
}

このコードで、空きオブジェクトが見つかったかを判断するのに、

if (i < arraysize) {

という文を使っていますが、これはあまり直感的ではありません。 1つ変数を導入して、

for (i = 0, found = 0; i < arraysize; i++) {
  if (array[i].stat == FREE) {
    found = 1;
    break;
  }
}
if (found) {
  /* 見つかった */
} else {
  /* 見つからなかった */
}

のようにする方が、直感的です。