忍者ブログ
Admin§Write

ぷろぐらみんぐ徒然

文系4大卒がなぜか職業ぷろぐらまになって早10年。 とっくに三十路は過ぎた。 いつまでも派遣プログラマだと死亡フラグが立つのでそれよりきり前に脱出成功。今後の事はまだ良くわからん。 ここのところ愛してやまないバンドはFuzzy Control。 興味を持ってもらえたらうれしいです。 この欄だとタグが使えないみたいなのでURLも。 http://www.fuzzycontrol.jp/

HOME ≫ Entry no.4 「Checkstyleプラグイン設定 - 設定はこうすれば楽なんじゃねーの?その1 - setterメソッドにフィールドを隠してるの警告はいらないと思う。」 ≫ [9] [8] [7] [6] [5] [4] [3] [2] [1]

[PR]

×

[PR]上記の広告は3ヶ月以上新規記事投稿のないブログに表示されています。新しい記事を書く事で広告が消えます。


Checkstyleプラグイン設定 - 設定はこうすれば楽なんじゃねーの?その1 - setterメソッドにフィールドを隠してるの警告はいらないと思う。

Coding Problems -> Hidden Field ->
ignore ConstructorParameter をオン
ignore Setter をオン
そもそもそのクラスメンバに値を入れ込みたいのにあえて別変数を指定する必要はないよね。
むしろsetterに限っては値を編集しないで設定する場合に限りだが、パラメタ名はメンバ変数名と同一にするべきだと思う。

頭悪い人が使うと引数名に"p"をつけるとかやってEclipseの補完機能で意味のわからない変数名が出てきて実装者をイラっとさせるですよね。<実際、このCheckstyleのWarning回避の為に全ての引数の変数名にp、もしくはvをつける、という酷いルールを見たことがあります。
最低なのは全ての引数がarg1,arg2・・・・(バグだらけで大変そうでした。)

ignoreAbstractMethodsはどうなんだろ?私はオフでいいとおもいますが・・・
メンバ変数も同時にいじりたいメソッドを抽象メソッドで定義する必要があるかどうかは知らないしもしあったとしても値をセットする部分はfinalメソッドかなんかに書いてその中身で抽象メソッドを呼んであげた方が安全かも。



PR

●Thanks Comments

●この記事にコメントする

お名前
タイトル
文字色
E-mail
URL
コメント
絵文字 Vodafone絵文字 i-mode絵文字 Ezweb絵文字
パスワード ※投稿者編集用
秘密? ※チェックすると管理人にしか見えません

●この記事へのトラックバック

TrackbackURL:

≪ Checkstyleプラグイン設定 - 設定はこうすれば楽なんじゃねーの?その2 - JavaDocコメントのデフォルトフォーマットがコードフォーマットの警告に引っかかってしまう点に対応 |PageTop| Checkstyleプラグイン ≫

※ 忍者ブログ ※ [PR]
 ※
Writer 【へっぽこPG】  Design by NUI.T  Powered by NinjaBlog