Objekte in Set nicht gefunden

[L0oNY]

Hallo,

ich versuche gerade in einer Hashmap ein Value auszulesen, denn ich aber nicht finde.

Ich rufe eine Methode auf, der ich als Parameter eine Map mit Key/Value Paaren übergebe. Ich kann an dieser Stelle die Datentypen (aus technischen Gründen) der Map nicht mitübergeben. Dann hole ich mir aus dem Map den Set der Schlüssel raus und suche in dem Set nach einem Schlüssel aus dem Set, der Set bekommt den richtigen Datentyp:

//schlüssel der Map holen
Set<KomplexType> setkeys = source.keySet();
//sicherstellen, dass mind. ein schlüssel vorhanden ist
if (setkeys.iterator().hasNext()) {
// set nach schlüssel suchen
            System.out.println("Contains: "
                    + setkeys.contains(setkeys.iterator().next())); // --> false
                  }

Der Schlüssel ist ein komplexer Datentyp mit überschriebenen hashCode() Methode:

public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + ((getId() == null) ? 0 : getId().hashCode());
        return result;
    }

Wie kann es sein, dass in meiner Ausgabe false ausgegeben wird. Es macht doch keinen Sinn, was habe ich übersehen?
Das ganze ist in Java 7 geschrieben.

 

[temi]

Ich verstehe nicht ganz, was du da vorhast. Warum holst du nicht einfach den benötigten Wert direkt aus der Hashmap mit get(key)?

 

[MoxxiManagarm]

Der Schlüssel ist ein komplexer Datentyp mit überschriebenen hashCode() Methode:

contains​

boolean contains(Object o)
Returns true if this set contains the specified element. More formally, returns true if and only if this set contains an element e such that
Objects.equals(o, e).

Set (Java SE 11 & JDK 11 )

docs.oracle.com

 

[L0oNY]

contains​

boolean contains(Object o)
Returns true if this set contains the specified element. More formally, returns true if and only if this set contains an element e such that
Objects.equals(o, e).

Set (Java SE 11 & JDK 11 )

docs.oracle.com

Hallo MoxxiManagarm,

danke für die Antwort. Die equals() Methode ist ebenfalls überschrieben mit

public boolean equals(Object obj) {
        if (this == obj)
            return true;
        if (obj == null)
            return false;
        if (getClass() != obj.getClass())
            return false;
        KomplexType other = (KomplexType) obj;
        if (getId() == null) {
            return false;
        } else if (!getId().equals(other.getId()))
            return false;
        return true;
    }

Ich lege meine Hand ins Feuer und behaupte, dass sie richtig funktioniert.

PS: Der KomplexType heißt natürlich anders, ich wollte es nur vereinfacht darstellen.

 

[Oneixee5]

Hallo MoxxiManagarm,

danke für die Antwort. Die equals() Methode ist ebenfalls überschrieben mit

public boolean equals(Object obj) {
        if (this == obj)
            return true;
        if (obj == null)
            return false;
        if (getClass() != obj.getClass())
            return false;
        KomplexType other = (KomplexType) obj;
        if (getId() == null) {
            return false;
        } else if (!getId().equals(other.getId()))
            return false;
        return true;
    }

Ich lege meine Hand ins Feuer und behaupte, dass sie richtig funktioniert.

PS: Der KomplexType heißt natürlich anders, ich wollte es nur vereinfacht darstellen.

Viel zu umständlich.

 

[Oneixee5]

Hallo,

ich versuche gerade in einer Hashmap ein Value auszulesen, denn ich aber nicht finde.

Ich rufe eine Methode auf, der ich als Parameter eine Map mit Key/Value Paaren übergebe. Ich kann an dieser Stelle die Datentypen (aus technischen Gründen) der Map nicht mitübergeben. Dann hole ich mir aus dem Map den Set der Schlüssel raus und suche in dem Set nach einem Schlüssel aus dem Set, der Set bekommt den richtigen Datentyp:

//schlüssel der Map holen
Set<KomplexType> setkeys = source.keySet();
//sicherstellen, dass mind. ein schlüssel vorhanden ist
if (setkeys.iterator().hasNext()) {
// set nach schlüssel suchen
            System.out.println("Contains: "
                    + setkeys.contains(setkeys.iterator().next())); // --> false
                  }

Der Schlüssel ist ein komplexer Datentyp mit überschriebenen hashCode() Methode:

public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + ((getId() == null) ? 0 : getId().hashCode());
        return result;
    }

