Javaではif (isAdmin == true)というコードを書いてはいけない - Qiita

Deleted articles cannot be recovered. Draft of this article would be also deleted. Are you sure you want to delete this article?

[fusionstar]

「書いてはいけない」とまで言うひとがいたら「なるほどー(鳴き声)」とだけ答えてそっと距離を置くかな。 真偽型の変数には Is* や Has* という名前を付けようってのが先で、そしたら自然に == true は不要とわかるじゃない?

[otihateten3510]

俺わざと冗長に書くよ　すべてはリーダブルのために

[umai_bow]

「冗長」は悪いことじゃないし、「シンプル」は短いことではない。「if 文の中身は比較演算子を使う」の方がよっぽどシンプルなルール

[w1234567]

!flag派は堅物で融通が効かない危険な奴が多いのでチームで仕事をするときは地雷を踏まないように!flagを選ぶけど、本当はflag == falseのほうが好き

[vvakame]

これ、脳の使い方でいくらでも変化するのでチームメンバーみんなが困らない書き方に合意できたら後はそれ使っとけばいいと思う

[atsushieno]

こういう「どうでもよい、しかも（bytecodeだけ見てJIT結果を見ない）技術的に誤ったルール」を追加することによる弊害の大きさ（コーディングコストの増大）は計り知れない

[marshi]

ブコメのリーダブルだからっていうのはよくわからない。それは他の言語に慣れてるからそう感じるとか個人の主観によるものであって誰でもそう感じるものではないと思う

[sifue]

if文を「if (真偽値) 処理;」 ではなく「if (A == B) 処理;」 として覚えられてしまっている問題だ...。そもそもif文は論理分割するためのものだけどもこういうの見ると人類の敗北を感じる。

[lenore]

下のブコメにあるけど変数名がisTrueなわけないでしょ。isReadable, isPretty, hasPaid などちゃんと具体的に状態がわかる名称にするのが基本。flagとか超悪手

[deamu]

おっ、宗教戦争もうはじまってる？（のれんを上げて顔を出しながら

[pmint]

「冗長である」…それJavaで言っちゃう？ / 「代入と間違える」…条件書くところで代入するほうがアンチパターン。/ ただ単に「遅いから」と書けばいいことで、書いてはいけない理由には不足。

[miruzouq]

よーだとよぶのか“ヨーダ記法4で書くのは本末転倒”

[l-_-ll]

コメントしてる方々は相応の経験者だと思うけど答えはそれぞれ違う……惑うね

[unyaa]

仮に==trueだとしても、if(true==isXxx)だろうよ、 冗長だからこの書き方はしたくはないが、、

[Error401]

付いたコメントに従って、内容変更（変数名を変えたり、どの言語でも同じというのを取り消したり）をがんがんやってるので、なんとも微妙なことになってきた

[tankdesant]

Javaでの==比較って初心者がプリミティブ型以外で使ってバグを仕込む箇所だからレビューするときに身構えてしまうので極力使わないでほしい。

[Kil]

主要なコードチェックのデフォルト設定で検出される程度には「誤った記法」とされているんだから、書いてはいけない、と言い切って良いよ。オープンソースでコミットしようものなら小指で弾かれるレベル。

[caynan]

馬から落ちて落馬したって日本語で書いて問題ないよって人は、javaでも気にせず冗長に書けばいいんじゃないの

[OkadaHiroshi]

冗長だし僕は書かないけど、可読性を大きく下げるわけではないので許容範囲。

[luccafort]

タイトルの「 ○○してはいけない」というのが語彙が強い気はする。書いてることは何十年も言われてそうな話しだしコーディング規約やLint、レビューで直していけばいいってだけの話かなー。

[crevice]

自分は記事に同感（flagという変数名は同意しない）。isEnabled == trueが『リーダブル』とか『シンプル』とかという人もいるのだなぁ。人それぞれか…。

[kazunori-kimura]

元凶は適切な変数名を付けていないことにあると思われる。"if (isXxx)" と簡潔に書くために、boolean型の変数名は"isXxx", "hasXxx"のようにひと目でbooleanと分かるようにしよう、という趣旨なら理解できる。

[dagama]

細かいことの積み重ねとか言い出したらJavaで書いてること自体クソってことになるからやめようね

[uunfo]

if ((hoge == fuga) == TRUE)のやばみ

[suien42]

“isFoo ll !isBar” のような並びだと “!” を使うのはためらう。まず条件を見直すパターンが多いだろうけれど。

[junglejungle]

if文内の"="の誤記は今時のコンパイラならちゃんとワーニングが出る。!のつけ忘れはコンパイル時にワーニングが出ない。よって"=="のが良いとも言える。自分的にはif(isXXX)派だけども。

[mas-higa]

flag みたいな変数名だから伝わらない。if (available) とかにすれば?

[shoechang]

Javaは老害が多くて終わった言語だという感がひしひしと。IDEやコンパイラでどうとでも吸収できるので人間が頭使うところではない。

[dowhile]

真と比較するのを明示したい場合は書いた方がよいよ。Nullがある場合もあるし

[chimerast]

昔のC言語からプログラム書いてる人は if (flag == TRUE) は絶対悪と言われて育ってるので許してね。言語によってtrueの表現が違ったあの頃。

[sho]

炎上狙いっぽいタイトルだなーと思って読んでみたら案の定。「新人プログラマ応援」ってタグが虚しい感じ。

[aya_momo]

「代入と間違える可能性がある」代入で値を返すってのが。

[trade_heaven]

動けば何でもいい派なのでこういう人と仕事するとどうでもいい議論に時間を費やす機会が多くて疲れます。

[regularexception]

こんなことより、if(f) {return true;} else {return false;}みたいなコードのほうが嫌だ。return f; でええやん

[taruhachi]

ぺちぱーなので参加できない。

[rizmhate]

こんなんかくやついるか？

[ykhmfst2012]

宗教論争でしかない。愚かしい記事。

[osnq]

ブコメの==のどこがリーダブルなんだろう…

[Andrion]

Qiitaって最近クオリティ下がったよな。

[aiueo1145140]

書いてはいけないというのは言葉が強すぎるな、てっきり意図したとおりの挙動にならないのかと思ったわ