“x!=y”的 PMD 验证

Question

这是我的代码：

private void showMessageForInvalid(File aFile) {
  if (!aFile.exists()) {
     // show not exists message
  } else if (!aFile.isDirectory()) {
     // show not directory message
  }
}

对于第一个 if 块，我收到 PMD 警告“避免 x！= y else”。我认为，这个警告是正确的，以防万一

if (!aFile.exists()) {
  // do something
} else {
  // do else
}

但在我的情况下，else if 声明并非如此。
这是 PMD 错误还是此警告正确（在这种情况下我应该如何编写代码）？

Answer 1

除了@Tim B 的回答：

它在Design rules as 'confusing ternary' 下列出。

在if-else 块中包含!condition 通常是不好的。在大多数情况下，这是由于结构不良：

if(healthy)
     business-logic
else
     error-handling

以上是大多数（诚然不是所有人）人们认为的自然思路，而不是

if(!healthy)
     error-handling first
else
     business-logic later

现在，情况并非总是如此 - 您的函数（在其名称中暗示）显式处理错误情况 - 是可能的异常之一。不幸的是，使用像 PMD 这样的静态代码分析工具，您可以捕获所有案例，也可以不捕获任何案例——在这种情况下，开发人员选择了全部；诚然让它有点过分热心。

无论如何，这就是它被列入 PMD 设计规则的原因。当遇到这样的情况时，我发现最简单的方法是指定a ruleset exclusion as described here，而不是将源代码实际修改为对我来说没有意义的东西。

Answer 2

先做exists分支：

if (aFile.exists()) {

   if (aFile.isDirectory()) {
       // Do stuff
   } else {
       // show not directory message
   }    
} else {
   // show not exists message
}

这具有相同的实际程序流程和逻辑结构，但“一目了然”发生了什么。