Porównanie dwóch obiektów za pomocą equals()

Question

Opis problemu: Stworzyłem prostą klasę Punkt opisującą punkt w przestrzeni 3D. Posiada 3 pola typu float reprezentujące wartośc x,y,z. Wszystkie są prawidłowo ustawione.

Napisałem funkcę porównującą dwa punkty w następujący sposób:

private static boolean comparePoints(Punkt p1, Punkt p2){
	return p1.equals(p2);
	}

Niestety funkcja ta zawsze zwrace false. Jakie mogą byćtego przyczyny?

Czy rozwiazanie tkwi nadpisaniu metody equals(), hashCode()?

 

Wywołanie metody:

System.out.println(comparePoints(p1,p2));

 

Answer 1

Punkt p1, Punkt p2

p1.equals(p2); //false

equals sprawdza czy dwie referencje do obiektow sa identyczne
a to oznacza że 

Punkt p1 = new Punkt(); 
Punkt p2 = p1; //referencja

p1.equals(p2) // wynik da true

Answer 2

A co dajesz w argument tej funkcji? W sensie jak ją wywołujesz - czym?

Comments

System.out.println(comparePoints(p1,p2));

 

---

No ale co to jest to p1 i p2? co jest nad tym?

---

p1.setPointValues(p1, sc);

p2.setPointValues(p2, sc);

gdzie:

private void setPointValues(Punkt point, Scanner sc)
	{
		point.setX(getUserPoint(sc));
		point.setY(getUserPoint(sc));
		point.setZ(getUserPoint(sc));
	}

 

Answer 3

Wywołujesz metodę equals klasy Object, która porównuje 2 obiekty. Metoda ta zawsze zwróci Ci false, jeśli będziesz porównywał 2 różne obiekty, nawet jeśli wszystkie pola tych obiektów będą identyczne. Jedyną opcją kiedy metoda equals klasy Object zwróci Ci true jest porównanie tego samego obiektu (w Twoim przypadku p1.equals(p1); )

Answer 4

Mikrokontroler xD i adas94 mają racje. Porównywałem tylko referencje tych dwóch obiektów, a przecież one są różne.

Rozwiazaniem problemu tkwi w odpowiednim nadpisaniu metody equals() tak aby porównywała każde pole obiektu, sprawdzała czy nie jest nullem, sprawdzała czy nie jest to ten sam obiekt i czy porównywane obiekty należą do tej samej klasy.

 

Dzięki za pomoc :) !