Health problem

I am updating one of my plugins finally and I am having problems with the Health and ItemStack, if you have any ideas please let me know Thanks!

Code:java

package me.mrgermanrain.moresoup;
 
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.Event.Result;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
 
public class MoreSoupEvents extends JavaPlugin implements Listener {
 
Boolean sh = getConfig().getBoolean("soupheals");
int hps = getConfig().getInt("hps");
 
@SuppressWarnings("deprecation")
@EventHandler
public void onPlayerFoodConsume(PlayerInteractEvent e) {
Player p = e.getPlayer();
 
if (sh != false) {
if (p.getItemInHand().getType() == Material.MUSHROOM_SOUP && p.hasPermission("moresoup.usesoup")) {
if (e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK) {
e.setUseItemInHand(Result.DENY);
p.getInventory().removeItem(new ItemStack(e.getItem().getTypeId(), 1));
} if (p.getHealth() != 20) {
p.setHealth(p.getHealth() + hps);
 
}
}
}
}
}

 

Not sure if you are updating from before this, but health was changed to a double awhile ago.

 

So how would I fix this? I think that I have read this before in "I didn't break your plugin." But my English isn't perfect so I didn't fully duplicate this.

 

Change things like:

Code:

int hps = getConfig().getInt("hps");

To:

Code:

Double hps = getConfig().getInt("hps");

Code:

} if (p.getHealth() != 20.0D) {
//continue
}

So that you are initializing things as Doubles instead of Integers, then that should fix it. Also your English is pretty good.

 

No he needs to change it to

Code:

double hps = getConfig().getDouble("hps");

haha

 

here an example i have used

Code:

Player target = Bukkit.getServer().getPlayer(args[0]);
                    Damageable dtarget = (Damageable)target;
                    if(target !=null){
                        sender.sendMessage(ChatColor.GOLD + "" + target.getName() +"'s" + ChatColor.RED +
                                " health is at " + dtarget.getHealth() *5 + "%." ); 

 

Am i the only one who misinterpreted the title a bit?

 

Oops, missed that part. Kinda just typed it fast outside of an IDE

 

Make sure you aren't giving them more than 20 health or you will get an error.

 

You don't need this line:

Damageable dtarget = (Damageable)target;

There just is not need.

 

i know just didnt

I know just was to lazy to change it btw love the videos keep them up!

 

Thanks!