Java String in Hashmap als Key NULL

[OnDemand]

Moin,

hab ein ganz merkwürdiges Verhalten. Untenstehender Code fügt mir Werte in die Map als null, obwohl die nicht null sind.

Der String lautet zb "Packung" ich sehe ihn im Debugger auch, aber in der Map landet es als Null ohne Fehler ohne alles. Hat jemand ne Idee was hier los sein könnte? Total misteriös

Map<String, Attribute> attributeMap = new HashMap<>();
    for (Attribute attribute : response.getBody()) {
        attributeMap.put(attribute.getName(), attribute);
    }
return attributeMap;

Hier sieht man im Debugger die Response, 13 Einträge welcje in die Map sollen


Und so sieht die Map nach 13 Durchläufen aus (mehr als 13 Einträge) sowas hab ich ja noch nie erlebt. Dachte erst an Intellij Cache oder so, aber im produktiven rennt das System an anderer Stelle gegen die Wand, weil ein Eintrag in der Map fehlt, der aber eigentlich da sein müsste

 

[thecain]

Gibt getName wirklich name zurück? Was ist der Unerschied zwischen Attribute und WooAttribute?
Sonst brauchts mehr Code grundsätzlich sieht das nämlich scon OK aus...

 

[OnDemand]

Ja das sollte getName zurück geben. Also der Getter ist standard. Ich werd das mal ausgeben lassen.
Hab jetzt spaßeshalber mal eine Prüfung reingemacht, ob ein Key schon existiert.

Unterschied gibts keinen, hab die Variablen nur umbenannt: So siehts nun aus

  Map<String, WooAttribute> wooAttributeMap = new HashMap<>();
            for (WooAttribute wooAttribute : response.getBody()) {
                if(wooAttributeMap.containsKey(wooAttribute.getName())==false) {
                    wooAttributeMap.put(wooAttribute.getName(), wooAttribute);
                }
            }
            return wooAttributeMap;

 

[OnDemand]

Also das macht mich ratlos:

Map<String, WooAttribute> wooAttributeMap = new HashMap<>();
    for (WooAttribute wooAttribute : response.getBody()) {
        if(wooAttributeMap.containsKey(wooAttribute.getName())==false) {
            System.out.println("woo "+wooAttribute.getName());
            wooAttributeMap.put(wooAttribute.getName(), wooAttribute);
        }
}
return wooAttributeMap;

Die Ausgabe:
woo Akku
woo Deck
woo Durchmesser
woo Farbe
woo Füllmenge
woo Geschmack
woo Leistung
woo Merkmale
woo Model
woo Packung
...

Die Map:




Jetzt teste ich es mal mit Stream:

Map<String, WooAttribute> wooAttributeMap = new HashMap<>();
    Map<String, WooAttribute> map = response.getBody().stream()
    .collect(Collectors.toMap(WooAttribute::getName, Function.identity()));
return map;

Edit: Collectors klappt auch nicht. sieht genau so aus. häää

 

[KonradN]

Überlege doch einmal, was für einen Typ Du da hast. Das ist eine HashMap! Was Du da siehst sind also vermutlich schlicht leere Buckets. (Siehe https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/util/HashMap.html)

Mit was schaust Du Dir die HashMap da denn an? Was ist das für ein Tool? IDEs sollten da eigentlich sowas ausblenden.
Und du dürftest da eine size = 13 sehen, denn die HashMap speichert auch die Größe.

Edit: Zumindest das wäre meine Erklärung für das, was Du da siehst

 

[Robert Zenz]

Sei vorsichtig wenn du so etwas betrachtest, du siehst dir hier gerade den internen Zustand der Map an. Was du hier siehst ist die, glaube ich, interne Art die Daten zu speichern und @KonradN war schneller...fast...fast!

 

[OnDemand]

@KonradN , ja das ist richtig. Ich habe mit Absicht mal die NULLer anzeigen lassen in IntelliJ. Ich sehe, dass es 13 Einträge hat aber wenn ich an anderer Stelle Packung rausholen möchte, dann bekomm ich ne NPE. Auch zurecht, denn "Packung" ist nicht in der Map


obwohl es definitiv in der Response vorkommt



Über Sout wird es mir auch alles gut ausgegeben, nur nicht in die Map gepackt.

Size ist 13 aber es sind nur 11 Einträge zu sehen, wo sind die anderen 2 und warum sind die nicht erreichbar?

 

[OnDemand]

Wenn ich mir die Map mit toString() anschaue, ist die Packung drin

