Verständnisproblem mit equals()

[Thomas4321]

Hallo,

ich glaube, ich habe ein Verständnisproblem mit der equals()-Methode. Nachfolgend mein Beispielcode, der nicht das tut, was ich eigentlich von ihm erwarte.

Meine Erwartung ist: ich definiere eine equals()-Methode, die behauptet, alle Objekte der Klasse MyTest seien gleich (nicht, dass das für eine Anwendung sinnvoll wäre, aber es bringt mein Problem auf den Punkt).

Ich füge das Objekt t1 in einen Vector ein und teste dann mit contains(), ob t2 im Vector enthalten ist. Da alle MyTest-Objekte in diesem Fall gleich sind, müsste t2 bereits "enthalten" sein. Ist es aber nicht. Warum?

Die Sun-Dokumentation sagt zu contains():
"true if and only if the specified object is the same as a component in this vector, as determined by the equals method; false otherwise."

% java MyTest
true
true
false

import java.util.Vector;

class MyTest
{
        public boolean equals(MyTest e)
        {
                return true;
        }

        public static void main(String args[])
        {
                Vector v = new Vector();

                MyTest t1 = new MyTest();
                MyTest t2 = new MyTest();

                v.add(t1);
                System.out.println(v.contains(t1));

                System.out.println(t1.equals(t2));
                System.out.println(v.contains(t2));
        }
}

[/code]

 

[Leroy42]

Der typische Anfängerfehler

public boolean equals(MyTest e)

Du hast die equals-Methode nicht überschrieben sondern überladen.

Zum Überschreiben einer Methode, muß die überschreibende Methode
dieselbe Signatur (gleicher Name, gleiche Parameterliste) haben.

Deine Methode wird nie aufgerufen: Richtig ist:

public boolean equals(Object o)

 

[Thomas4321]

Danke für den Hinweis. Ich war beim Lesen der Dokumentation wohl nicht aufmerksam genug.

 

[Murray]

Deine Methode wird nie aufgerufen

Das ist nicht ganz korrekt: wenn zur Compilezeit klar ist, dass das übergebene Element vom Typ MyTest ist, dann wird die Methode sehr wohl aufgerufen; siehe auch die 2. Zeile in der geposteten Ausgabe: hier wird tatsächlich die überladene Methode aufgerufen.
Wenn aber zur Compilezeit nicht bekannt ist, welchen Typ das Argument nun wirklich hat, dann greift der kleinste gemeinsame Nenner, nämlich die von Object geerbte Methode equals( Object).

Seit Java 5 kann man - etwas Disziplin vorausgesetzt - solchen Fehlern vorbeugen: wenn man vorhat, eine Methode zu überschreiben, dann sollte man die Annotation @Override verwenden; wenn man sich dann mit der Sigantur verhaut und die Methode eben nicht überschreibt, sondern überlädt, dann bekommt man vom Compiler eine Fehlermeldung.

 

[Leroy42]

Hast recht!

Ich hätte genauer schreiben müssen, das sie nicht durch Vector-Routinen aufgerufen wird.