Enchantment in Schleife erstellen

[Sabbertran]

Moin,
ich arbeite im Moment an einem Plugin, dass einem Spieler ein Item mit angegebenen Verzauberungen in die Hand geben soll.
Das ganze funktioniert auch schon, nur wollte ich den Code durch eine For-Schleife verschönern und verkürzen.
Der ItemStack wird erstellt und die Enchantments werden drauf gemacht, nur wenn der Spieler das Item erhalten soll funktioniert das nicht.

Mein Code:

                    Player invpl = Bukkit.getServer().getPlayer(args[0]);
                    ItemStack Item = new ItemStack(Integer.parseInt(args[1]));
                    
                    for(int i = 0; i< args.length; i++)
                    {
                        for(int j = 0; j < (args.length/2)-1;)
                        {
                            Enchantment ench = Enchantment.getById(Integer.parseInt(args[2] + j));                            
                            Item.addUnsafeEnchantment(ench, Integer.parseInt(args[3] + j));
                            invpl.getInventory().addItem(Item);
                            j = j+2;
                        }
                    }

Der Fehler:

2013-06-22 21:13:48 [INFO] Sabbertran issued server command: /ie Sabbertran 1 16 50
2013-06-22 21:13:48 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'ie' in plugin InvEnchant v1.0
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46)
	at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:189)
	at org.bukkit.craftbukkit.v1_5_R3.CraftServer.dispatchCommand(CraftServer.java:523)
	at net.minecraft.server.v1_5_R3.PlayerConnection.handleCommand(PlayerConnection.java:965)
	at net.minecraft.server.v1_5_R3.PlayerConnection.chat(PlayerConnection.java:883)
	at net.minecraft.server.v1_5_R3.PlayerConnection.a(PlayerConnection.java:840)
	at net.minecraft.server.v1_5_R3.Packet3Chat.handle(Packet3Chat.java:44)
	at net.minecraft.server.v1_5_R3.NetworkManager.b(NetworkManager.java:292)
	at net.minecraft.server.v1_5_R3.PlayerConnection.d(PlayerConnection.java:109)
	at net.minecraft.server.v1_5_R3.ServerConnection.b(SourceFile:35)
	at net.minecraft.server.v1_5_R3.DedicatedServerConnection.b(SourceFile:30)
	at net.minecraft.server.v1_5_R3.MinecraftServer.r(MinecraftServer.java:581)
	at net.minecraft.server.v1_5_R3.DedicatedServer.r(DedicatedServer.java:226)
	at net.minecraft.server.v1_5_R3.MinecraftServer.q(MinecraftServer.java:477)
	at net.minecraft.server.v1_5_R3.MinecraftServer.run(MinecraftServer.java:410)
	at net.minecraft.server.v1_5_R3.ThreadServerApplication.run(SourceFile:573)
Caused by: java.lang.NullPointerException
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftMetaItem.applyEnchantments(CraftMetaItem.java:323)
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftMetaItem.applyToItem(CraftMetaItem.java:293)
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftItemStack.setItemMeta(CraftItemStack.java:356)
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftItemStack.setItemMeta(CraftItemStack.java:338)
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftItemStack.<init>(CraftItemStack.java:97)
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftItemStack.<init>(CraftItemStack.java:90)
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftItemStack.asCraftCopy(CraftItemStack.java:69)
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftInventory.firstPartial(CraftInventory.java:254)
	at org.bukkit.craftbukkit.v1_5_R3.inventory.CraftInventory.addItem(CraftInventory.java:281)
	at me.sabbertran.invenchant.InvEnchant.onCommand(InvEnchant.java:51)
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44)
	... 15 more

Ich hoffe, dass mir jemand behilflich sein kann.

Gruß Sabbertran

 

[TimBone]

Caused by: java.lang.NullPointerException

 

[Sabbertran]

Das habe ich auch schon gesehen, nur weiß ich nicht, worauf sich das bezieht.

Ich führe den befehl so aus: /ie Sabbertran 1 16 5

 

[games6471]

Ach Exceptions sind doch so schon gemacht. Du bekommst sogar schon die Zeile und die Quelle (falls nicht obfuskatet) angezeigt.

me.sabbertran.invenchant.InvEnchant.onCommand(InvEnchant.java:51)

 

[Sabbertran]

Auch das habe ich schon gesehen. In der Zeile steht das folgende:

invpl.getInventory().addItem(Item);

Ich verstehe allerdings nicht, was daran falsch ist bzw. warum es nicht funktioniert, zumal es außerhalb der Schleife so funktioniert hat

 

[games6471]

Schon beim ersten Durchlauf schlägt das Teil fehl, da kein Enchantment mit der ID 160 existiert. Somit wäre Enchantment schon null.

Integer.parseInt(args[2] + j);

Hier werden zwei String aneinander gefügt und es werden keine mathematischen Berechnungen durchgeführt, da die VM immer noch args[2] als String kennt. So wäre es richtig:

final Enchantment ench = Enchantment.getById(Integer.parseInt(args[2][COLOR="#FF0000"])[/COLOR] + j);

Nun wird der Wert auch ausgerechnet.

Wenn man sich die zweite Schleife mal genauer anschaut, macht diese auch nicht wirklich sinn.

Noch eine Frage: Wozu die erste Schleife? Soll man etwa pro Argument ein Item bekommen (Sogar immer das gleiche)?


Außerdem solltest du noch die Konventionen einhalten.

 

[Sabbertran]

Ok, danke für die Hilfe.
ich hatte den Fehler grade gefunden.
Und zwar habe ich das Enchantment mit der ID args[x] + j erstellt und nicht aus args[x+j] ausgelesen.
Deswegen hat er die Enchantment ID nicht erkannt und das Enchantment war null.
Zur 1. Schleife hatte ich mir etwas gedacht, weiß es nur jetzt auch nicht mehr. Funktioniert auch ohne, und ich weiß nicht, was mich da geritten hat
Zu den Konventionen: Ich will es grade nur einmal proof of concept haben und bin nur am probieren

Danke für die Hilfen

Gruß Sabbertran