If the player does not Have the specific item remove potion effect

So im making this plugin for a guy and i have never done this type of plugin. If the player does not have a CHEST in his/her inventory it will remove a potion effect i have added. I tried the code below but it no work! PLEASE HELP ASAP

Code:java

                if(!(player.getInventory().contains(Material.CHEST));
                player.removePotionEffect(PotionEffectType.SLOW, Integer.MAX_VALUE, 0);

 

I don't think CHEST is a Material... try Material.DIAMOND_CHESTPLATE ?

Edit:
It actually is a material. But not what you want to check for. You can check for a specific iteam (if the inventory contains it) or check if the chestplate armour is worn.

 

Im using CHEST for my code and if they player has the chest it will make them slow.... so i need help with if the player does not have the chest the potion effect goes away

 

Well when do you run this code?
Can you post the full code?

 

Code:java

  public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
                if (sender instanceof Player) {
                Player player = (Player)sender;
 
                if (cmd.getName().equalsIgnoreCase("TradeChest")) {
                    if(!player.hasPermission("cie.trade.chest")) {
                          sender.sendMessage(ChatColor.RED + "You Do Not Have Permission To use this Command");
                          return true;
                      }
                        player.sendMessage(ChatColor.GREEN + "You Know have Chest");
                        player.getInventory().clear();
                        ItemStack HeavyChest = new ItemStack(Material.CHEST);
                        ItemMeta meta1 = (ItemMeta) HeavyChest.getItemMeta();
                        meta1.setDisplayName(ChatColor.GREEN + "TradeChest");
                        HeavyChest.setItemMeta(meta1);
                        player.getInventory().addItem(HeavyChest);
                        player.addPotionEffect(new PotionEffect(PotionEffectType.SLOW, Integer.MAX_VALUE, 0));
                }
                if(!(player.getInventory().contains(Material.CHEST));
                player.removePotionEffect(PotionEffectType.SLOW, Integer.MAX_VALUE, 0);
 
 
                } return false;
          }
}

The Fancy Whale

 

Ula492 Player#removePotionEffect() only takes in one arguement (which is the PotionEffectType)

 

First of all check this line:

Code:java

if(!(player.getInventory().contains(Material.CHEST));

Next as of now you always get the chest if you do or do not have the chest. (not sure if you intended that)
Also, if you use .equalsIgnoreCase() you shouldn't use capitals in there (just a syntax thing)
But your main issue is the line I pointed out above.

EDIT: Also as xTigerRebornx pointed out you are using removePotionEffect() wrong.

 

So how do i fix it so it detects if the player has the chest or dosent? and if it does it keeps the Potion effect and if it dosent it will remove the potion effect and i also need help so i dont need to use a command give a chest slowness

 

Please re-read this line and tell me if you see anything wrong:

Code:java

if(!(player.getInventory().contains(Material.CHEST));

To be honest it seems you do not have too much knowledge of java at the moment. I suggest learning java before jumping into coding plugins. I made that mistake at first as well.

 

IK i tried that just incase it was right ik it is wrong thats why i posted this and not to be shited on

 

Look at the end. It is an IF statement. You do not end an IF statement with a semicolon. Also, swearing was not really necessary I am just trying to help. Also, if you expect a response please do tahg me.

 

The Fancy Whale OK! I kinda figured it out But i still ned help with the potion remove :/

Code:java

if(player.getItemInHand(Material.CHEST))
                player.removePotionEffect(PotionEffectType.SLOW, Integer.MAX_VALUE, 0);

 

Please check out this link for the removing potion effect part:
http://jd.bukkit.org/rb/apidocs/org...ionEffect(org.bukkit.potion.PotionEffectType)
**Bukkit java docs can be VERY helpful.
Also, if you do not know how to use an IF statement, I do not believe you are ready to make bukkit plugins. Again I suggest you learn java first.