PMD：变量的 DD 异常

Posted 2023-03-31

【中文标题】PMD：变量的 DD 异常

【英文标题】：PMD : DD anomaly for variable

【发布时间】：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 是的，这是完美的。谢谢！

我会将其添加为答案，以防其他人有类似问题。

【参考方案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;