Wie kann es sein, dass in meiner Ausgabe false ausgegeben wird. Es macht doch keinen Sinn, was habe ich übersehen?
Das ganze ist in Java 7 geschrieben.

Ich verstehe nicht was der Datentyp damit zu tun haben soll!? Key ist Key - egal welcher Datentyp. Dein Hashcode ist ebenfalls Unsinn überlege doch mal was passiert wenn die getId() wirklich null ist.

 

[L0oNY]

Ich verstehe nicht ganz, was du da vorhast. Warum holst du nicht einfach den benötigten Wert direkt aus der Hashmap mit get(key)?

genau das ist das Problem, er findet den Key, bzw. den dazugehörigen Value nicht, obwohl beides drin ist.
Der Code oben soll nur das Problem verdeutlichen. Erwartest du da ein false aus Ausgabe?

 

[L0oNY]

Ich verstehe nicht was der Datentyp damit zu tun haben soll!? Key ist Key - egal welcher Datentyp. Dein Hashcode ist ebenfalls Unsinn überlege doch mal was passiert wenn die getId() wirklich null ist.

Die HashCode() und equals() Methoden werden mir von eine Framework generiert, somit habe ich keinen Einfluss darauf.

Du hast Recht, key ist key, warum kommt dann im dem Quellcode false raus?

 

[kneitzel]

Also auf den ersten Blick sieht das soweit korrekt aus. Was sein könnte: Wenn das erste Element des Iterators keine id hat, dann bekommst du false (Da equals dann immer false zurück gibt). Daher könntest Du die Ausgabe noch erweitern und getId() aufrufen um dies auszuschließen, also etwas wie:

//schlüssel der Map holen
Set<KomplexType> setkeys = source.keySet();
//sicherstellen, dass mind. ein schlüssel vorhanden ist
if (setkeys.iterator().hasNext()) {
    // set nach schlüssel suchen
    KomplexType firstKey = setkeys.iterator().next();
    System.out.println("Contains: "
                       + setkeys.contains(firstKey)
                       + " / "
                       + firstKey.getId(); // --> false
}

 

[L0oNY]

Also auf den ersten Blick sieht das soweit korrekt aus. Was sein könnte: Wenn das erste Element des Iterators keine id hat, dann bekommst du false (Da equals dann immer false zurück gibt). Daher könntest Du die Ausgabe noch erweitern und getId() aufrufen um dies auszuschließen, also etwas wie:

//schlüssel der Map holen
Set<KomplexType> setkeys = source.keySet();
//sicherstellen, dass mind. ein schlüssel vorhanden ist
if (setkeys.iterator().hasNext()) {
    // set nach schlüssel suchen
    KomplexType firstKey = setkeys.iterator().next();
    System.out.println("Contains: "
                       + setkeys.contains(firstKey)
                       + " / "
                       + firstKey.getId(); // --> false
}

if (setkeys.iterator().hasNext()) {
            DokInfoKey firstkey = setkeys.iterator().next();
            System.out.println("Contains: " + setkeys.contains(firstkey)
                    + " / " + firstkey.getId());
        }

liefert : "Contains: false / 8a918cf85b8a38b6015b8a53a9600006"

 

[L0oNY]

Hier ist noch die Screenshots des Eclipsedebuggers, in dem man sieht, dass es sich wirklich um das gleiche Objekt handelt, zumindest meines Verständnisses nach.

 

[LimDul]

Aus dem Bauch heraus kann das nicht sein, sofern equals und hashcode korrekt überschrieben sind.

Probier mal folgendes auszugeben:
2x hintereinander firstKey.hashCode();
firstKey.equals(firstKey);

(Ich sehe den Code oben, dass die eigentlich korrekt implementiert sind, aber andererseits kann es eigentlich nur daran liegen)

 

[L0oNY]

Um weder eure noch meine Zeit zu verschwenden werde ich dieses Rätsel einfach ungelöst sein lassen und dumm sterben.
Vor allem, weil ihr das Problem nicht nachstellen könnt.

Ich habe es jetzt so "gelöst", dass ich die Map, die ich bekomme in eine Map mit mit Typparameter überführe, also in Map<key, value>. Sollte zwar unnötig sein, der Schritt, aber hier scheint es anders nicht funktionieren.

Vielleicht ist das ja die Ursache des Problems, dass beim Vergleich nicht die equals Methode des Key Typs genommen wird, sondern die von Object.

Danke allen für die Hilfe

 

[mihe7]

Kann es sein, dass Dein KomplexType mutable ist?