public boolean equals(Object that) {
//...
}Aber that.instanceof geht irgendwie nicht.Ich versuche es ganz kurz zu halten:
zeigt es (auf Englisch) sehr schön auf incl. Beispiel.Overriding equals method in Java - GeeksforGeeks
Your All-in-One Learning Portal: GeeksforGeeks is a comprehensive educational platform that empowers learners across domains-spanning computer science and programming, school education, upskilling, commerce, software tools, competitive exams, and more.www.geeksforgeeks.org
Im groben sind da mehrere Schritte üblich:
- Optional: Prüfung der Referenz - ist die Referenz gleich, dann muss man nichts weiter prüfen.
- Prüfen, ob es eine Instanz der Klasse ist, dann ein cast zu der Klasse und Vergleich der Attribute.
Das ist die einfache Variante. Das wird alles prinzipiell noch komplexer:
- Teilweise wird die Logik wichtig. So kann bei Datenbank-Entities ggf. nur die ID geprüft werden.
- hashcode kommt mit rein. Wenn equals überschrieben wird, sollte auch hashcode überschrieben werden.
- Bei abgeleiteten Klassen kommt auch der Aufruf vom Parent mit dazu ...
- Bezüglich Prüfung der Klasse kann man sich auch noch weitergehende Gedanken machen ...
Was wäre z.B. wenn man das Objekt nicht casten kann?
public boolean equals(Object that) {
boolean erg = false;
if (that instanceof AbstractPoint) {
if (((AbstractPoint) that).getX() == z.getX() && ((AbstractPoint) that).getY() == z.getY() && ((AbstractPoint) that).getZ() == z.getZ()) {
return true;
}
}
return erg;
}In diesem Fall wieder mit equals vergleichen, nicht mit ==.
Aber sowohl z.getZ() als auch ((AbstractPoint) that).getZ() liefern beide int Werte
publi class Point3D {
// ...
public boolean equals(Object that) {
if (this == that) return true;
if (that instanceof Point3D) {
Point3D other = (Point3D) that;
return other.getX() == getX() &&
other.getY() == getY() &&
other,getZ()== getZ();
}
return false;
}
) if (that instanceof Point3D) {
Point3D other = (Point3D) that;
// ...
} if (that instanceof Point3D other) {
// ...
}Wenn man da noch nach weiteren Alternativen schauen möchte:
👍
import java.util.Comparator;
public class PointComparator implements Comparator<Point> {
private Point z;
public PointComparator (Point z) {
if (z == null) {
z = new Point(0,0,0);
} else {
this.z = z;
}
}
public int compare(Point X, Point Y) throws NullPointerException {
if (X == null || Y == null) {
throw new NullPointerException();
}
int erg = 0;
int eukX = Math.abs(X.getX() - z.getX());
int eukY = Math.abs(Y.getX() - z.getX());
if (eukX < eukY) {
return -1;
}
if (eukX > eukY) {
return 1;
}
if (eukX == eukY) {
if (X.getX() == Y.getX()) {
if (X.getY() == Y.getY()) {
erg = Math.abs(X.getZ() - Y.getZ());
}
else {
erg = Math.abs(X.getY() - Y.getY());
}
}
else {
erg = Math.abs(X.getX() - Y.getX());
}
}
return erg;
}
public boolean equals(Object that) {
boolean erg = false;
if (that instanceof AbstractPoint) {
if (((AbstractPoint) that).getX() == this.z.getX() && ((AbstractPoint) that).getY() == this.z.getY() && ((AbstractPoint) that).getZ() == this.z.getZ()) {
erg = true;
} else if (that instanceof Point) {
if (((Point) that).getX() == z.getX() && ((Point) that).getY() == z.getY() && ((Point) that).getZ() == z.getZ()) {
erg = true;
}
}
}
return erg;
}
}public class Point extends AbstractPoint implements Comparable<AbstractPoint> {
public Point (int x, int y, int z) {
super(x, y, z);
}
@Override
public double euclidean(AbstractPoint that) {
if (!(that == null)) {
double erg = Math.sqrt(Math.pow((that.getX() - this.getX()), 2) + Math.pow((that.getY() - this.getY()), 2) +
Math.pow((that.getZ() - this.getZ()), 2));
return erg;
}
else {
return Double.NaN;
}
}
@Override
public int manhattan(AbstractPoint that) throws IllegalArgumentException {
if (!(that == null)) {
int erg = Math.abs(that.getX() - this.getX()) + Math.abs(that.getY() - this.getY()) + Math.abs(that.getZ() - this.getZ());
return erg;
}
else {
throw new IllegalArgumentException();
}
}
@Override
public boolean equals(Object that) {
that = (AbstractPoint) that;
if (((AbstractPoint) that).getX() == this.getX() && ((AbstractPoint) that).getY() == this.getY() && ((AbstractPoint) that).getZ() == this.getZ()) {
return true;
}
else {
return false;
}
}
public int compareTo(AbstractPoint a) {
int erg = 0;
double eukX0 = Math.sqrt(Math.pow((a.getX() - 0), 2) + Math.pow((a.getY() - 0), 2) +
Math.pow((a.getZ() - 0), 2));
double eukY0 = Math.sqrt(Math.pow((0 - this.getX()), 2) + Math.pow((0 - this.getY()), 2) +
Math.pow((0 - this.getZ()), 2));
if (eukX0 > eukY0) {
return -1;
}
if (eukX0 < eukY0) {
return +1;
}
if (eukX0 == eukY0) {
if (a.getX() == this.getX()) {
if (a.getY() == this.getY()) {
erg = Math.abs(a.getZ() - this.getZ());
}
else {
erg = Math.abs(a.getY() - this.getY());
}
}
else {
erg = Math.abs(a.getX() - this.getX());
}
}
return erg;
}
public static void sort(Point[] ps) {
}
public static void sort(Point[] ps, Point z) {
}
public static Point[] getAllMax(Point[] ps) {
return new Point[2];
}
}public abstract class AbstractPoint {
private final int x;
private final int y;
private final int z;
public AbstractPoint(int x, int y, int z) {
this.x = x;
this.y = y;
this.z = z;
}
public final int getX() {
return x;
}
public final int getY() {
return y;
}
public final int getZ() {
return z;
}
public abstract double euclidean(AbstractPoint that);
public abstract int manhattan(AbstractPoint that);
@Override
public abstract boolean equals(Object that);
@Override
public final String toString() {
return "P(" + x + "," + y + "," + z + ")";
}
}Also bei mir funktioniert die Methode equals() von Point. Der spezielle Konstruktor ist vorgegeben und auch eine equals methode
Sie funktioniert halt nicht korrekt. Übergib der Methode doch mal etwas anderes als einen Point.
Was für ein totaler Unsinn
. Bist du da sicher? Die Methode compare() vergleicht einfach die beiden übergebenen Parameter, eine Instanzvariable ist da überflüssig.
PointComparator soll vermutlich nicht die Punkte direkt, sondern deren Abstand zu einem Punkt Z vergleichen, die compare- und auch die equals-Methode sehen allerdings ziemlich grausam aus, ich würde drauf Wetten, dass dort an passende Methoden in Point delegiert werden soll.
Hm, das wäre möglich. Ist doch aber ein ziemlich doofes Konzept, das zur Fehlbenutzung förmlich einlädt. Ich erwarte von einer Klasse, die Comparator implementiert etwas anderes.
Aber was soll man denn von einer Comparator-Klasse für einen (mindestens zweidimensionalen) Punkt erwarten? Nach meiner (bescheidenen) mathematischen Einschätzung lässt sich da überhaupt keine ">"-Relation definieren. Es sei denn vielleicht, man beschränkt sich auf einen Teilaspekt, wie z.B. den Abstand im Posting von @mrBrown. Dann müsste man aber konsequenterweise auch eine "seltsame" equals-Methode vorsehen, damit es nicht widersprüchlich wird.
