【问题标题】:PMD : DD anomaly for variablePMD:变量的 DD 异常
【发布时间】:2016-06-14 21:09:31
【问题描述】:

我正在使用带有 PMD 插件 (4.0.0.v20130510-1000) 的 Eclipse,遇到了很多违规问题并解决了大部分问题,但我不明白为什么 DD anomaly 出现在下面的代码中

private boolean createFile(final String szFileName){
    final File oFile = FileUtils.getFile(szFileName);
    boolean bStatus;
    try {
        FileUtils.touch(oFile);
        bStatus = true; // DD anomaly
    } catch (IOException e) {           
        log.error("Failed to create new file. "+ e);
        bStatus = false;
    }
    return bStatus;
}

我不能使用多个 return 语句,因为 multiple exit point 规则被破坏了。

尝试了另一种方式,但将 DD 异常转移到另一条线路,

private boolean createFile(final String szFileName){
        final File oFile = FileUtils.getFile(szFileName);
        boolean bStatus = true; // DD anomaly
        try {
            FileUtils.touch(oFile);
        } catch (IOException e) {           
            log.error("Failed to create new file. "+ e);
            bStatus = false;
        }
        return bStatus;
    }

【问题讨论】:

  • 为什么不最初将bStatus 设置为true,如果抛出异常,将其设置为false?无需在try 块中将其设置为true
  • @JonnyHenly 试过这个,但现在 boolean bStatus = true; 得到 DD 异常。
  • 为什么不让createFile 抛出一个IOException 并在上游处理它?然后你可以让createFile 返回void,或者更好的是让它返回它创建的文件。
  • @JonnyHenly 是的,这是完美的。谢谢!
  • 我会将其添加为答案,以防其他人有类似问题。

标签: java pmd


【解决方案1】:

另一种方法是让createFile 抛出一个IOException 并在上游处理它。然后你可以让createFile 返回void,或者更好的是让它返回它创建的文件。这将摆脱您收到的DD anomaly

替代createFile:

private File createFile(final String szFileName) throws IOException {
    final File oFile = FileUtils.getFile(szFileName);        
    FileUtils.touch(oFile);

    return oFile;
}

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2012-12-29
    • 1970-01-01
    • 2013-01-01
    • 2020-06-26
    • 2011-05-10
    • 1970-01-01
    相关资源
    最近更新 更多