Item Stack abfragen

[CreepyCat]

Gute Morgen,
Ich hätte mal eine Frage. Ich möchte ein bestimmtes Item was mit einer Variable definiert ist, aus einem Spielerinventar abfragen und auch die Anzahl. Sollte dies der Fall sein, soll das Item aus dem Spielerinventar enfernt werden. Noch dazu bekommt der Spieler einen bestimmten Betrag an Geld.
Mein Problem:
Ich habe schon mehrere Varianten Probiert,bei dieser sagt er einfach immer das ich das Item nicht habe(oder nicht genug).
Hier der Code;

package Money;

import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;

public class verkaufen implements CommandExecutor {

    @Override
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {

        if (args.length == 3) {
            Player player = (Player) sender;
            String item = (args[0]);
            int menge = Integer.parseInt(args[1]);
            double preis = Double.parseDouble(args[2]);
            int mengerund = (int) Math.ceil(menge);
            for (ItemStack stack : player.getInventory().getContents()) {
                if(stack!=null) {
                    if (stack.getType() == Material.getMaterial(item)) {
                        if (stack.getAmount() >= menge) {
                            FileConfiguration config = Abfrage.getPlugin().getConfig();
                            double balance = config.getDouble(player.getName() + ".Money");
                            config.set(player.getName() + ".Money", balance + preis);
                            Abfrage.getPlugin().saveConfig();
                            Bukkit.getServer().dispatchCommand(Bukkit.getServer().getConsoleSender(), "clear " + player.getName() + " " + item + " " + mengerund);
                            sender.sendMessage(ChatColor.YELLOW + "[Money] " + ChatColor.GREEN + "Du hast " + menge + " " + item + " verkauft");
                        }
                        else {
                            sender.sendMessage(ChatColor.RED + "Du hast nicht genug" + item);
                        }
                    }
                }
                else {
                    sender.sendMessage(ChatColor.RED + "Du hast kein");
                }
            }
        }
        return false;
    }
}

 

[Taminoful]

Wieso rundest du die Anzahl der Items? Es gibt keine halben Items.

 

[CreepyCat]

Die Items werden gerundet für die Textausgabe, sonst würde stehen 32.0 Items. Das dass nicht passiert werden sie zu einem Int gemacht

 

[Taminoful]

Die Items werden gerundet für die Textausgabe, sonst würde stehen 32.0 Items. Das dass nicht passiert werden sie zu einem Int gemacht

Ja, das .0 kommt durch deine Rundung zustande, das liegt daran, dass die ceil Methode der Klasse Math einen double zurückgibt.
Das beißt dir hier nicht ins Bein, da Java hier implizit den Datentyp, durch die Automatic Type Conversion, int zu einem double erweitert, sollte aber trotzdem nicht notwendig sein, da ItemStack.getAmount einen Integer zurückgibt.

Ansonsten liegt dein Fehler vermutlich in dem Materialvergleich. Du übergibst hier in die Methode Material.getMaterial einen String. Diese Methode erfordert, dass der übergebene String exakt mit dem Namen des Materials im Material Enum übereinstimmt.
Bspw. würde "Brick" oder "brick" nicht mit Material.BRICK übereinstimmen. Eine schnelle Lösung dafür wäre den übergebenen String entsprechend zu "Uppercasen".

 

[CreepyCat]

Ok. Wie lege ich ein Uppercase drüber?

 

[Taminoful]

Ok. Wie lege ich ein Uppercase drüber?

Schau dir einmal an, was die Klasse String so für Methoden zur Verfügung stellt. Ich bin mir sicher, dass du dann schnell die passende Methode finden wirst.

 

[CreepyCat]

Jo funktioniert, danke

 

[Chrisliebär❤️]

Deine Schleife über das Inventar ist allerdings noch etwas fragwürdig. Pack dir mal mehrere Itemstacks ins Inventar, von denen einige zu klein sind und guck was passiert.

 

[CreepyCat]

Naja also wenn ich zb 32 Items verkaufen möchte und 2x 16 im inventar habe passier nichts, die müsste ich dann noch zusammenrechnen

 

[Chojo-chan]

Eigentlich kann das alles Minecraft für dich übernehmen

    /**
     * Takes the item from the inventory.
     *
     * @param player player to take items from
     */
    protected void takeFromInventory(Player player) {
        player.getInventory().removeItem(getItem());
        player.updateInventory();
    }

    /**
     * Checks if a player has a item in his inventory.
     *
     * @param player player to check
     * @return true if the player has the item in his inventory.
     */
    protected boolean hasPlayerItemInInventory(Player player) {
        PlayerInventory inventory = player.getInventory();
        return inventory.containsAtLeast(getItem(), getItem().getAmount());
    }