我有一个界面：

public interface FileRepository {
    String insert(File file) throws IOException;
    // other methods ...
}

我的实现插入（文件文件）使用本地（以避免并发问题）java.security。MessageDigester抛出检查异常java.security。从其工厂方法中的NoSuch算法异常。

public FileRepositoryImpl(String digestAlgo) throws NoSuchAlgorithmException {
    this.digestAlgo = digestAlgo;
    MessageDigest.getInstance(digestAlgo);
    }
@Override
public String insert(File file) throws IOException {
// initialize message digest
    MessageDigest messageDigest = null;
    try {
        messageDigest = MessageDigest.getInstance(digestAlgo);
    } catch (NoSuchAlgorithmException e) {
        LOGGER.fatal(MD_INIT_ERROR, e);
        return null;
}
    // other code ....
}
// other methods (may contain local MessageDigest)

我的实践:由于< code > nosuchalgorithm exception 总是一个致命的错误(这使得模块完全不可用)，我尝试在我的构造函数中初始化一个< code>MessageDigest来测试参数< code>digestAlgo，因此异常可以由构造函数抛出，而不是(早于)从< code>insert(File)抛出。另一个原因是接口定义不允许抛出< code > NoSuchAlgorithmException 。

我的问题：在我的实现中，代码

   } catch (NoSuchAlgorithmException e) {
        LOGGER.fatal(MD_INIT_ERROR, e);
        return null;
   }

因此我认为应该有更好的解决方案，这样可以避免（逻辑上和实际上）无法访问的代码。

欢迎任何解决方案/建议，谢谢。

编辑：

在运行代码时，这不是一个真正的问题。但是在测试中，由于代码是不可达的，加上一些“资源的尝试捕捉”，质量分析工具(sonar，pmd)会认为代码“单元测试的分支覆盖不足”，这是分析报告中的一个主要问题，这就是为什么我想避免这段代码。

另一个问题，测试MessageDigest. getInstance（digestAlgo）是一个好的做法吗？在我的构造函数中？还是最好让插入（文件）全权负责NoSuch算法异常？