contains (für LIST) funktioniert nicht - ich weiß nicht warum?

[Halunken-Joe]

Moin,

ich habe die folgende Klasse UserStammDaten (int ID, String Name, String Vorname, int PLZ). Dazu eine Menge Methoden mit denen ich die Werte schreiben und auslesen kann.

Im Lauf des Programms habe ich mir eine 'List vom Typ <UserStammDaten> meineListe' erstellt in der sich z.B. alle User aus dem PLZ Bereich 12345 befinden.

Nun möchte gern wissen ob sich dieser Liste der Datensatz mit der ID=55 befindet. Dafür gibt es die Methode findUserById(int ID) mit dem Rückgabewert UserStammDaten xyz.

Folgende Abfrage funktioniert nicht, bzw gibt immer das Ergebnis 'false'

if ( meineListe.contains (findUserById(55) ) )

und das verstehe ich nicht. Vergleiche ich da Objekte deren Wert zwar übereinstimmen aber nicht deren Referenzen?

Gehe ich vorher aber einmal mit dem Iterator über die frisch erstellte List<UserStammDaten> und erstelle mir eine 'List<Integer> meineNeueListe' mit allen ID's der User die im PLZ Bereich 12345 leben, und frage dann

if ( meineNeueListe.contains (55) )

um herauszufinden ob der Datensatz mit der ID 55 darin enthalten ist, dann klappt es.

Ich würde mir gern den Schritt mit dem Iterator sparen und mit einer List weniger hantieren. Ich bin für alle Tipps dankbar.

 

[Spacerat]

Wenn "meineListe" und "meineNeueListe" die selben sind, verwaltet sie in beiden Fällen UserIDs (Integer). Mit UserStammDaten kann sie deswegen nichts anfangen.
[highlight=java]if ( meineListe.contains (findUserById(55).getID() ) );[/highlight]sieht zwar böse aus (könnte ne NullPointerException geben) aber sollte helfen dem Problem auf die Schliche zu kommen.
@Edit: besser wäre natürlich, die Liste auf UserStammDaten umzurüsten.

 

[hdi]

[HIGHLIGHT="Java"]if ( meineListe.contains (findUserById(55) ) ) [/HIGHLIGHT]

meiner Meinung nach liegt hier im contains() das Problem. die Methode contains() einer Liste
nutzt intern einen equals()-Ablgleich, der für eigene Klassen so per sé nicht (sinnig) implementiert ist.
D.h. er weiss gar nicht, was genau er bei den Objekten vergleichen soll, wenn er
danach in der Liste sucht - es kommt immer false raus.
Woher soll Java auch wissen, wann zwei UserStammDaten gleich sind? Das ist eine
x-beliebige Klasse, die irgendetwas beschreibt, das du dir ausgedacht hast.
Ich denke also, du musst die euqals-Methode in der Klasse UserStammDaten überschreiben.

Am logischsten so:

[HIGHLIGHT="Java"]
@Override
public boolean equals(Object other){
if(other == null) return false;
if(!(other instanceof UserStammDaten)) return false;
UserStammDaten usd = (UserStammDaten)other;
return this.id == usd.id;
}[/HIGHLIGHT]

...deswegen klappt übrigens dein 2.Versuch mit

[HIGHLIGHT="Java"]if ( meineNeueListe.contains (55) ) [/HIGHLIGHT]

denn ein Integer ist eine Standard-Klasse, und deren equals() Methode ist richtig implementiert.
(edit: Hier stand eine Halbwahrheit)

Auf jeden Fall: Jede eigens geschrieben Klasse erbt ja implizit von Object und bekommt damit
diese equals-Methode vererbt. Ich weiss nicht wie die bei Object genau aussieht, aber
die Implementierung macht bei den meisten Klassen keinen Sinn.
Also überschreibt man sie, wenn Java wissen muss wann 2 Instanzen dieser Klasse gleich sind.
Das muss es zB (!) bei contains() auf einer Liste wissen

edit: Übrigens sollte man hashCode() auch überschreiben, wenn man equals überschreibt.
(Das ist auch eine von Object geerbte Methode).
Allerdings kann ich dir dazu nichts weiter sagen, weil ich das selber noch nicht verstehe.
Es gab hier aber vor paar Tagen einen ausführlichen Thread darüber, find ihn grad nicht.
Mach dich mal bei google usw schlau
(Für deinen Zweck wird das glaube ich nicht nötig sein, irgendwas mit hashCode zu machen,
aber es "gehört sich so")

 

[Ebenius]

hdi, in Zeile 4 fehlen Klammern.

Object prüft bei equals() auf Instanzgleichheit (this == theOther).

hashCode() nicht vergessen, wenn man equals() implementiert!

Ebenius

 

[hdi]

Danke, habs korrigiert!

 

[hdi]

Hab jetzt den Hashcode Thread gefunden (danke Ebenius für push )

http://www.java-forum.org/java-basi...1069-equals-hashcode-contract.html#post502795

 

[mvitz]

...
[HIGHLIGHT="Java"]
@Override
public boolean equals(Object other){
if(other == null) return false;
if(!(other instanceof UserStammDaten)) return false;
UserStammDaten usd = (UserStammDaten)other;
return this.id == usd.id;
}[/HIGHLIGHT]
...

Hier sei hoffentlich noch der Hinweis gestattet, dass
[HIGHLIGHT="Java"]if (other == null) return false;[/HIGHLIGHT]
überfllüssig ist, da dies bei
[HIGHLIGHT="Java"]if(!(other instanceof UserStammDaten)) return false;[/HIGHLIGHT]
mit erledigt wird.

 

[Halunken-Joe]

Vielen Dank für die ausführlichen Antworten. Jetzt ist es mir die Sache etwas klarer. Ich bin gerade dabei das Programm zu durchforsten und bin auf 93 contains gestossen. Hoffentlich dauert es nicht allzu lange die alle durchzusehen.