「if（0 == value）…」は、害よりも害をもたらすのではないでしょうか？[閉まっている]

これは、他の誰かのコードで見たときに最も嫌いなものの1つです。私はそれが何を意味するのか、なぜこのようにするのかを知っています（「誤って代わりに「=」を入れたらどうなりますか？」）。私にとっては、子供が階段を降りて階段を数えて声を出しているようなものです。

とにかく、これに対する私の議論は次のとおりです。

• プログラムコードを読み取る自然な流れを混乱させます。私たち人間は、「値がゼロの場合」ではなく「値がゼロの場合」と言います。
• 最新のコンパイラーは、条件に割り当てがある場合、または実際には条件がその割り当てだけで構成されている場合に警告を発します。
• プログラマーである場合、値を比較するときは二重の「=」を忘れないでください。「！」の入力を忘れることもあります。不平等をテストするとき。

ああ、はい、「ヨーダの条件」（「値がゼロの場合、このコードを実行する必要があります！」）。私は、lint（1）のようなツールで「より良い」と主張する人を常に指します。この特定の問題は、70年代後半から解決されました。ほとんどの現代言語はif(x = 10)、ブール値への代入の結果を強制することを拒否しているため、のような式さえコンパイルしません。

他の人が言ったように、それは確かに問題ではありませんが、それは少しの認知的不協和を引き起こします。

それは小さいが目立った精神税を課すので不快です。

ほぼすべてのプログラミング言語（およびほとんどの自然言語）で左から右に読みます。

を見る123 == xと、私が精神的に解析する方法は次のとおりです。

• 123- だから何？不完全な情報。
• == -まあ、123は123です、なぜそれをテストします...
• x-OK、それが私たちが心配していることです。今だけコンテキストがあります。
• もう一度考え直して123、なぜxと比較するのかを考え直してください。

x == 123メンタルパーシングを見ると：

• x-コンテキストを提供し、条件が何であるかを知っています。残りを無視することもできます。前のフローに基づいて、私はなぜ、そして何が来るのかについて良いアイデアを持っています（そして、異なる場合は驚かれます）。
• == - 私はそうだと思いました。
• 123 - うん。

混乱はわずかですが（単純な例では）、私はいつもそれに気付きます。

たとえば、値に注意を引きたい場合は、値を最初に置くことをお勧めしますif (LAUNCH_NUKES == cmd)。通常、これは意図ではありません。

有害ですか？いいえ。どちらの方法でも機能します。

悪い練習？せいぜい議論の余地がある。単純な防御的なプログラミングです。

寝る価値がありますか？いや

これは基本的にフレイムベイトです。

いいえ、それは良いことよりも害はありません。シンプル。

もっと言葉？

コンパイラー引数？えー、いや、多分-自分自身からあなたを救うためにコンパイラーに過度の信頼を置かないでください。

「あなたは忘れてはならない」 -まあ当たり前 -もちろん何をあなたはその間、私は疲れているべきではない、私は人間の私のメイクされ、時には、ちょうど時々 2つの異なる言語を使用していたとしたすべての日をコーディングしないてきました間違い。

この種の動作のポイントは、あなたがクラッシュすることを期待しているために保険を持っているよりも間違いを犯すことを期待しているため、防御的であり、そこにないということです...

読みにくい？あなたは、まともなプログラマーが==ハードワイヤード（すべての種類の悪い仮定をする）を持つべきであるが、同じまともなプログラマーは0 == valueを読むことができないと文句を言っていますか？

害はなく、潜在的な利点があり、ばかげた質問があり、他の人が望むなら先に進んでください。

私はそれを害とは言いませんが、それは不快です。だから私はそれがそうだとは言いません。

「=を忘れたらどうなる？」これまで本当に多くの重量を保持していました。はい、あなたはタイプミスをするかもしれませんが、私たちはすべてタイプミスをします。間違いを犯すことを恐れているので、コーディングスタイル全体を変えるのはばかげているようです。いつか何かを大文字にしたり、アンダースコアを忘れたりするかもしれないので、すべての変数と関数を句読点なしですべて小文字にしてみませんか？

一部の人々は、条件文が何をしているのかを明確にするためにそれを使用しています。例えば：

方法1：

FILE *fp;

fp = fopen("foo.txt", "w+");
if (fp == NULL) {

方法2：

FILE *fp;

if (NULL == (fp = fopen("foo.txt", "w+"))) {

一部の人々は、2番目の例がより簡潔であると感じている、または逆の議論がテスト自体の前のテスト（条件付き）のポイントを示しています。

実際のところ、私はどちらの方法でも構わない。私はスタイルについてのペットのぞき見を持っていますが、最大のものは矛盾です。だから、同じように、一貫して、あなたのコードを読むことを気にしません。

独自のスタイルを持つ6人の異なる人々が一度に取り組んでいるように見えるまで、それを混ぜて、私は少しイライラします。

私にとって、それは単純な調整です。（90年代に）CとC ++を学んだ人として、私はそれに慣れて、今でもそれを使っています。理由は教訓になっています。

「定数」の左側を見るために「条件付けられ」たら、それは第二の性質になります。

また、これは等価（または等価の否定）にのみ使用し、より大きい/より小さいには使用しません。

@Wonkoの答えに完全に同意します。

私が役立つと思うのは、ifの変数部分が非常に長く、値を見るとコードが読みやすくなる場合です。点線の名前空間言語には、この最高の例があります。

たとえば、ある種のエラーが発生し、特定の方法で回復された場合、2つの同時セッションが発生する可能性がありました。このようなもの：

if (2 <= application.httpcontext.current.session["thenameofmysessiontoken"].items.count())

確かにこの例にはこれを行う他の方法がありますが、これはナンバーファーストバージョンが潜在的に読みやすい場合です。

それでも、エラーが発生します。また、ループ演算子で割り当てが必要な場合があります。そうでない場合は、等価性をチェックするか、少なくともそれを使用する標準的な方法です。

（おそらくCode Completeから）従ったアドバイスは、比較で左側の低い値を維持することです。私は以前同僚とこれについて話し合っていましたが、彼はそれがちょっとおかしいと思っていましたが、私はそれに非常に慣れました。

だから私は言うだろう：

if ( 0 <= value )

しかし、私も言うでしょう：

if ( value <= 100 )

平等私は左の変数でチェックする傾向がありますが、それはより読みやすいです。