checkstyle.xmlに<module name="IllegalCatch"/>を追加したい

[kharada7]

概要

dressca-backendのcheckstyle設定ファイル（ dressca-backend\config\checkstyle\checkstyle.xml ）には現状汎用検査例外（ java.lang.Exception ）の catch を禁止する設定がない。
ほとんどの開発プロジェクトでは汎用検査例外を catch すべきでないと思われるので、追加してほしい。

コーディング規約に、カスタマイズしたルールに関して追記したい。

詳細 / 機能詳細（オプション）

追加する設定は以下。

<module name="Checker">
  <module name="TreeWalker">
    <module name="IllegalCatch"/>
  </module>
</module>

checkstyle – IllegalCatch

完了条件

• 以下のファイルに IllegalCatch の設定が追加されていること。
  dressca-backend\config\checkstyle\checkstyle.xml

• 汎用検査例外のcatchがcheckstyleの警告として検出されること。

• コーディング規約にルールが追加されていること。

[kenjiyoshid-a]

IllegalCatchの追記によって、禁止になるデフォルトの項目

をcheckstyle.xmlに追記することによって、デフォルトで以下の例外のcatchが禁止される。

• Error
• Exception
• RuntimeException
• Throwable
• java.lang.Error
• java.lang.Exception
• java.lang.RuntimeException
• java.lang.Throwable

以上の例外以外のcatchを禁止したい場合

以上の一覧以外の例外のcatchを禁止したい場合には以下の記述を含める。
以下は、ArithmeticExceptionおよびOutOfMemoryErrorのcatchを禁止する例である。

<module name="Checker">
  <module name="TreeWalker">
    <module name="IllegalCatch">
      <property name="illegalClassNames"
           value="ArithmeticException,OutOfMemoryError"/>
    </module>
  </module>
</module>

デフォルトの設定でも、本Issueのクローズの要件を満たすことができるが、AlesInfiny Maiaとして、catchを禁止した方がいい例外を検討する必要がある。