SONAR 抱怨改变条件，使其并不总是评估为“假”

Question

public String generateURLSafeToken(String username, char[] password) throws CredentialTokenException {
    this.tokenValid = false;

    String token = null;
    if ((username.length() < 1) || (username == null)) {
        throw new CredentialTokenException("Username cannot be an empty string or null.");
    }
    if ((password.length < 1) || (password == null)) {
        throw new CredentialTokenException("Password cannot be an empty or null.");
    }

我在第 4 行和第 7 行遇到此错误（用户名 == null 和密码 == null）

我的代码中需要这部分。我正在尝试 isEmpty() 而不是 null 但也面临着问题。解决此 SONAR 错误的替代方法或解决方案是什么

Answer 1

总是计算为false 的条件是username == null 和password == null。

我们以username 为例。运算符|| 是short-circuiting，这意味着如果左侧是true，它将不会评估右侧。基本上有两种情况：

• 给定的username 不是null。评估条件username.length() < 1
  • 如果结果是true，我们直接返回，进入if分支
  • 如果结果是false，我们会尝试评估username == null。但由于给定的username 不是null，因此这个总是评估为false。
• 给定的username 是null。评估条件username.length() < 1。这实际上停在那里：它会抛出一个 NullPointerException 并且不会评估右侧。

因此，您可以看到，无论何时实际评估 username == null 条件，结果始终为 false。这就是 SonarQube 警告告诉您的内容。

这里的解决方案是颠倒您的 2 个条件。考虑拥有

if (username == null || username.length() < 1)

相反。如果您重新开始并检查每个案例，您会注意到没有一个表达式将始终具有相同的结果：

• 给定的username 不是null。第一个条件明确计算为false，第二个条件被计算，可能返回true 或false。
• 给定的username 是null。第一个条件明确评估为true 和短路。