Spigot Block Lokalisierung + Casting block als Chest

[BrutalVerzockt]

Hallo Minecraft-Server.EU-ler,
ich brauche eure Hilfe um meinen Code weiterzuverwenden,
ich code derzeitig an einem Chest-Shop-System und versuche aus einer Kiste die einen Block weit weg ist das Inventar
aufzurufen und eine Vorschleife zu erstellen und mit dem ersten Item, dass sich in dieser Kiste befindet auszugeben und den Type als dritte Zeile vom Schild zu setzen.

               try {
                    int i = Integer.parseInt(e.getLine(1));
                    if (i >= 1) { // Anzahl
                        int i2 = Integer.parseInt(e.getLine(2));
                        if (i2 >= 0) { //Preis
                            if (e.getLine(3).equalsIgnoreCase("?")) {
                                e.getPlayer().sendMessage(Strings.prefix + "Du hast §aerfolgreich §7ein Verkaufsschuld erstellt");
                                e.setLine(0, "§7Verkaufe§8§l:");
                                e.setLine(1, i + " Mal");
                                e.setLine(2, "§6" + i2 + "$");

                                if(e.getBlock().getLocation().subtract(0, 0, 1).getBlock().getType().equals(Material.CHEST)) {
                                    Chest chest = (Chest) e.getBlock().getLocation().subtract(0, 0, 1).getBlock();
                                    for (ItemStack is : chest.getInventory()) {
                                        if (is.getType() != Material.AIR) {
                                            int i3 = chest.getInventory().first(is);
                                            ItemStack is1 = chest.getInventory().getItem(i3);
                                            e.setLine(3, is1.getType().toString());
                                        }
                                    }
                                } else
                                    e.getPlayer().sendMessage("TREST");
                            }
                        } else
                            e.getPlayer().sendMessage(Strings.prefix + "Die §aZahl §7in §2Zeile 3 §7muss §e0 §7und oder §emehr §7sein");
                    } else
                        e.getPlayer().sendMessage(Strings.prefix + "Die §aZahl §7in §2Zeile 2 §7muss §e1 §7und oder §emehr §7sein");
                } catch (NumberFormatException e1) {
                    e.getPlayer().sendMessage(Strings.prefix + "Die angegebenen Zahlen sind §4§n§lungültig");
                }

Ich weiß, dass sich der Code nicht ganz an den Conventions orientiert, aber es ist auch nicht für die Veröffentlichung geplant. Ich danke schonmal allen für eure Beiträge und hoffe, dass ich auch etwas aus diesem Thread mitnehmen kann

 

[DerFrZocker]

Und was genau funktioniert daran nicht, gibt es eine Fehlermeldung? Passiert etwas anderes als erwartet?

 

[SirYwell]

Chest in org.bukkit.block ist ein Subinterface von BlockState, nicht von Block. Ein Block hat aber die Methode getState(), das zurückgegebene Objekt kannst du dann casten.

 

[BrutalVerzockt]

Für den Fehlerhaften Cast habe ich eine else Abzweigung geschireben @DerFrZocker

 

[BrutalVerzockt]

@SirYwell, ich danke dir für deinen Beitrag allerdings bin ich in der Praxis etwas schlechter als in der Theorie , Meintest du, ich solle Chest chest = (Chest )e.getBlock().getLocation().subtract(0, 0, 1).getBlock().getState(); so casten ?

 

[SirYwell]

Jup, das sollte stimmen. Ob der Rest dann wie erwartet funktioniert, würd ich jetzt aber nicht direkt garantieren, so genau bin ich den Code nicht durchgegangen.

 

[BrutalVerzockt]

Okay, das Casten hat funktioniert, aber jetzt habe ich das Problem, dass an meiner For-Schleife etwas nicht ganz stimmt.

for (ItemStack is : chest.getInventory()) {
                                        if (is.getType() != Material.AIR) {
                                            int i3 = chest.getInventory().first(is);
                                            ItemStack is1 = chest.getInventory().getItem(i3);
                                            e.setLine(3, is1.getType().toString());
                                        }
                                    }

 

[Gelöschtes Mitglied 135174]

Der ItemStack könnte in diesem Fall auch null sein, also muss das vor dem Aufruf von ItemStack#getType() überprüft werden.