Java — абстрактный класс, equals() и два подкласса

Как говорят другие, сигнатура метода, который вы переопределяете, должна быть точно такой же. При переопределении методов, чтобы убедиться, что вы переопределяете, используйте аннотацию @Override над функцией, чтобы такие IDE, как Eclipse, предупредили вас, если вы изменили метод.

Вот как это будет выглядеть:

@Override
public boolean equals(Object obj){
...Your code here...
}

Я бы предложил также переопределить hashCode(), потому что при вставке элементов в списки, наборы, hastables и т. д. для равенства (и производительности) используется hashCode() (а иногда equals() нет!)

Таким образом, ваш окончательный код будет:

@Override
public boolean equals(Object obj){
...Your code here...
}

@Override
public int hashCode(){
...Your code here...
}

Дополнительную информацию можно найти в javadoc.

Я понимаю, почему: equals() не переопределяется в абстрактном классе Xpto.

На самом деле equals() не переопределяется где-либо в вашем коде. Чтобы переопределить его, ваш метод должен иметь Object в качестве типа параметра, и вы должны привести его (после тестирования с instanceof, чтобы вернуть false при сравнении экземпляров двух разных подклассов).

объявление общедоступного логического равенства (Person p) или общедоступного логического равенства (Car p) не отменяет общедоступное логическое равенство Object (Object o), это просто новый метод, который никогда не вызывается.

Вот как бы я поступил:

public  abstract class Xpto {

}

public class Person extends Xpto{

    protected String name;

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

    public boolean equals(Object o){
       if(o == null || !getClass().equals(o.getClass())
          return false;
       Person p = (Person) o;
       System.out.println("Person equals()?");
       return this.name.compareTo(p.name) == 0 ? true : false;
    }
}

public class Car extends Xpto {
    protected String registration;

    public Car(String registration){
            this.registration = registration;
    }

    public boolean equals(Object o){
       if(o == null || !getClass().equals(o.getClass())
          return false;
       Car car = (Car) o;
       System.out.println("Car equals()?");
       return this.registration.compareTo(car.registration) == 0 ? true : false;
    }
}

public class Teste {

    public static void foo(Xpto xpto1, Xpto xpto2){
            if(xpto1.equals(xpto2))
                    System.out.println("xpto1.equals(xpto2) -> true");
            else
                    System.out.println("xpto1.equals(xpto2) -> false");

    }

    public static void main(String argv[]){
            Car c1 = new Car("ABC");
            Car c2 = new Car("DEF");
            Person p1 = new Person("Manel");
            Person p2 = new Person("Manel");

            foo(p1,p2);
    }
}

Каждый класс наследует метод equals(Object) от класса Object. Таким образом, Xpto не нужно определять такой метод.

Когда кто-то переопределяет этот метод в подклассах (а именно: Person, Car), его необходимо определить с точно такой же сигнатурой. Другими словами, параметр метода equals должен иметь тип Object, и реализация метода должна приводить его к понижению.

Javadoc утверждает, что вам нужно переопределить метод equals с объектом в качестве параметра.

Указывает, является ли какой-либо другой объект "равным" этому.

Поэтому ваши методы equals подклассов должны выглядеть примерно так:

public class Car extends Xpto
{
    protected String registration;

    public Car(String registration)
    {
        this.registration = registration;
    }

    public boolean equals(Object obj)
    {
        if (obj == null)
        {
            return false;
        }
        if (obj == this)
        {
            return true;
        }
        if (!obj.getClass().isAssignableFrom(getClass()))
        {
            return false;
        }
        Car car = (Car) obj;
        return this.registration.compareTo(car.registration) == 0 ? true : false;
    }
}

Как правило, очень сложно/невозможно полностью выполнить контракт equals и при этом иметь два разных класса в иерархии, равных друг другу, и обычно это не делается. Как правило, метод equals проверяет, является ли класс одинаковым (поэтому два экземпляра одного и того же подкласса будут равны друг другу, а два экземпляра двух разных подклассов - нет).

Однако в вашем случае можно реализовать equals в Xpto, поскольку есть только одно свойство. Очевидный способ сделать это — определить абстрактный метод в Xpto, а затем также переопределить equals в Xpto:

 public class Xpto {
        protected abstract String getIdentity();

        @Override
        public boolean equals(Object o) {
            if (o == null) return false;
            //Typical implementation
            //if (getClass() != o.getClass()) return false;
            if (!(o instanceof Xpto)) return false; //risky implementation, but will allow a car to compare to a person
             return getIdentity().equals((Xpto) o.getIdentity());
        }

        @Override
        public int hashCode() {
             return getIdentity().hashCode();
        }
  }

Другие указали, что вы на самом деле не переопределили equals в своей реализации. В будущем вы можете заставить компилятор помочь вам с этим, используя аннотацию @Override. В вашем случае вы бы получили ошибку компиляции раньше, что сэкономило бы вам некоторое время.

Ваш метод equals должен выглядеть так:

@Override public boolean equals(Object o) {
   if (!(o instanceof YourType)) {
      return false;
   }
   YourType yt = (YourType)o;
   ... // rest here
}

Кроме того, не забудьте также переопределить hashCode, чтобы иметь возможность правильно использовать ваши типы в коллекциях.

Вы не переопределяете метод equals(), а перегружаете его. Изменить подпись на

public boolean equals(Object o)

А затем примените к Человеку/Автомобилю и сравните.

Кстати, вы также можете сравнить строки с equals():

return registration.equals(car.registration);

Ваши подклассы определяют equals(Person) или equals(Car), ни одному из которых не понравится передача Xpto. Если вы объявите их обоих равными (Xpto) или, что еще лучше, равными (Object), чтобы они работали в коллекциях, тогда ваша проблема должна исчезнуть.

Обратите внимание: если вы повторно объявите методы equals() таким образом, (1) вам нужно будет проверить классы объектов, которые вы передаете, поскольку вы больше не можете гарантировать, что они являются автомобилями или людьми, и (2) вы вероятно, вы также захотите переопределить getHashCode(), особенно если вы решите сделать их обоих равными (Object), потому что getHashCode() должен возвращать одинаковые хэш-коды для двух равных объектов.