Метод переопределения равный дает ошибку

Question

Я студент-программист, и я начал Java неделю назад. Я изучал эти общие типы, и я хотел смешивать их с равными и Overriding, поэтому я написал программу, которая создает объект под названием Punto с двумя атрибутами (pointX, pointY), чтобы имитировать координаты. Я написал статический метод вне основного класса, который использует два «Puntos» в качестве параметров и равен им. Вот код этого метода:

public static boolean iguales(PuntoImpl<Double> p1, PuntoImpl<Double> p2){ 
    return p1.equals(p2); 
} 

И вот моя попытка переопределения равна:

@Override 
public boolean equals(final Object obj) 
{ 
    if (obj == null || !(obj instanceof PuntoImpl)) 
     return false; 

    PuntoImpl<T> other = (PuntoImpl<T>) obj; 

    if (other.puntoX != this.puntoX)  return false; 
    if (other.puntoY != this.puntoY)  return false; 

    return true; 
} 

Пытаюсь равно две точки с одинаковыми параметрами координат X и координату Y, но она возвращает я ложный. Можете ли вы помочь мне найти ошибку?

Answer 1

Вы сравниваете Double значения по ссылочному равенству. Я подозреваю, что вы хотите if (!other.puntoX.equals(this.puntoX)) и т.д. Я бы на самом деле писать этот код, как:

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

    PuntoImpl<T> other = (PuntoImpl<T>) obj; 

    return other.puntoX.equals(this.puntoX) && 
      other.puntoY.equals(this.puntoY); 
} 

Не забудьте переопределить hashCode, а также.

Также обратите внимание, что сравнение значений с плавающей запятой для точного равенства часто дает неожиданные результаты. Вместо переопределения equals вы можете указать способ нахождения расстояния между точками, чтобы вы могли сравнить их с определенным допуском.