{Durchmesser=WooAttribute(links=null, name=Durchmesser, orderBy=null, id=59, type=label, slug=pa_durchmesser, hasArchives=null), Zuganzahl=WooAttribute(links=null, name=Zuganzahl, orderBy=null, id=62, type=label, slug=pa_zuganzahl, hasArchives=null), Füllmenge=WooAttribute(links=null, name=Füllmenge, orderBy=null, id=45, type=label, slug=pa_fuellmenge, hasArchives=null), Stärke=WooAttribute(links=null, name=Stärke, orderBy=null, id=55, type=label, slug=pa_staerke, hasArchives=null), Deck=WooAttribute(links=null, name=Deck, orderBy=null, id=50, type=label, slug=pa_deck, hasArchives=null), Akku=WooAttribute(links=null, name=Akku, orderBy=null, id=47, type=label, slug=pa_akku, hasArchives=null), Packung=WooAttribute(links=null, name=Packung, orderBy=null, id=54, type=label, slug=pa_packung, hasArchives=null), Geschmack=WooAttribute(links=null, name=Geschmack, orderBy=null, id=44, type=label, slug=pa_geschmack, hasArchives=null), Merkmale=WooAttribute(links=null, name=Merkmale, orderBy=null, id=46, type=label, slug=pa_merkmale, hasArchives=null), Farbe=WooAttribute(links=null, name=Farbe, orderBy=null, id=57, type=select, slug=pa_farbe, hasArchives=null), Widerstand=WooAttribute(links=null, name=Widerstand, orderBy=null, id=56, type=label, slug=pa_widerstand, hasArchives=null), Model=WooAttribute(links=null, name=Model, orderBy=null, id=58, type=label, slug=pa_model, hasArchives=null), Leistung=WooAttribute(links=null, name=Leistung, orderBy=null, id=52, type=label, slug=pa_leistung, hasArchives=null)}

na das soll mal einer verstehen.

 

[KonradN]

Wenn Du da die Buckets siehst: In einem Bucket können mehrere Elemente sein. Das ist die Funktionalität der HashMap. Daher hast Du bei jedem Bucket auch den Haken zum öffnen (nehme ich an).

Daher noch einmal: Mach Dich mit den Grundlagen der jeweiligen Klassen vertraut! Es macht keinen Sinn, sich das Innere einer Instanz anzusehen, wenn man keine Ahnung hat, was diese Instanz an Daten wie vorhält.

 

[Robert Zenz]

Jetzt verstehe ich was du meinst! Kann es sein dass du hier gerade zwei Sachen mischt?

Einmal die interne Art der HashMap die Dinge zu speichern, und einmal dass es Attribute gibt welche bei getName beinhart null retournieren?

Lass' dir mal in der Schleife ausgeben welche Attribute du gerade in die Map einfuegst:

Map<String, Attribute> attributeMap = new HashMap<>();
    for (Attribute attribute : response.getBody()) {
        System.out.println(attribute.getName() + " -> " + attribute);
        attributeMap.put(attribute.getName(), attribute);
    }
return attributeMap;

Sollte die Sache eigentlich klarer machen.

 

[OnDemand]

Also hier sind nun alle 13 Einträge drin im EntrySet - aber nicht in Table. Jedoch wird das Objekt mit dem Key Packung nicht gefunden wenn ich

wooAttributeMap.get("Packung");

mache.

Das ist extrem seltsam, nutze HashMap nicht zum ersten mal, aber sowas ist mir neu.

getName gibt niemals null zurück (siehe System.out) oben, da ist kein Eintrag null.

 

[Robert Zenz]

Ja, ich sehe. Ohne jetzt ein reproduzierbares Beispiel ist das jetzt aber schwer zu sagen. Importierst du die richtige java.util.HashMap?

 

[OnDemand]

Hab ich auch schon überprüft, das passt (leider)

import java.util.HashMap;
import java.util.Map;

 

[Robert Zenz]

Ansonsten kannst du bitte mal folgendes machen:

for (Attribute attribute : response.getBody()) {
	String key = attribute.getName();
	
	System.out.println("Key: " + key + "@" + System.identityHashCode(key));
	System.out.println("Attribute: " + System.identityHashCode(attribute));
	System.out.println("Map size before insert: " + attributeMap.size());
	
	attributeMap.put(key, attribute);
	
	System.out.println("Map size after insert: " + attributeMap.size());
	System.out.println("Attribute: " + System.identityHashCode(attributeMap.get(key)));
	System.out.println();
	System.out.println();
}

 

