例外処理の書き方について

Question

Unit testを勉強中です。以下のクラスで、メソッドごとのテストケースを考えていたのですが、例外を発生させるテストについてお伺いしたいです。try-catchの中のworkDirectory()のメソッドの中で例外を発生させてExceptionをキャッチさせてCommandStatus.FATALをリターンさせたいのですが、workDirectoryメソッドの中にもIllegalArgumentExceptionがあり、ここで例外が起きると
IllegalArgumentExceptionがcatchされ、呼び出し元のexecuteメソッドでExceptionがcatchされ、例外が重なってしまって分かりにくい実装になっていると思いました。IllegalArgumentExceptionを消してもエラーにならないため、例外になる可能性がないと思います。IllegalArgumentExceptionを使っている理由はディレクトリが存在しないときに"ディレクトリがありません。"というメッセージを出して異常終了したいからだと思います。このようなコードをもっと分かりやすくリファクタリングするにはどうすればよいのでしょうか。IllegalArgumentExceptionを使わずにディレクトリが存在しない場合、"ディレクトリがありません。"のメッセージを出してcatch (final Exception e)にcatchさせて異常終了(return CommandStatus.FATAL)させることは可能でしょうか。

@Service
public class CalculateService {
    @Autowired
    private Messages messages;

    /**
     * 計算<br>
     * <br>
     *
     * @param arguments
     * @return non-null 0:正常終了、1:異常終了.
     */
    public CommandStatus execute() {

        // 起動
        log.info(messages.format("START"));

        // 対象ファイル変数定義
        Collection<File> files = null;

        try {

            // 対象ファイル取得処理
            files = workDirectory();

            if (files.size() == 0) {

                // ファイル存在しない。異常終了
                log.error(messages.format("No file"));

                // 異常終了
                return CommandStatus.FATAL;
            }

            // 計算とTSVファイル出力
            TsvFileWriter(files);

        } catch (final Exception e) {

            // エラーログ出力する
            log.error(messages.format("FAILURE_FINISH"), e);

            // 異常終了
            return CommandStatus.FATAL;
        }

        //計算処理正常終了。
        log.info(messages.format("SUCCESS_FINISH_END"));
        return CommandStatus.SUCCESS;
    }


    /**
     * TSVファイル取得
     *
     * @return List<File> ファイルリスト
     * @throws IllegalArgumentException
     */
    private Collection<File> workDirectory() throws IllegalArgumentException {

        try {
            File dir = new File(config.getImportPath());

            IOFileFilter filter = new WildcardFileFilter(Constants.WILD_CARD);

            Collection<File> files = FileUtils.listFiles(dir, filter, null);

            // 受信ディレクトリに受信対象ファイル取得する
            files = FileUtils.listFiles(dir, filter, null);

            // 受信ファイルがある場合、ループを終了
            if (files.size() != 0) {
                log.info(messages.format("ファイルが見つかりません"));
            }

            return files;

        } catch (IllegalArgumentException e) {

            // ディレクトリが存在しません。処理を異常終了。
            throw new IllegalArgumentException(messages.format("ディレクトリがありません。"));
        }
    }

}
コード

Answer 1

IllegalArgumentExceptionを消してもエラーにならないため、

コンパイルエラーのことかと思いますが, IllegalArgumentException は RuntimeException ですので, 非チェック例外です. NullPointerException と同様, throws に書いていなくても発生する可能性はあります.

IllegalArgumentException に因らずに同様の動作をさせるのでしたら, その発生元が実行される前に, 発生する状況かをチェックすれば良いのではないでしょうか.

Answer 2

workDirectoryメソッドの実装で、IllegalArgumentsExceptionがスローされる状況は、

FileUtils.listFiles(dir, filter, null);

の dir がディレクトリではなくファイルだった場合に発生します。