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

テクノロジーカテゴリーの変更を依頼記事元:

370 usersがブックマークコメント

183

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

Deleted articles cannot be recovered. Draft of this article would be also deleted. Are you sure y... 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 は不要とわかるじゃない?

2018/02/20 リンク

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とか超悪手

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

2018/02/27 リンク

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

2018/02/21 リンク

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

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

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

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

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

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

katsyoshi flagにtrue false意外が入ってるとブチ切れる皆さん（flagがよくない名前であるのはわかるが、この例で変数名考えるのはアホくさい

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

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

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

programming

はてなブックマーク

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

記事へのコメント183件

関連記事