[OnDemand]

Mach ich direkt mal, danke. Baut und dauert nen Moment. Hab auch nochmal den RAM geprüft - ist ausreichend frei. Will ja alles ausschließen

 

[KonradN]

Sowas in der Art wäre auch mein Gedanke - ich hätte da mal vom Key die Bytes ausgegeben um sicher zu gehen, dass es da nicht irgend ein nicht angezeigtes Sonderzeichen gibt oder so. Aber dazu könnte man auch hashCode() nutzen - dann kann man das Vergleichen mit "Package".hashCode().

 

[httpdigest]

Hab auch nochmal den RAM geprüft - ist ausreichend frei. Will ja alles ausschließen

Naja, ich würde erstmal auf die wahrscheinlichsten Ursachen gucken, statt die unwahrscheinlichsten auszuschließen...
Du hast zu 100% irgendwo einen Fehler im Code, den du gerade nicht siehst.
Du sagst zwar, dass bei wooAttributeMap.get("Packung"); das "Objekt nicht gefunden wird", aber wie ganz exakt stellst du das denn fest?
Also hast du im Debugger wirklich einen "Watch" auf diese Expression an der Stelle gesetzt, wo sie das Objekt liefern sollte?
Hast du das Ergebnis dieser Expression z.B. auch mit einem einfach System.out.println() ausgeben lassen?
Oder verarbeitest du das Ergebnis davon nur irgendwo anders, wo es aber aufgrund eines Programmierfehlers nicht ankommt? Z.B. lokale Variable statt Instanzvariable zugeweisen, etc.
Die Ursache wird eine ganz ganz einfache sein, nehme ich an.

Das allerwichtigste ist aber: Bau dir ein MCVE! Ein "Minimal", "Complete" and "Verifiable" "Example".

 

[Robert Zenz]

@KonradN Ja, habe ich mir auch gerade noch gedacht. Aber das sollte keine Rolle spielen, Kollision gibt es anscheinend auch nicht (sonst waere Attribute@13875 irgendwo anders einsortiert worden).

Also eine Version mit HashCode vom Schluessel:

for (Attribute attribute : response.getBody()) {
	String key = attribute.getName();
	
	System.out.println("Key: " + key + "@" + System.identityHashCode(key) + " -> " + key.hashCode());
	System.out.println("Attribute: " + System.identityHashCode(attribute));
	System.out.println("Map size before insert: " + attributeMap.size());
	
	attributeMap.put(key, attribute);
	
	System.out.println("Map size after insert: " + attributeMap.size());
	System.out.println("Attribute: " + System.identityHashCode(attributeMap.get(key)));
	System.out.println();
	System.out.println();
}

Etwas jucken tut mich die Ausgabe "Attribute@13875", weil das ist kein Identity-Hash wie er von einer "normalen" JDK erzeugt wird..nund die beiden fehlenden Eintraege sind "13875" und "13876" was irgendwie komisch riecht.

 

[OnDemand]

@Robert Zenz

Key: Akku@469557378
Attribute: 314443162
Map size before insert: 0
Map size before after: 1
Attribute: 314443162
Key: Deck@1771324552
Attribute: 428580073
Map size before insert: 1
Map size before after: 2
Attribute: 428580073
Key: Durchmesser@1778835175
Attribute: 1573041624
Map size before insert: 2
Map size before after: 3
Attribute: 1573041624
Key: Farbe@381225708
Attribute: 1393598470
Map size before insert: 3
Map size before after: 4
Attribute: 1393598470
Key: Füllmenge@255704421
Attribute: 1192867119
Map size before insert: 4
Map size before after: 5
Attribute: 1192867119
Key: Geschmack@430516554
Attribute: 1332722409
Map size before insert: 5
Map size before after: 6
Attribute: 1332722409
Key: Leistung@769388788
Attribute: 660703405
Map size before insert: 6
Map size before after: 7
Attribute: 660703405
Key: Merkmale@2001868402
Attribute: 1171800707
Map size before insert: 7
Map size before after: 8
Attribute: 1171800707
Key: Model@930502300
Attribute: 477729855
Map size before insert: 8
Map size before after: 9
Attribute: 477729855
Key: Packung@2080370756
Attribute: 2083809762
Map size before insert: 9
Map size before after: 10
Attribute: 2083809762
Key: Stärke@2037584462
Attribute: 695974195
Map size before insert: 10
Map size before after: 11
Attribute: 695974195
Key: Widerstand@763498888
Attribute: 1915094691
Map size before insert: 11
Map size before after: 12
Attribute: 1915094691
Key: Zuganzahl@329681867
Attribute: 1742740141
Map size before insert: 12
Map size before after: 13
Attribute: 1742740141

