Checking a player has an item in their slot

Working on a kitpvp soup server and for some reason this never returns true?

Code:

p.getInventory().getItem(0) == new ItemStack(Material.MUSHROOM_SOUP)

Thanks in advance

 

Try p.getInventory().getItem(0).getType() == Material.MUSHROOM_SOUP instead.

 

@HeadGam3z
Thanks it works but it launches an error too...

Code:

[17:51:39 WARN]: [MindlessKits] Task #3 for MindlessKits v1.0 generated an excep
tion
java.lang.NullPointerException
        at Customkits.minecraft.other.Refiller.ingame(Refiller.java:139) ~[?:?]
        at Customkits.minecraft.essentials.MainClass$1.run(MainClass.java:174) ~
[?:?]
        at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftTask.run(CraftTask.java
:53) ~[bukkit.jar:git-Bukkit-1.6.4-R2.0-26-g31d7c5f-b2943jnks]
        at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.mainThreadHea
rtbeat(CraftScheduler.java:345) [bukkit.jar:git-Bukkit-1.6.4-R2.0-26-g31d7c5f-b2
943jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.u(MinecraftServer.java:5
83) [bukkit.jar:git-Bukkit-1.6.4-R2.0-26-g31d7c5f-b2943jnks]
        at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:2
50) [bukkit.jar:git-Bukkit-1.6.4-R2.0-26-g31d7c5f-b2943jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:5
41) [bukkit.jar:git-Bukkit-1.6.4-R2.0-26-g31d7c5f-b2943jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java
:453) [bukkit.jar:git-Bukkit-1.6.4-R2.0-26-g31d7c5f-b2943jnks]
        at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:6
17) [bukkit.jar:git-Bukkit-1.6.4-R2.0-26-g31d7c5f-b2943jnks]

 

@benzimmer123
use an if statement something like:
if(player.getInventory().contains(Material.MUSHROOM_SOUP))

 

@ProMCKingz
I'm trying to check if their hotbar has soup in it not the whole inventory.

 

@benzimmer123
You get an NPE, because you aren't checking if it is null, make a null checker before

 

@benzimmer123 Check if the slot is null, not if the inventory contains the item.

 

@SuperOriginal
Thanks very much!

@ProMCKingz
Only just saw your comment, thanks too :3

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Just a note, using the == operator against anything where you create a new instance will always return false.

 

@1Rogue
Ah! Never knew that thanks, will keep that in mind next time