if-statement - グッドプラクティス：値をブール値と比較しますか？

Question

ほとんどの場合、私はこれを書くことを好みます：

if(isWelcome() == true){}
if(isWelcome() == false){}

これの代わりに

if(isWelcome()){}
if(!isWelcome()){}

読みやすいと感じているからです（でも意味がないことは理解しています）。

この慣行について共通の合意があるかどうか知りたいのですが。ほとんどの開発者は何をしますか？そして、コンパイラーが余分な比較を行っているのか、それともそれが役に立たないことを理解しているのか疑問に思います。

score 1 · Accepted Answer

Resharperのようなツールを使用する場合、isWelcome（）==trueを使用するのは冗長であるというヒントが得られます。コンパイラーは比較を最適化することを選択できるので、私はそれについてあまり心配しません。

score 1 · Accepted Answer

読みやすさは重要であり、単語を追加しても必ずしも読みやすさが向上するとは限りません。この間

if(isWelcome() == false){}

これより読みやすいかもしれません

if(!isWelcome()){}

何に慣れていない人のために！します、これ

if(isWelcome()){}

実際にはこれよりも読みやすいです

if(isWelcome() == true){}

これで、情報価値のない「ノイズワード」が導入され、何が起こっているのかを理解するために目がスキャンする必要のある領域に追加されたためです。プログラマーの意図を完全に伝える最小限のコードを目指して努力する必要があります。

実際にあなたの質問に答えるために、私はほとんどのプログラマーがより短いバージョンを使用すると言うでしょう。経験の浅いプログラマーは、「冗長」バージョンを使用する傾向があります。

2 に答える 2