Packung wird hinzugefügt, wie es scheint aber es ist einfach nicht erreichbar. Das Objekt mit der Packung als Key, ist einfach nicht drin. Ich hab mir auch mal die Klassen angesehen, letzte Änderung vor 4 Monaten. Seit heut tritt das Problem erst auf.

 

[Robert Zenz]

Doch, ist es.

Key: Packung@2080370756
Attribute: 2083809762
Map size before insert: 9
Map size before after: 10
Attribute: 2083809762

Es kann auch wieder geholt werden mit dem Schluessel.

Dann spinnen wir die Sache mal weiter:

for (Attribute attribute : response.getBody()) {
	String key = attribute.getName();
	
	System.out.println("Key: " + key + "@" + System.identityHashCode(key) + " -> " + key.hashCode());
	System.out.println("Attribute: " + System.identityHashCode(attribute));
	System.out.println("Map size before insert: " + attributeMap.size());
	
	attributeMap.put(key, attribute);
	
	System.out.println("Map size after insert: " + attributeMap.size());
	System.out.println("Attribute: " + System.identityHashCode(attributeMap.get(key)));
	System.out.println();
	System.out.println();
}

// Refetch
String refetchKey = "Packung";
System.out.println("Refetch-Key: " + refetchKey + "@" + System.identityHashCode(refetchKey) + " -> " + refetchKey.hashCode());
System.out.println("Attribute: " + System.identityHashCode(attributeMap.get(refetchKey)));

return attributeMap;

Wenn das jetzt auch geht, wird das Ding korrupt irgendwo zwischen deinem erzeugen und deinem auslesen.

 

[httpdigest]

Packung wird hinzugefügt, wie es scheint aber es ist einfach nicht erreichbar. Das Objekt mit der Packung als Key, ist einfach nicht drin.

Und das stellst du ganz genau wie fest?
Bevor du jetzt auf die Idee kommst, in Astronomie-Daten zu schauen, ob es vielleicht seit heute irgendwelche Sonneneruptionen gab, die irgendwelche Teilchen durch deinen PC geschossen haben, würde ich an deiner Stelle erstmal ein reproduzierbares Beispielprogramm bauen.

 

[OnDemand]

@Robert Zenz scheint zu klappen.

Key: Akku@538512198 -> 2042676
Attribute: 1024106044
Map size before insert: 0
Map size after insert: 1
Attribute: 1024106044


Key: Deck@63772175 -> 2126025
Attribute: 1605258934
Map size before insert: 1
Map size after insert: 2
Attribute: 1605258934


Key: Durchmesser@148632002 -> -1574203861
Attribute: 908549293
Map size before insert: 2
Map size after insert: 3
Attribute: 908549293


Key: Farbe@1571903752 -> 67648890
Attribute: 1883731958
Map size before insert: 3
Map size after insert: 4
Attribute: 1883731958


Key: Füllmenge@658886109 -> -604340866
Attribute: 722157484
Map size before insert: 4
Map size after insert: 5
Attribute: 722157484


Key: Geschmack@2091207388 -> -1927646634
Attribute: 924220025
Map size before insert: 5
Map size after insert: 6
Attribute: 924220025


Key: Leistung@1043624547 -> 1868995325
Attribute: 1278270550
Map size before insert: 6
Map size after insert: 7
Attribute: 1278270550


Key: Merkmale@446012582 -> -433115266
Attribute: 517273663
Map size before insert: 7
Map size after insert: 8
Attribute: 517273663


Key: Model@65943276 -> 74517257
Attribute: 1232509099
Map size before insert: 8
Map size after insert: 9
Attribute: 1232509099


Key: Packung@2133876111 -> 857610261
Attribute: 1184651094
Map size before insert: 9
Map size after insert: 10
Attribute: 1184651094


Key: Stärke@876040255 -> -1804714007
Attribute: 1241069010
Map size before insert: 10
Map size after insert: 11
Attribute: 1241069010


Key: Widerstand@878236999 -> 283450199
Attribute: 282722941
Map size before insert: 11
Map size after insert: 12
Attribute: 282722941


