IF文における条件の書き方

Question

if(flg == true){
（trueの場合の処理）
}

とJavaでは上記のように書くのは良くないというふうなことを聞きました。
本当でしょうか。

またPHPやJavascriptも同じことが言えるのでしょうか？

Answer 1

Javaの場合、if文のカッコ内にはtrueあるいはfalseを取る条件式しか置けませんので、==trueと比較してコンパイルが通るようなboolean型の変数であれば、カッコ内にそのまま置いても動作は全く同じですし、逆にそうでなければ型の一致しない比較になってコンパイルが通りませんので、付ける意味はゼロです。

JavaScriptやPHPの場合は正反対に、if()のカッコ内にはなんでも置けますが、これらの言語ではif($a)とif($a==true)で意味が違ってきますし、==では曖昧な比較になるので、逆の意味で注意が必要です（厳密にtrueかどうかを調べる必要がある場合、if($a===true)とすればそう動きます）。

Comments

[karain]

そのような差があるとは、、、勉強になりました。

Answer 2

他の言語は分かりませんが、javascriptではこんなことが起きます。

function test(flg){
  if(flg  == true){ alert(flg.toSource() + ' is true !') }
}
test(true); //"(new Boolean(true)) is true !"
test(1); //"(new Number(1)) is true !"
test('1'); //"(new String("1")) is true !"
test([1]); //"[1] is true !"

比較演算子 === を使うことによって型を含めた厳密な比較をしてくれます。

Comments

[karain]

===にするか、どうかでも大きな違いがあるんですね、、、うまく使い分けるようにします。

Answer 3

if()構文自体が、真偽値を問う構文ですので、以下の場合は、そもそも比較する必要もなくtrue/falseです。

var hoge = true;
if (hoge) {
  // true
}
if (!hoge) {
  // false
}

これを以下のようにすることで、無駄な比較が行われます。

var hoge = true;
if (hoge == true) {
  // true
}
if (hoge == false) {
  // false
}

これは他の言語でも同じです。
ただし、他の方の回答にもある通り、===演算子を利用することで厳密な比較を行います。
これも、JavaScript、PHPともに同じような意味合いを持ちます。
厳密に比較する必要がある時を除いて、わざわざ比較する必要はありません。

例えばJavaScriptでいえば、以下のような場合が問題視されるでしょう。(PHPも似た感じだったはず)
こういう、文字と数値が混合してたり、選択されてない場合は空白値、とかってのは時々ありますよね。

test("");   // false
test(0);    // false
test("0");  // true
test(1);    // true
test("1");  // true

function test(value) {
  if (value) {
    alert("true");
  } else {
    alert("false");
  }
}

Answer 4

if(flg == true)と書くときに、下のように書き間違えてしまう可能性があるためif(flg)と書いたほうがよいという解説を読んだことがあります。

// この場合flgにtrueが代入された後、()内が評価される。
// このため、常にtrueの場合の処理が実行されてしまう
if(flg  = true){ 
 （trueの場合の処理） 
} 

Answer 5

既に回答頂いている他に、質問文にあるような記述が良くない理由としてよく挙げられるのが

if (flg == true)のような比較を書く際に、間違えて
if (flg = true)と書いてしまうことがあるということですね。
この場合、代入文になってしまうため、意図しない動作になる可能性が高いです。

これを防ぐため、if (true == flg)というように、
比較演算子の前後を入れ替える記述が推奨されている場合もあるようです。

参考まで