ItemEntfernung

[Walfisch]

Ich hab diese kleine Methode geschrieben um eine bestimmte Anzahl an Items aus einem Inventar zu entfernen (für ein ShopSystem). Das funktioniert auch so weit, biss man zwei- (oder mehrmals) den selben Stack im Inventar hat (z.B. 64 + 64 oder 32 + 32 + 32). Dann werden direkt alle gleichen Stacks entfernt, obwohl ich nur einen entfernen will. (-> Amount)

public static void removeInventoryItems(Inventory inv, Material type, int amount) {
        for (ItemStack is : inv.getContents()) {
            if (is != null && is.getType() == type) {
                int newamount = is.getAmount() - amount;
                if (newamount > 0) {
                    is.setAmount(newamount);
                    break;
                } else {
                    inv.remove(is);
                    amount = -newamount;
                    if (amount == 0)
                        break;
                }
            }
        }
    }

Vielen Dank schon mal im Voraus!

 

[JTK222]

Probier mal amount -= newamount;
Deine Variante hat den Stack Negativ gemacht
Also anstelle von:
64 - 32 = 32
64 = -32

 

[Walfisch]

Probier mal amount -= newamount;
Deine Variante hat den Stack Negativ gemacht
Also anstelle von:
64 - 32 = 32
64 = -32

Funktioniert leider auch nicht.

 

[JTK222]

Ah stimmt also:

                } else {
                    inv.remove(is);
                    amount -= newamount;
                    if (amount == 0)
                        break;
                }

an dieser Stelle ist newamount immer negativ.
Von daher war = -newamount eigentlich richtig sorry dafür ein Denkfehler meiner seits.

Dass einzige was mir jetzt noch einfallen würde ist mit einem Label zu arbeiten.

 

[xKev]

public static void removeInventoryItems(Inventory inv, Material type, int amount) {
        for (ItemStack is : inv.getContents()) {
            if (is != null && is.getType() == type) {
                int newamount = is.getAmount() - amount;
                if (newamount > 0) {
                    is.setAmount(newamount);
                    break;
                } else {
                    inv.remove(is); <--
                    amount = -newamount;
                    if (amount == 0)
                        break;
                }
            }
        }
    }

Das Problem liegt in der Zeile inv.remove(is);
Aus der Javadoc von Inventory#remove:
Removes all stacks in the inventory matching the given stack.
This will only match a slot if both the type and the amount of the stack match.

Es werden also alle Stacks entfernt mit dieser Methode wenn du mehrmals den gleichen Stack im Inventar hast.

Edit: JTK222 war schneller

 

[Walfisch]

Ah stimmt also:

                } else {
                    inv.remove(is);
                    amount -= newamount;
                    if (amount == 0)
                        break;
                }

an dieser Stelle ist newamount immer negativ.
Von daher war = -newamount eigentlich richtig sorry dafür ein Denkfehler meiner seits.

Dass einzige was mir jetzt noch einfallen würde ist mit einem Label zu arbeiten.

Das Problem liegt in der Zeile inv.remove(is);
Aus der Javadoc von Inventory#remove:
Removes all stacks in the inventory matching the given stack.
This will only match a slot if both the type and the amount of the stack match.

Es werden also alle Stacks entfernt mit dieser Methode wenn du mehrmals den gleichen Stack im Inventar hast.

Edit: JTK222 war schneller

Schonmal danke für die Hilfe, wie kann man das ganze denn über Labels lösen (Code)?
Grüße Walfisch

 

[JTK222]

for(loop1){
   for(loop2){
      break;
   }
}

Durch dass break landest du wieder im 1. Loop

label1:
for(loop1){
   for(loop2){
      break label1;
   }
}

Mit dem Label brichst du die 1. Schleife ab.

Eigentlich sollte es bei dir kein Problem darstellen, vielleicht entsteht aber ein Compiler Fehler.
Mal schauen wenn dass nichts Hilft bin ich ahnungslos