Checkstyleプラグイン設定 - 設定はこうすれば楽なんじゃねーの?その1 - setterメソッドにフィールドを隠してるの警告はいらないと思う。
2008.11.17 |Category …Eclipse
Coding Problems -> Hidden Field ->
ignore ConstructorParameter をオン
ignore Setter をオン
そもそもそのクラスメンバに値を入れ込みたいのにあえて別変数を指定する必要はないよね。
むしろsetterに限っては値を編集しないで設定する場合に限りだが、パラメタ名はメンバ変数名と同一にするべきだと思う。
頭悪い人が使うと引数名に"p"をつけるとかやってEclipseの補完機能で意味のわからない変数名が出てきて実装者をイラっとさせるですよね。<実際、このCheckstyleのWarning回避の為に全ての引数の変数名にp、もしくはvをつける、という酷いルールを見たことがあります。
最低なのは全ての引数がarg1,arg2・・・・(バグだらけで大変そうでした。)
ignoreAbstractMethodsはどうなんだろ?私はオフでいいとおもいますが・・・
メンバ変数も同時にいじりたいメソッドを抽象メソッドで定義する必要があるかどうかは知らないしもしあったとしても値をセットする部分はfinalメソッドかなんかに書いてその中身で抽象メソッドを呼んであげた方が安全かも。
ignore ConstructorParameter をオン
ignore Setter をオン
そもそもそのクラスメンバに値を入れ込みたいのにあえて別変数を指定する必要はないよね。
むしろsetterに限っては値を編集しないで設定する場合に限りだが、パラメタ名はメンバ変数名と同一にするべきだと思う。
頭悪い人が使うと引数名に"p"をつけるとかやってEclipseの補完機能で意味のわからない変数名が出てきて実装者をイラっとさせるですよね。<実際、このCheckstyleのWarning回避の為に全ての引数の変数名にp、もしくはvをつける、という酷いルールを見たことがあります。
最低なのは全ての引数がarg1,arg2・・・・(バグだらけで大変そうでした。)
ignoreAbstractMethodsはどうなんだろ?私はオフでいいとおもいますが・・・
メンバ変数も同時にいじりたいメソッドを抽象メソッドで定義する必要があるかどうかは知らないしもしあったとしても値をセットする部分はfinalメソッドかなんかに書いてその中身で抽象メソッドを呼んであげた方が安全かも。
PR
●Thanks Comments
●この記事にコメントする
●この記事へのトラックバック
TrackbackURL: