TK Factory

厳密な例外に関するルールセットです。
※PMD 3.9に対応しています。

AvoidCatchingThrowable

Throwableをcatchしていないかチェックします。
Throwableをcatchしてしまうと、OutOfMemoryErrorのような例外もcatchしてしまいます。

public class Foo
{
    public void bar()
    {
        try
        {
            // …
        }
        catch (Throwable th)
        {
            // ThrowableはcatchすべきではないのでＮＧ！
            th.printStackTrace();
        }
    }
}

SignatureDeclareThrowsException

Exceptionをthrowしているメソッドがないかチェックします。

// ExceptionをthrowすべきではないのでＮＧ
public void methodThrowingException() throws Exception
{
    // …
}

ExceptionAsFlowControl

GOTO文のように例外を使用していないかチェックします。

public class Foo
{
    void bar()
    {
        try
        {
            try
            {
                // …
            }
            catch (Exception e)
            {
                // GOTO文のように
                // WrapperExceptionのcatchブロックへ処理が移る
                throw new WrapperException(e);
            }
        }
        catch (WrapperException e)
        {
            // …
        }
    }
}

AvoidCatchingNPE

NullPointerExceptionがcatchされていないかチェックします。

public class Foo
{
    void bar()
    {
        try
        {
            // …
        }
        catch (NullPointerException npe)
        {
            // NullPointerExceptionがcatchされているのでＮＧ！
        }
    }
}

AvoidThrowingRawExceptionTypes

RuntimeException、Throwable、Exception、Errorがthrowされていないかチェックします。
通常はこれらのサブクラスをthrowすべきです。

public class Foo
{
    public void bar() throws Exception
    {
        // ExceptionがthrowされているのでＮＧ！
        throw new Exception();
    }
}

AvoidThrowingNullPointerException

NullPointerExceptionをthrowしていないかチェックします。
NullPointerExceptionはJVMがthrowするものなので、明示的に記述すべきではありません。

public class Foo
{
    void bar()
    {
        // NullPointerExceptionが明示的にthrowされているのでＮＧ！
        throw new NullPointerException();
    }
}

AvoidRethrowingException

catchブロックで例外をそのままthrowしていないかチェックします。

public class Foo
{
    void bar()
    {
        try
        {
            // …
        }
        catch (SomeException se)
        {
            // そのままthrowしているだけなのでＮＧ！
            throw se;
        }
    }
}

関連記事

• PMD – Java Loggingルール・セット
• PMD – Unused Codeルール・セット
• PMD – Finalizerルール・セット
• PMD – Import Statementルール・セット
• PMD – Cloneルール・セット