【发布时间】: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 是的,这是完美的。谢谢!
-
我会将其添加为答案,以防其他人有类似问题。