「ついつい使ってしまうプログラミングの悪いテクニックは？」記事へのコメント

カンマ式、3項演算子の乱用

[Anonymous Coward]

式と文がごっちゃになってて言語デザイン的に気持ち悪い構文だけど、
コードがウダウダと長くなりそうな場面ではつい使ってしまう。

Re:カンマ式、3項演算子の乱用

[Anonymous Coward]

カンマ式、3項演算子もそうだけど、短絡演算子使いがち。
if (a()) { b(); } else { c(); }
の代わりに
a() && b() || c();
みたいな。

Re:カンマ式、3項演算子の乱用

[Anonymous Coward]

で、「b()がエラー返すとなぜかc()も実行される！」と大騒ぎ…。

A「"なんで素直に3項演算子を使わないんですか", a() ? b() : c();」
B「 "elseが要らない時もあるから…";」
A「a() ? b() : (void)0, "で良いでしょうが";」
B「"それなら", a() && (b(), true) || c(), "の方が素直かなー";」

# Lispが俺にもっとカッコ付けろと囁いている

Re:

[Anonymous Coward]

それって、b()やc()の戻り値がvoidではダメなんじゃないかな。破壊的操作を期待しているなら、if文で書いてもらった方がその意図が明確になるし、戻り値を使うなら、三項演算子の方が分かりやすいよね。

if (a()) {
  d = b();
} else {
  d = c();
}

よりも

d = a() ? b() : c();

の方が、この処理によってdの値が変化するということが明確だし、代入先がdからeに変わっても、1箇所書き換えるだけで済む。