2017-11-02 9 views
0

チェックスタイルでは、閉じ括弧がインデントレベルが低すぎ(8)、それが12である必要があることがわかります。しかし、視覚的に外側のレベルにしたいので、チェックが1つであることがはっきりします「ブロック」または集合的な結果。checkstyleを持つメソッドの中で閉じ括弧のインデントを減らす方法はありますか?

これを行うにはどうすればよいですか(閉じ括弧のみ)。

private boolean someMethod() 
{ 
    return (
     someCheck() && 
     anotherCheck() && 
     aThirdCheck() 
    ); //This is the line it complains about 
} 

私が取得エラー:

'メソッドdefを' 子供が間違ったインデントレベル8を持って、期待されるレベルは12

+0

このルールが適切でない場合は、このルールを削除/無効にします。 – davidxxx

+0

@ davidxxxこれは、インデント規則を強制するのを防ぎます –

+0

外かっこを省略することはできますか?彼らは冗長であり、読みやすさに何も追加していないようです。運が良ければ、問題はもう解消します。 –

答えて

0

あなたのインデントが本当に標準ではありませんする必要があります。
さらに、メソッドの呼び出しを伴う一連の&&では、かっこは必要ありません。
読書が難しくなることさえあります。

個人的に、私はそれを見つける:それよりも

private boolean someMethod() 
{ 
    return (
     someCheck() && 
     anotherCheck() && 
     aThirdCheck() 
    ); //This is the line it complains about 
} 

少ないクリア:

private boolean someMethod(){ 
    return someCheck() && anotherCheck() && aThirdCheck(); 
} 
+0

実際、小切手はずっと長いので、ラインが本当に長くなって複数のラインに欲しい理由です。 –

+0

わかりました。しかし、これらのブール式をすべて括弧で囲んでおきたいのはなぜですか?たぶんあなたの例はあまりにも単純です。 – davidxxx

+0

おそらく、囲まれた式の間に、かっこ内にいくつかの論理演算があります。しかし、単純な例では、そのようなものは全く表示されません。 – Vlasec

0

'メソッドdefは' 子供が間違ったインデントレベル8を持って、期待されるレベルはどのように12
する必要があります私はこれを行うのですか(閉じ括弧のみ)?

IndentationCheckでこれを行う方法はありません。この小切手をどのくらいカスタマイズできるかは非常に狭いです(http://checkstyle.sourceforge.net/config_misc.html#Indentationを参照)。このケースをサポートするためにCheckstyleに変更要求を提出することができますが、インデントは最も複雑なチェックの1つであり、多くの人がそれに取り組んでいるわけではありません。

抑制を使用して違反を抑制することができます。また、他の人のように、小切手を削除/無効にすることもできます。自分のグループがEclipseを使用しているため、私はそれを無効にして、Eclipseで任意の保存時にコードをフォーマットするように強制します。