Getting a player's item in hand

Hi everyone.
I'm trying to make a plugin that if a player has a sponge, portal, end portal frame or melonstem in their hand and do /tenchant the item will be enchanted. However I'm having a problem with the server detecting that the player has the specified item in their hand. I've tried it with Item IDs however that gives me the same problem
Also if the player left/right clicks it will say the else statement message.

Code:java

 
 
package mc.katsaroucraft;
 
import java.util.logging.Logger;
 
import net.minecraft.server.v1_6_R1.Block;
 
import org.bukkit.Bukkit;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class TokenEnchant extends JavaPlugin{
    private static final Logger log = Logger.getLogger("Minecraft");
    private final TEListener playerListener = new TEListener(this);
    public static TokenEnchant plugin;
 
 
    @Override
    public void onEnable(){
        log.info("==TokenEnchant has been enabled!==");
        PluginManager pm = Bukkit.getPluginManager();
        pm.registerEvents(this.playerListener, this);
    }
    @Override
    public void onDisable(){
        log.info("==TokenEnchant has been disabled!");
    }
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
        Player player = (Player)sender;
 
 
        //Checks to see if /tenchant is said
        if((cmd.getName().equalsIgnoreCase("TEnchant"))){
            player.sendMessage("Test to make sure plugin is functioning");
            //String to match items with their respective names
            String mstem = "Melonstem";
            String spnge =  "Sponge";
            String endpFrame = "End Portal Frame";
            String portal = "Portal";
            //String of what the player is holding in game.
            String y = new ItemStack(player.getInventory().getItemInHand()).toString();
 
            if(player.isOp() || player.hasPermission("tokenenchant.use")){
                //Checks to see if player's in hand item matches up with String.
                if(spnge == y || mstem == y || endpFrame == y || portal == y){
                    player.sendMessage("Enchanting...");
 
 
                }
                else{
                    player.sendMessage("You don't have a token ");
                }
 
 
            }
 
            else{
                player.sendMessage("You don't have permissions");
            }
        }
 
 
            return false;
 
 
 
 
    }
}
 

 

Just try
if(player.getInventory().getItemInHand() == Material.SOMETHING || player.getInventory().getItemInHand() == Material.SOMETHING)){
//run code

 

Gopaintman
player.getItemInHand().getType().equals(Material.SPONGE)

 

Henzz No Material is an Enum

 

Jake0oo0
You are checking if an itemstack is equal to a Material.

 

Oops derp then:
if(player.getInventory().getItemInHand().getType() == Material.SOMETHING || player.getInventory().getItemInHand() == Material.SOMETHING)){

 

You *could* do this if you have a huge list of items.

Code:

if (player.getItemInHand() != null && Arrays.asList(Material.SOMETHING1, Material.SOMETHING2).contains(player.getItemInHand().getType())) {
    // Code         
}

However I'm unsure on how efficient it would be :x

 

Thanks! Alright you guys got it right.