Hashmap Holen gibt null zurück

Habe ich zwei Hashmaps

HashMap<Integer, Integer> inventoryRequirements = new HashMap<Integer, Integer>();
HashMap<Integer, Integer> inventory = new HashMap<Integer, Integer>();

Ich habe eine Schleife, die überprüft, aktuelle und fügt hinzu:

for(Item item : items){
    int currentRequirement = 0;
    currentRequirement = inventoryRequirements.get(item.get_id());
    inventoryRequirements.put(item.get_id(), currentRequirement++);         
}

Habe ich noch eine Schleife, die überprüft, Inventar und fügt hinzu:

for(Item item : items){
    int currentInventory = 0;
    //this next line returns null
    currentInventory = inventory.get(item.get_id());
    inventory.put(item.get_id(), currentInventory++);           
}

Die erste funktioniert einwandfrei, aber die zweite kotzt und eine null zurück. Ich verstehe nicht, warum die zweite nicht funktioniert. Beide sind zunächst in den gleichen Zustand, wie dargestellt in der code sample.

BEARBEITEN

Den beiden HM bekommen bevölkert, wie Sie hier sehen - wirklich!!!! Ich weiß, es ist schwer zu glauben, aber die erste funktioniert und die zweite nicht.

Haben Sie jemals put alles in inventory? Auch, warum nicht Map<Item, Integer>s?
Wenn inventoryRequirements beginnt leer, der erste loop wird auch scheitern.
Es wird nie auf die Linie zu setzen alles in es. Warum nicht die Karte statt - ich habe keine gute Antwort für diese - ist es ein echter Vorteil?
Jeder Aufruf get(int) auf eine leere HashMap zurück null.
Weil Sie haben offensichtlich andere code zwischen der ersten und zweiten snippet, das Sie sich nicht uns zu zeigen.

InformationsquelleAutor Roy Hinkley | 2013-07-02

  1. 12

    Sowohl die Schleifen, die Sie zeigen, werfen ein NullPointerException (NPE), wenn der Schlüssel, den Sie anfordern über get() ist nicht in der Map.

    Map.get() zurück null wenn der Schlüssel nicht vorhanden ist (oder natürlich, wenn der gespeicherte Wert mit dem Schlüssel null). Ein Integer (autoboxed-Typ), ist null können nicht autounboxed in eine int so dass es wirft eine NPE.

    Den sicheren Art und Weise der Durchführung ist dies:

    for (Item item : items) {
    Integer currentRequirement = inventoryRequirements.get(item.get_id());
    if (currentRequirement != null) {
        inventoryRequirements.put(item.get_id(), currentRequirement++);         
    }
}

    Natürlich, es ist auch absolut möglich, dass Sie eine Item in Ihrer Sammlung, die null und , dass ist, was wirft der NPE.

    +1 für den letzten Punkt, aber er sagt, dass es null zurückgeben und nicht werfen null, aber das ist auch nicht sinnvoll, da es ein int.
    Wenn get() zurück null es wirft. Oder, seine JVM/computer kaputt ist 😉 ich bin Neigung in Richtung einer Item wird null selbst.

    InformationsquelleAutor Brian Roach

  2. 1

    Wenn der code, den Sie gestellt haben, abgeschlossen ist, dann haben Sie nicht setzen alles in Ihren hashmap.
    So wird es immer wieder null.

    Wahr!!!! Aber in diesem Fall Frage ich mich, warum erste Stück code, der einwandfrei läuft ???
    Das ist unwahr. Die foreach Schleife wird nicht Durchlaufen die HashMap, wenn so ein Fall.
    Wie kommt das ?? Foreach-Schleife wird Durchlaufen, die auf der Liste<Items - >.. habe ich Kraft bekommen, warum Sie scheitern würde..
    Was er bedeutet, ist der code innerhalb der foreach-wird scheitern, weil von dem, was Brian sagt in seiner Antwort.

    InformationsquelleAutor Prasad Kharkar

  3. 0

    Sind Sie immer ein Element aus inventory aber es ist leer

    InformationsquelleAutor lleon

  4. 0

    Ist die beste Methode schreiben Sie den code in der folgenden Weise: 

    for(Item item : items){
int currentRequirement = 0;
currentRequirement = inventoryRequirements.get(item.get_id());
if(currentRequirement!=null){
//update currentRequirement only if it exists in the map.
   inventoryRequirements.put(item.get_id(), currentRequirement++); 
} else {
   //add it to the map otherwise.
   inventoryRequirements.put(item.get_id(), 1);       
}

    InformationsquelleAutor

Schreibe einen Kommentar