Set health on Food...

I am trying to make it so that when a player right clicks a cookie, the cookie will turn to air and the players health will be 100%.

Code:java

package me.mrgermanrain.scheal;
 
import java.util.logging.Logger;
 
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
 
    public class scheal extends JavaPlugin{
        public final Logger logger = Logger.getLogger("Minecraft");
        public static scheal plugin;
 
        @Override
        public void onDisable() {
            PluginDescriptionFile pdfFile = this.getDescription();
            this.logger.info(pdfFile.getName() + " has been Disabled!");
        }
 
        @Override
        public void onEnable() {
            PluginDescriptionFile pdfFile = this.getDescription();
            this.logger.info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " has been Enabled!");
        }
 
        public boolean onPlayerRightClick(Player p, PlayerInteractEvent event){
            Player player = (Player) p;
            if(player.getItemInHand().getType() != Material.COOKIE){
                player.setItemInHand(new ItemStack(Material.AIR));
                player.setHealth(20);
            }
            return false;
 
        }
    }

 

MrGermanrain NONONONONONO. Use a PlayerInteractEvent as a void and check if the action is right clicking air or right clicking a block, check if them item in hand is a cookie, decrease the amount of the cookies by 1, and set the player's health to 20.

 

Code:java

 
package me.mrgermanrain.scheal;
 
import java.util.logging.Logger;
 
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.event.EventHandler;
 
 
public class scheal extends JavaPlugin implements Listener{
public final Logger logger = Logger.getLogger("Minecraft");
public static scheal plugin;
 
@Override
public void onDisable() {
PluginDescriptionFile pdfFile = this.getDescription();
this.logger.info(pdfFile.getName() + " has been Disabled!");
}
 
@Override
public void onEnable() {
PluginDescriptionFile pdfFile = this.getDescription();
this.logger.info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " has been Enabled!");
getServer().getPluginManager().registerEvents(this, this);
}
 
@EventHandler
public void onFoodConsume(PlayerItemConsumeEvent event){
Player player = event.getPlayer();
if(event.getItem().getType() == Material.COOKIE){
player.setHealth(20);
}
return false;
 
}
}
 

Fixed code ^

There were a few things wrong with it.

-> You used PlayerInteractEvent, that would do it instantly and when you left or right click.. I changed it to the consume event so that it will only do it when you eat it (Might not be what you want)
-> You need to register events in onEnable()
-> You need to add @EventHandler to the events (nothing else)
-> Events need to be void, not boolean and they only have one parameter
-> If the class contains listeners it needs to implement Listener.

All code is untested, I'm about to go so I can't help you any more.

 

thanks ill test it