Minecraft Plugin

Question

マインクラフトのプラグインを作って試してみると、謎のエラーが出ました。
いろいろ他の方法で試してみたんですけど、毎回エラーが出たりエラーが出なかったときは書いたことがが動作しなかったりします。

    public void InventoryDrop(Player p) {
        ItemStack[] is = new ItemStack[36]; // インベントリースロット指定
        for(int i = 0; i < is.length; i++) {
            is[i] = new ItemStack(p.getInventory().getItem(i)); // インベントリーのアイテムをゲット
            p.getWorld().dropItem(p.getLocation().add(0,1,0),is[i]); // ゲットしたアイテムを落とす。
            p.getInventory().setItem(i, new ItemStack(Material.AIR)); // 落としたアイテムをインベントリーから消去する。
        }
    }

このコードはインベントリーのアイテムをすべて落とすというコードです。
これを実行すると、

[02:15:23 ERROR]: null
org.bukkit.command.CommandException: Unhandled exception executing command 'testdrop' in plugin Repick_Test v1.0
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46) ~[Spigot.jar:git-Spigot-db6de12-18fbb24]
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:141) ~[Spigot.jar:git-Spigot-db6de12-18fbb24]
        at org.bukkit.craftbukkit.v1_8_R3.CraftServer.dispatchCommand(CraftServer.java:641) ~[Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.PlayerConnection.handleCommand(PlayerConnection.java:1162) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.PlayerConnection.a(PlayerConnection.java:997) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.PacketPlayInChat.a(PacketPlayInChat.java:45) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.PacketPlayInChat.a(PacketPlayInChat.java:1) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.PlayerConnectionUtils$1.run(SourceFile:13) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) [?:?]
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) [?:?]
        at net.minecraft.server.v1_8_R3.SystemUtils.a(SourceFile:44) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.MinecraftServer.B(MinecraftServer.java:715) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.DedicatedServer.B(DedicatedServer.java:374) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.MinecraftServer.A(MinecraftServer.java:654) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:557) [Spigot.jar:git-Spigot-db6de12-18fbb24]
        at java.base/java.lang.Thread.run(Thread.java:832) [?:?]
Caused by: java.lang.IllegalArgumentException: Cannot copy null stack
        at org.apache.commons.lang.Validate.notNull(Validate.java:192) ~[Spigot.jar:git-Spigot-db6de12-18fbb24]
        at org.bukkit.inventory.ItemStack.<init>(ItemStack.java:135) ~[Spigot.jar:git-Spigot-db6de12-18fbb24]
        at com.gmail.leakerpvp.repicktest.RepickTest.InventoryDrop(RepickTest.java:34) ~[?:?]
        at com.gmail.leakerpvp.repicktest.RepickTest.onCommand(RepickTest.java:24) ~[?:?]
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44) ~[Spigot.jar:git-Spigot-db6de12-18fbb24]
        ... 15 more

というエラーが出ます。
インベントリーをすべて埋めたらエラーは出ないんですけど一マスでも空いていたら、エラーが出ます。
エラー内容か対処法があったら教えてください。

Comments

[m.ts10806]

ドキュメントにもちゃんとあるので「謎のエラー」ではないかと思いますが、
https://hub.spigotmc.org/javadocs/bukkit/org/bukkit/command/CommandException.html
これに限らず、大抵のことは「既知のエラー」です。
調べたこと試したことを記載してください。

[quickquip]

is[i] = new ItemStack(p.getInventory().getItem(i));
の行がRepickTest.javaの34行目で合ってますか?

Answer 1

まずここを読んでみましょう。
https://hub.spigotmc.org/javadocs/bukkit/org/bukkit/inventory/Inventory.html#getItem-int-
@Nullable getItem(int index)となっていますが、
@Nullableというのはこの関数がnullを返す可能性があることを示しています。
具体的には、そのスロットにアイテムがなかった場合にnullを返すのでしょう。

次にnew ItemStackの方を見てみましょう。
https://hub.spigotmc.org/javadocs/bukkit/org/bukkit/inventory/ItemStack.html
こちらには、
ItemStack(@NotNull ItemStack stack)となっています。
@NotNullは、逆にnullは受け付けない事を示しています。

つまり、@Nullable getItem(int index)で返されたnullを、
nullを受け付けられないnew ItemStack(@NotNull ItemStack stack)にそのまま渡しているのが原因です。

渡してしまう前に == nullなどでnullが返っていないか確認する必要があります。

ちなみに、
Caused by: java.lang.IllegalArgumentException
とエラーが出ていますが、IllegalArgumentExceptionは引数がおかしい場合に返される例外です。
その後に続く
Cannot copy null stack
というのは、そのまんまの意味で「nullをコピーできません」という事になります。
エラー内容と行数を見れば原因は大きく絞られるので、質問する前にもう少し調べてみましょう。

Comments

[退会済みユーザー]

とても分かりやすく回答してくれたおかげで、解決出来ました！ありがとうございます！
今度からはもうちょっと調べてから質問します！