解决SonarQube中测试equals()时的‘空值比较’Bug

兄弟，你碰到的这个问题我之前也遇到过——SonarQube的代码规范检查和咱们对自定义equals实现的正确性验证撞车了，咱们一步步理清楚怎么解决：

为什么SonarQube会报这个警告？ #

SonarQube的提示其实是基于Java官方对equals方法的约定：任何非null对象调用equals(null)必须返回false。它默认认为你应该遵守这个约定，所以直接写equals(null)的代码是冗余的——毕竟按照规范结果是确定的。但你这里是想验证自己重写的equals确实正确处理了null参数，这个测试需求完全合理，毕竟自定义equals时很容易漏写null判断。

可行的解决办法 #

1. 给测试代码加Sonar忽略注释 #

既然这个测试是你故意写的验证逻辑，直接在对应代码行加个Sonar忽略标记就行，告诉它这里不用报警：

@Test
public void test() {
    MyTestedObject testedObject = new MyTestedObject();
    boolean nullValue = testedObject.equals(null); // NOSONAR
    Assert.assertFalse(nullValue);
}

如果你想更精准（避免跳过所有规则），可以去SonarQube的警告详情里找到这个规则的ID，比如java:Sxxxx，然后写成：

boolean nullValue = testedObject.equals(null); // sonar:java:Sxxxx

2. 调整测试逻辑，间接验证null处理 #

要是不想用忽略注释，也可以换个方式验证，比如结合其他equals特性一起测：

@Test
public void testEqualsWithNull() {
    MyTestedObject testedObject = new MyTestedObject();
    // 核心验证：equals(null)返回false
    Assert.assertFalse(testedObject.equals(null));
    // 顺便验证和自身比较的正确性，确保整体逻辑没问题
    Assert.assertTrue(testedObject.equals(testedObject));
}

不过本质上还是在测同一个点，Sonar可能还是会警告，但你心里清楚这是验证自定义实现的必要步骤就行。

3. 项目级配置忽略该规则（针对测试代码） #

如果你的测试代码里经常需要写这类验证，可以在SonarQube的规则配置里，给测试代码目录单独关闭这个规则。这样既不影响业务代码的规范检查，又能让测试代码的验证逻辑顺利通过扫描。

要不要测试equals的null处理？ #

必须要！虽然Java规范要求equals(null)返回false，但自定义equals时很容易犯傻——比如漏写obj == null的判断，或者逻辑写反。这个测试就是给你的实现上一道保险，防止后续修改equals时破坏规范，绝对不能忽略这个场景。

内容的提问来源于stack exchange，提问作者Yassine Badache