Key: Zuganzahl@509746929 -> -1905820796
Attribute: 506851146
Map size before insert: 12
Map size after insert: 13
Attribute: 506851146


Refetch-Key: Packung@1823863590 -> 857610261
Attribute: 1184651094

 

[httpdigest]

Übrigens: Wenn das hier StackOverflow wäre, hätte es bereits reihenweise downvotes und requests-for-close für deine initiale Frage gehagelt, weil einfach von außen das Problem überhaupt nicht analysiert werden kann, da vermutlich 90% des Codes (inklusive dem Teil, der den eigentlichen Fehler enthält) nicht sichtbar ist.
Du musst dringend bei Problemen ausführlichere Problembeschreibungen bereitstellen und im besten Fall ein minimales, reproduzierbares Beispiel beilegen, bzw. von dir aus aus deinem eigenen Interesse heraus zur Lösung des Problems selbst bauen.
Ansonsten bist du einfach nur auf das Wohlwollen der Leute angewiesen, die hier ihre Lebenszeit dafür opfern, mit dir zusammen das Problem zu analysieren und dir notwendige Informationen aus der Nase zu ziehen versuchen.

 

[Robert Zenz]

Tja, dann ist das jetzt dein Problem. Irgendwo zwischen deinem return und deinem get wird das Ding veraendert. Das koennen wir dir aber nicht sagen wo. Irgendwas macht was komisches damit, aber das musst jetzt du suchen.

Das und beachte dass du die Instanzen von Attribute direkt verwendest, eventuell haben die einen anderen Lebenszyklus als du erwarten wuerdest.

 

[OnDemand]

Ich weis @httpdigest. Die Zeit der Leute weiss ich sehr zu schätzen und ich fordere hier auch niemanden auf zu helfen.

Wenn du nicht möchtest ist das völlig ok. Meine initale Frage stellt das Problem für mich ausreichend dar meiner Meinung nach. Dachte ich zumindestens... Meine Methode wie ich die Werte rein stecke und das Ergebnis aus dem Debugger.

Vielen Dank @Robert Zenz dann kann ich mich mal weiter auf die Suche machen. Ich habe nun mal lokal gebaut und per Hand veröffentlicht > funktioniert! Wenn Jenkins aber baut, dann gehts in die Hose.

Danke euch.

 

[Robert Zenz]

Vielen Dank @Robert Zenz dann kann ich mich mal weiter auf die Suche machen. Ich habe nun mal lokal gebaut und per Hand veröffentlicht > funktioniert! Wenn Jenkins aber baut, dann gehts in die Hose.

Unterschiedliche JDK (sowohl Build als auch Runtime)?

Mal abgesehen davon wuerde ich dann jeden Schritt durch den die Map durchgereicht wird pruefen ob es dort passiert ist und dann Stueck fuer Stueck rantasten. Irgendwo muss es sich ja aendern.

Multithreading auf der Map hast du ausgeschlossen? Also du schiebst die nicht in ein Feld einer Klasse wo von mehreren Ecken darauf zugegriffen wird?

 

[OnDemand]

Unterschiedliche JDK (sowohl Build als auch Runtime)?

Mal abgesehen davon wuerde ich dann jeden Schritt durch den die Map durchgereicht wird pruefen ob es dort passiert ist und dann Stueck fuer Stueck rantasten. Irgendwo muss es sich ja aendern.

Multithreading auf der Map hast du ausgeschlossen? Also du schiebst die nicht in ein Feld einer Klasse wo von mehreren Ecken darauf zugegriffen wird?

Ich glaube es sind verschiedene JDK, muss ich mal prüfen. Multithreading hab ich an der Stelle nirgendwo. Ich glaube schlicht, Jenkins hat was nicht richtig ausgecheckt. Muss ich mir morgen mal in aller Ruhe ansehen. Danke nochmals für Deine Hilfe und einen angenehmen Abend.

 

[mihe7]

Mal ins Blaue hinein: das Auslesen passiert an anderer Stelle und da wird schlicht eine andere Instanz der HashMap verwendet. Evtl. fehlt irgendwo die Zuweisung des Rückgabewerts?

Füge hier

Map<String, Attribute> attributeMap = new HashMap<>();
    for (Attribute attribute : response.getBody()) {
        attributeMap.put(attribute.getName(), attribute);
    }
return attributeMap;

unmittelbar vor dem return System.out.println(attributeMap.get("Packung"); ein. Ich gehe jede Wette ein, dass dort kein null rauskommt (sofern der Eintrag existiert, versteht sich).