Java

JAVA.STRUCT.EXCP.GEH : Generic Exception Handler (Java)

要旨

An exception handler is used for a very generic exception type.

This checker controls if exception handlers might be incorrect. For instance, an exception handler with an empty body might be an unfinished snippet of code; an exception handler for a very generic exception type might end up catching too many exceptions, also some that were not meant to be caught there.

プロパティ

クラス名 Generic Exception Handler (Java)
日本語クラス名 Generic Exception Handler (Java)
クラス分類 信頼性 (reliability)
ニーモニック JAVA.STRUCT.EXCP.GEH
カテゴリー
CWE CWE:396 Declaration of Catch for Generic Exception
CERT-Java CERT-Java:ERR08-J Do not catch NullPointerException or any of its ancestors
対応言語 Java で利用可能です。
有効/無効設定 このワーニングクラスのチェックはデフォルトで有効になっています。チェックを無効にするにはプロジェクト設定ファイル （configuration file）に以下の WARNING_FILTER ルールを追加してください。 
WARNING_FILTER += discard class="Generic Exception Handler (Java)"

例 

// Main.java 
import java.io.FileWriter;

public class Main {
  public static void main(String args[]) {
    FileWriter fw = null;
    try {
      fw = new FileWriter(args[0]);
      fw.close();
    }
    catch (IndexOutOfBoundsException e) {} // Empty Exception Handler (Java) warning issued here
    catch (Exception e) {                  // Generic Exception Handler (Java) warning issued here
      System.out.println("unusual");
    }
  }
}

These issues can be resolved as follows.

// Main.java, after modification 
import java.io.FileWriter;

public class Main {
  public static void main(String args[]) {
    FileWriter fw = null;
    try {
      fw = new FileWriter(args[0]);
      fw.close();
    }
    catch (IndexOutOfBoundsException e) {
      System.out.println("usage: Main filename");
    }
    catch (IOException e) {
      System.out.println("I/O error");
    }
  }
}

解決法

Add a body to the exception handler, or narrow to the exception types.

関連のある設定ファイルパラメータ

設定ファイルの以下のパラメータがこのワーニングクラスのチェックに影響します。