Speed Clicking Ice

Im coding it so if you right click ice, it gives you speed2 for 5 seconds, and takes 1 ice away. Problem is, you can right click any block, and it works. Can anyone try to fix my code?
Here it is:

Code:

package me.Alex.FrostIce;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.OfflinePlayer;
import org.bukkit.Sound;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
public class Code extends JavaPlugin implements Listener {
   
    public void onEnable(){
        Bukkit.getServer().getPluginManager().registerEvents(this, this);
    }
   
    @EventHandler
        public void InventoryClick(InventoryClickEvent e){
        Player p1 = e.getPlayer();
        if(getClick().isRightClick().p1.getInventory().getItemInHand().getType() == Material.ICE);
        p1.getInventory().removeItem(new ItemStack(Material.ICE, 1));
        p1.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 100 , 2));
        p1.playSound(null, Sound.GLASS, 1, 0);
        p1.sendMessage(ChatColor.AQUA + "You have used your speed item!");
  }
}

 

MCraftGamer35

Code:java

 @EventHandler
public void InventoryClick(InventoryClickEvent e){
Player p1 = e.getPlayer();
if(getClick().isRightClick() && p1.getInventory().getItemInHand() != null && p1.getInventory().getItemInHand().getType() == Material.ICE) {
p1.getInventory().removeItem(new ItemStack(Material.ICE, 1));
p1.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 100 , 2));
p1.playSound(null, Sound.GLASS, 1, 0);
p1.sendMessage(ChatColor.AQUA + "You have used your speed item!");
}
}

 

Im getting two errors Jalau http://gyazo.com/3e0f028d67d0c3a8e6451432dac95ca9

 

MCraftGamer35
e.getClick() and (Player) e.getWhoClicked() ... don't blindly copy/paste code.