FindBugs提示EQ_OVERRIDING_EQUALS_NOT_SYMMETRIC缺陷如何解决

**EQ_OVERRIDING_EQUALS_NOT_SYMMETRIC缺陷的解决方案**

在Java编程中，`equals()`方法是Object类的一个重要方法，它用于比较两个对象是否相等。然而，在实际开发过程中，我们经常会遇到一个问题：当我们重写`equals()`方法时，不小心忘记了对称性（即，如果a.equals(b)返回true，那么b.equals(a)也应该返回true）。这种情况下，FindBugs工具就会提示我们这个缺陷，即EQ_OVERRIDING_EQUALS_NOT_SYMMETRIC。

**什么是EQ_OVERRIDING_EQUALS_NOT_SYMMETRIC?**

EQ_OVERRIDING_EQUALS_NOT_SYMMETRIC是一种常见的错误，它指的是在重写`equals()`方法时，忘记了对称性。也就是说，如果我们有两个对象a和b，且a.equals(b)返回true，那么b.equals(a)应该也返回true。但是，由于我们的实现不正确，这个条件并没有得到满足。

**如何解决EQ_OVERRIDING_EQUALS_NOT_SYMMETRIC?**

要解决这个问题，我们需要确保在重写`equals()`方法时，严格遵守对称性原则。具体来说，我们应该检查一下，如果a.equals(b)返回true，那么b.equals(a)也应该返回true。

**示例代码**

假设我们有一个Person类，它有两个属性：name和age。

javapublic class Person {
 private String name;
 private int age;

 public Person(String name, int age) {
 this.name = name;
 this.age = age;
 }

 @Override public boolean equals(Object obj) {
 if (this == obj)
 return true;
 if (obj == null || getClass() != obj.getClass())
 return false;

 Person person = (Person) obj;

 // 这里我们只比较name属性，不考虑age属性 return name.equals(person.name);
 }
}

在这个例子中，我们重写了`equals()`方法，只比较了name属性。然而，这个实现是不正确的，因为如果两个Person对象有相同的name，但不同的age，那么a.equals(b)返回true，但是b.equals(a)返回false。

**如何修复这个问题?**

要修复这个问题，我们需要修改`equals()`方法，使得它严格遵守对称性原则。具体来说，我们应该检查一下，如果a.equals(b)返回true，那么b.equals(a)也应该返回true。

java@Overridepublic boolean equals(Object obj) {
 if (this == obj)
 return true;
 if (obj == null || getClass() != obj.getClass())
 return false;

 Person person = (Person) obj;

 // 这里我们比较name和age属性 return name.equals(person.name) && age == person.age;
}

在这个修复版中，我们比较了name和age两个属性。这样一来，如果a.equals(b)返回true，那么b.equals(a)也应该返回true。

**总结**

EQ_OVERRIDING_EQUALS_NOT_SYMMETRIC是一种常见的错误，它指的是在重写`equals()`方法时，忘记了对称性。要解决这个问题，我们需要确保在重写`equals()`方法时，严格遵守对称性原则。具体来说，我们应该检查一下，如果a.equals(b)返回true，那么b.equals(a)也应该返回true。