java 悪い習慣のコードとは

Question

checkstyleの機能で
RegexpMultilineというモジュールがあり
デフォルトの規約ではSystem.out.printlnでエラーが出るように設定されているようです

解説では
一般的に悪い習慣のコード、ex.printStacktrace(),System.out.println(), System.exit()
となっていたのですが、これらがどうして
悪い習慣のコードとして出ていたのかが分かりません

Answer 1

System.exit は、JVMが停止するのでまずコード中にあらわれてはいけません。

CWE-382: J2EE Bad Practices: Use of System.exit()

System.out.printlnは、基本的にロガーを使ったほうが細かい制御がやりやすいです。
printStackTraceも同じ同様です。

基本的に、標準出力、標準エラーはログが記録されないので、ファイルに出力しましょう。
みたいな感じです。

Answer 2

悪い習慣というよりは、改行の扱いとして問題が発生する可能性を示唆しているのではないかと思います。

システムが自動で改行を付与するようなメソッド（printStacktrace()、println()等）は
JVMの実行環境によって、付与（出力）される改行コードが違います。

• Windows：CR+LF
• Linux：LF

そういった差異が出るといった意味でのチェックをしてほしいのではないでしょうか？

あと、exit()もチェックエラーの対象になっていたのでしょうか？
下記のページでは特にそういった記載が見られなかったので。
http://checkstyle.sourceforge.net/config_regexp.html#RegexpMultiline

Comments

[Natsumi_k]

納得できるご回答ありがとうございました！
あとは自分で確認してみますm(__)m