java - 来自 PMD 的数据流异常分析警告

Question

我将 Eclipse 与 PMD 插件 (4.0.0.v20130510-1000) 一起使用，并遇到了很多违规行为:

发现变量“freq”的“DD”异常(行“187”-“189”)。
发现变量“freq”的“DU”异常(行“189”-“333”)。

在this所以回答，它说这些异常与分配从未读取过的值有关。但是在这种情况下我遇到了违规行为:

// here I get a DD anomaly
double freq = 0;
try {
  // here I get a DU anomaly
  freq = Double.parseDouble(getFrequencyTextField().getText());
} catch (final NumberFormatException e) {
  Log.e(e.getMessage());
}
if (freq < 10E6) doSomething();

如果我删除初始化并在 catch block 中添加 freq = 0; 行，则 DD 异常消失，但我在两个分配上都得到 DU 异常.

现在我的问题是:我该如何处理？ PMD 的首选解决方案是什么？这条规则到底想阻止什么(即为什么这是不好的做法)？

Answer 1

double freq; // (1)
try {
  // here I get a DU anomaly
  freq = Double.parseDouble(getFrequencyTextField().getText());
} catch (final NumberFormatException e) {
  Log.e(e.getMessage());
  freq = 0; // (2)
}
if (freq < 10E6) doSomething();

第一个问题是在 catch 中没有对 freq 进行 parseDouble 赋值。在异常情况下，频率仍然为 0。可能是可标记的。因此在 catch 中分配给 freq 时它消失了。

当在 catch (2) 中分配给 freq 时，初始分配 (1) 将永远不会被读取，因此只需声明就足够了。

关于更好的风格:

try {
  // here I get a DU anomaly
  double freq = Double.parseDouble(getFrequencyTextField().getText());

  if (freq < 10E6) doSomething();
  ...

} catch (final NumberFormatException e) {
  Log.e(e.getMessage());
}

或者按照@JoachimSauer 的回答，使用不抛出异常的双重转换。日志记录将表明优先于上述风格的严重程度。在简单的转换函数中记录错误可能不是好的风格:记录太多，忽略记录(？)，难以修复。