在 IF 条件中使用 NOT 运算符

Question

避免在 IF 条件中使用 NOT 运算符以使代码更具可读性真的是一个好习惯吗？我听说 if (doSomething()) 比 if (!doSomething()) 更好。

Answer 1

如果可以选择的话，避免使用 ! 运算符通常不是一个坏主意。一个简单的原因是它可能成为错误的来源，因为有可能忽视它。在某些情况下，更具可读性的是： if(conditionA==false) 。如果你跳过 else 部分，这主要起作用。
无论如何，如果您有 else 块，则不应在 if 条件中使用否定。

除了这样的组合条件：

if(!isA() && isB() && !isNotC())

这里你必须使用某种否定来获得所需的逻辑。
在这种情况下，真正值得思考的是函数或变量的命名。
尝试为它们命名，以便您可以经常在简单的条件下使用它们而无需否定。

在这种情况下，您应该考虑 isNotC() 的逻辑，以及如果有意义的话是否可以用 isC() 方法替换它。

最后，你的例子在可读性方面还有另一个问题，这个问题比是否使用否定的问题更严重：代码的读者是否真的知道 doSomething() 何时返回 true，何时返回 false？
如果它是假的，那么它还是完成了吗？这是一个非常常见的问题，最终导致读者试图找出函数返回值的真正含义。

Answer 2

这实际上取决于您想要实现的目标。如果您没有 else 子句，那么 if(!doSomething()) 似乎没问题。但是，如果您有的话，

if(!doSomething()) {
    ...
}
else {
    // do something else
}

我可能会反转该逻辑以删除 ! 运算符并使 if 子句稍微更清晰。

Answer 3

作为一般性陈述，最好使 if 条件尽可能具有可读性。对于您的示例，使用 !没问题。问题是，当事情看起来像

if ((a.b && c.d.e) || !f)

您可能想做类似的事情

bool isOk = a.b;
bool isStillOk = c.d.e
bool alternateOk = !f

时，您的 if 语句会被简化为

if ( (isOk && isStillOk) || alternateOk)

它只会使代码更具可读性。如果必须调试，您可以调试 isOk 变量集，而不必挖掘作用域中的变量。它对于处理 NPE 也很有帮助——将代码分解成更简单的块总是好的。

Answer 4

不，在 if..then..else 语句中使用 ! 运算符绝对没有问题。

变量的命名以及您的示例中的方法的命名很重要。如果您使用：

if(!isPerson()) { ... } // Nothing wrong with this

但是：

if(!balloons()) { ... } // method is named badly

这一切都取决于可读性。始终以最具可读性为目标，这样就不会出错。始终尝试保持代码的连续性，例如，看看 Bill the Lizards

Answer 5

一般来说， ！是一个非常好的、可读的布尔逻辑运算符。没有理由不使用它，除非您通过删除双重否定或应用摩根定律来简化。

!(!A) = A

或者

!(!A | !B) = A & B

根据经验，请保持布尔返回方法的签名易于记忆并符合约定。 @hvgotcodes 提出的场景的问题在于，ab 和 cde 当然一开始就不是非常友好的示例。假设您有一个航班预订应用程序的航班和座位类别。那么预订航班的条件完全可以像

if(flight.isActive() && !seat.isTaken())
{
    //book the seat
}

这样完全可读且易于理解的代码。不过，您可以重新定义 Seat 类的布尔逻辑，并将条件改写为此。

if(flight.isActive() && seat.isVacant())
{
    //book the seat
}

从而删除 !运算符（如果它确实困扰您），但您会发现这完全取决于您的布尔方法的含义。

Answer 6

尝试这样

if (!(a | b)) {
    //blahblah
}

它是一样的

if (a | b) {}
else {
    // blahblah
}

Answer 7

我以前从未听说过这个。

How

if (doSomething()) {
} else {
   // blah
}

比

if (!doSomething()) {
   // blah
}

后者更清晰简洁。

除了！运算符可以出现在复杂的条件中，例如 (!a || b)。那你如何避免呢？

使用！需要时操作员。

Answer 8

我认为何时使用 if (!doSomething()) 的一个很好的例子是在 Java 11 之前使用 Optional。Java 11 添加了 isEmpty code>，但在此之前只有 isPresent。您试图从函数中尽早返回的图像（常见的编程最佳实践）

if (!option.isPresent()){
  return -1;
}

将是一种非常常见的做法。或者，如果您确实想避免 if 中的 !，您可以自己引入一个临时变量

var isEmpty = !option.isPresent();
if (isEmpty){
  return -1;
}