在Java中与boolean常量进行显式比较是否不好（例如，如果（b == false））？

写不好吗：

if (b == false) //...

while (b != true) //...

是不是总是更好，而不是写：

if (!b) //...

while (!b) //...

大概在性能上没有区别（或存在什么区别？），但是您如何权衡两者之间的明确性，简洁性，清晰度，可读性等？

更新资料

为了限制主观性，我还要感谢权威编码风格指南中的所有引号，这些引号始终是更可取的或在何时使用。

注意：变量名称b仅用作示例alafoo和bar。

不一定坏，只是多余的。同样，实际的变量名也很重要。例如if (userIsAllowedToLogin)，我更希望以上if (b)，甚至更差if (flag)。

至于性能问题，编译器会以任何方式对其进行优化。

更新：关于权威资料，我在《Sun Coding Conventions》中找不到明确的内容，但是至少Checkstyle有一个SimplifyBooleanExpression模块会对此发出警告。

您不应该使用第一种样式。我见过人们使用：

• if ( b == true )
• if ( b == false )

我个人觉得很难阅读，但是可以通过。但是，这种风格的一个大问题是它导致了您所展示的令人难以置信的违反直觉的例子：

• if ( b != true )
• if ( b != false )

读者需要花更多的精力来确定作者的意图。就我个人而言，我发现将对真或假的显式比较包括在内是多余的，因此很难阅读，但这就是我。

这完全是一个品味问题。

就个人而言，我发现它的可读性比我if (!a) {低（编辑：对我而言），因此在以后维护代码时更容易出错，并且我已经转换为使用后一种形式。if (a == false) {

基本上，我不喜欢为逻辑运算而不是单词（C与Pascal）选择符号，因为对我 a = 10 and not b = 20而言a == 10 && !(b==20)，它比容易阅读，但这就是Java中的用法。

任何赞成“ == false”的人都赞成“！” 显然，从来没有盯着代码太久而错过了感叹号。是的，您会遇到代码盲的情况。

您不应该使用第一种样式的主要原因是，这两种都是有效的：

if (b = false) //...

while (b = true) //...

也就是说，如果您不小心遗漏了一个字符，那么您将创建一个分配而不是一个比较。赋值表达式的计算结果为已分配的值，所以第一个语句的上方将值false到b和计算结果为false。第二个值分配true给b，因此true无论您b在循环内进行什么操作，它始终求值为。

除了初学者编写的代码外，我从未见过。总是后者，而且我认为没有人对此感到困惑。另一方面，我认为

int x;
...
if(x) //...

与

if(x != 0) //...

更值得商bat，在那种情况下，我确实更喜欢第二种

恕我直言，我认为，如果仅使bool变量名称带有"Is"，这是不言而喻的，更有意义，然后，您可以删除带有true或的显式比较false

例：

isEdited  // use IsEdited in case of property names
isAuthorized // use IsAuthorized in case of property names

等等

我更喜欢第一个，因为它更清晰。机器可以很好地阅读，但是我尝试编写代码供其他人阅读，而不仅仅是机器。

我更喜欢长距离的方法，但是我比较使用==而不是!=99％的时间。

我知道这个问题是关于Java的，但是我经常在各种语言之间切换C#，例如，== false在处理nullable bool类型时，与（比较）可以比较有用。所以我有比较使用true或不false使用==运算符的习惯。

我做这些：

if(isSomething == false) 要么 if(isSomething == true)

但我讨厌这些：

if(isSomething != false) 要么 if(isSomething != true)

出于明显的可读性原因！

只要您保持代码可读性，就没有关系。

就个人而言，我将重构代码，以便不使用负面测试。例如。

if (b == false) {
   // false
} else {
   // true
}

要么

boolean b = false;
while(b == false) {
  if (condition)
      b = true;
}

恕我直言，在90％的情况下，代码可以重构，因此不需要否定测试。

我认为这很烦人。不过我不会引起什么骚动。

通常的准则是永远不要测试布尔值。有人认为，额外的冗长性增加了清晰度。添加的代码可能会对某些人有所帮助，但是每个读者都需要阅读更多的代码。

今天早上，我花了1/2个小时才找到一个错误。该代码是

    if ( !strcmp(runway_in_use,"CLOSED") == IPAS_FALSE)
      printf(" ACTIVE    FALSE \n");   else
      printf(" ACTIVE    TRUE \n");

如果使用常规惯例进行编码，我会发现它错了的速度要快得多：

    if (strcmp(runway_in_use, "CLOSED"))
      printf(" ACTIVE    FALSE \n");   else
      printf(" ACTIVE    TRUE \n");

这是我对StackOverflow的第一个答案，所以很好。...最近在重构时，我注意到2个代码块几乎具有完全相同的代码，但其中一个使用了

for (Alert alert : alerts) {
    Long currentId = alert.getUserId();

    if (vipList.contains(currentId)) {
        customersToNotify.add(alert);

        if (customersToNotify.size() == maxAlerts) {
            break;
        }
    }
}

另一个有

for (Alert alert : alerts) {
    Long currentId = alert.getUserId();

    if (!vipList.contains(currentId)) {
        customersToNotify.add(alert);

        if (customersToNotify.size() == maxAlerts) {
            break;
        }
    }
}

因此在这种情况下，有必要创建一种方法来处理这样的两种情况，并使用boolean == condition翻转含义

private void appendCustomersToNotify(List<Alert> alerts
        List<Alert> customersToNotify, List<Long> vipList, boolean vip){

    for (Alert alert : alerts) {
        Long currentId = alertItem.getUserId();

        if (vip == vipList.contains(currentId)) {
            customersToNotify.add(alertItem);

            if (customersToNotify.size() == maxAlerts) {
                break;
            }
        }
    }
}

我会说这很糟糕。

while (!b) {
    // do something 
}

阅读比

while (b != true) {
    // do something 
}

不赞成使用第一个（b == false）的原因之一是，初学者通常根本没有意识到第二个选择（！b）完全可行。因此，使用第一种形式可能会误解布尔表达式和布尔变量。这样，使用第二种形式已成为一种sjiboleth：当某人写下此内容时，他/她可能知道发生了什么。

我认为，这已导致差异比实际更重要。

虽然两者都是有效的，但对我而言，第一种感觉就像是类型错误。

对我来说b == false好像是错误的(i == 0) == false。就像：是吗？

布尔值不是具有2个可能值的枚举。您不比较它们。布尔是谓词，表示某些事实。它们有像特定运营商&，|，^，!。

要反转表达式的真相，请使用运算符“！”，并将其发音为“ not”。

正确命名后，它就变得自然了：!isEmpty对我来说，可读性很强。
虽然isEmpty == false读到诸如“它为空是错误的”之类的信息，但我需要更多时间来处理。