質問

Checkstyle苦情に以下のもの:

return (null == a ? a : new A());

するものとしてのパーは必要ありません。

の声明確実に動作しないようで遠く読みやすくするためにその存在---などいくつかについて触れたいと思いますが、見

return null

最初において一時停止の残存

== a ? a : new A(); 

続けるために、シェフラーグルー私の脳がいないとのことです。

また、私が同じことを いただternaryオペレーターは実際のものと異なる場合がありグルーピングパー.

い:べきパーの三元系のデファクトスタンダード?はありません入ってありますか?

役に立ちましたか?

解決

まあ、Checkstyleは、右括弧は、実行のために役に立たないです。しかし、実行のために役に立たないはあなたのコードの良い読書のために役に立たないという意味ではありません。それは読むために、より理にかなっている場合は、それらを残す必要があります。

私が思うに、このコードは、より多くの必要はありません括弧ます:

int number = (myBoolean)? 1 : 2;

が、あなたの場合はreturnキーワードとあなたのブール式あなたが文を読む方法を変更することができているという事実ます。

他のヒント

return文を読んだとき、私は「リターン」との間のすべてを知っています「;」返却されようとしているものですので、あなたがそれを読んで主張するようリターンnullは一部の記号が続くように私はあなたのコードサンプルを読むことができる方法はありません。

おそらく技術を解析する上に読んで、私はそうであるようにあなたがそれを見るために役立つかもしれません。それは私が長年にわたって一緒にいくつかのパーサを石畳ましたけれども、私は本当に、パース技術上に読んでいない、と述べています。

私はいつも、不要な括弧を削除します。私はかなりよくJavaの演算子の優先順位を知っている彼らは、コードの理解を支援しません。私はわからないよ奇数時間は、私は、括弧を追加し、IDEAは、彼らしている冗長な私に指示するかどうかを参照するのを待ちます。その後、私はそれらを削除し、メモリに私が発見した優先順位のルールをコミットしようとします。

Iが継承コードベースで、私は私は2つを関連付けるように、他の理由のために貧しいコードの領域に冗長な括弧の最大数を見つける傾向がある。

いいえ、それは事実上の標準ではありません。私は括弧なしでそれを好むます。

私はそれらを置くための唯一の理由は、評価の順序を強制したり、混乱ラインを明確にすることがあると思います。

両方のオプションが正しい、使用どのようなあなたのチームの用途または何あなたのようなあなたは一人で作業している場合。
デフォルトのCheckstyleによって
IIRCは、あなたがそれに耳を傾け、括弧を削除し、標準スタイルに準拠するようにしたいのであれば、Sunの(r.i.p)スタイルガイドラインを使用します。

一般に、 no.

括弧内は 不要 周辺ternary(条件付き)-事業者またはその部分は、その優先順位は非常に低めの操作だけで下記の論理演算子および上記課題).のリンクは以下の表に示す。

できると主張し、このような不要なパー 視覚的にクラッタ このコードは、いわ な理解 のプログラマを交換してください。

る例外を要する可能性があると使用のパー周辺ternariesすることはできない。

  • 場合はternaryが複雑なものに十分なメリットを複数行;だがその周囲をお書パーを防止するための自動セミコロン挿入出来ます。

  • 場合はternaryは入れ子やその他のternary.

まMDN:

あなたの質問の根拠は、コードを読むという行為に関係していますので、私はその観点から問題にアプローチします。

いわゆる「高速読み取り」トレーニングプログラムの基本原則の一つは、彼らがリーダーがのゲシュタルトを開発するために取得しようということであるテキストの行のではなく、順次言葉のそれを読んでバイ言葉。それはトークンによってトークンの構文解析の行為であるかのようにフルラインの感覚ではなく、読書の行為の治療を取得するために文字通りnecessary--場合 - あなたはあなたのコードから自分の本やステップバックからページを取るしようとします。

それはあなたに飛び出すので、あなたは三項演算子異なる色作ることができます:

また、あなたはスタイルを設定できますエディタを使用することができます。メモ帳++、例えば、などの多くの他のエディタを行う、これを行う組み込みのテーマの数を持っています。

ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top