Is there anything wrong with this code developers?

The code I posted below. It's buggy for some reason, I get no errors and yes I have listeners and they're registered.

I have a class that controls the "BarAPI", which is a custom boss bar with custom messages. I'm showing the boss bar like this, player.getHealth/player.getMaxHealth.

So it's showing the players' normal health then its maximum health. But when it tries to update the numbers go crazy and random and it doesn't match the players current health.

Any ideas?

Code:java

@EventHandler
public void onLogin(PlayerJoinEvent e) {
final Player p = e.getPlayer();
final int health = (int)p.getHealth();
int health1 = (int)p.getMaxHealth();
final String max = Integer.toString((int)health1);
Plugin plugin = Bukkit.getPluginManager().getPlugin("TubeRed");
Bukkit.getServer().getScheduler().runTaskLater(plugin, new Runnable() {
public void run() {
try {
BarAPI.getInstance().setStatus(p, "" + ChatColor.RED + ChatColor.BOLD + health + "/" + max, (float) (((p.getHealth() + 1) / p.getMaxHealth())) * 100, true);
} catch (IllegalArgumentException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (SecurityException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (InstantiationException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (IllegalAccessException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (InvocationTargetException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (NoSuchMethodException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
}
}
}, 0);
}
@EventHandler
public void onDamage(EntityDamageEvent e) {
if(e.getEntity() instanceof Player) {
final Player p = (Player)e.getEntity();
final int health = (int)p.getHealth();
int health1 = (int)p.getMaxHealth();
final String max = Integer.toString((int)health1);
Plugin plugin = Bukkit.getPluginManager().getPlugin("TubeRed");
Bukkit.getServer().getScheduler().runTaskLater(plugin, new Runnable() {
public void run() {
try {
BarAPI.getInstance().setStatus(p, "" + ChatColor.RED + ChatColor.BOLD + health + "/" + max, (float) (((p.getHealth() + 1) / p.getMaxHealth())) * 100, true);
} catch (IllegalArgumentException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (SecurityException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (InstantiationException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (IllegalAccessException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (InvocationTargetException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (NoSuchMethodException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
}
}
}, 0);
}
}
 
 
@EventHandler
public void onPlayerRegain(EntityRegainHealthEvent e) {
if(e.getEntity() instanceof Player) {
final Player p = (Player)e.getEntity();
final int health = (int)p.getHealth();
int health1 = (int)p.getMaxHealth();
final String max = Integer.toString((int)health1);
Plugin plugin = Bukkit.getPluginManager().getPlugin("TubeRed");
Bukkit.getServer().getScheduler().runTaskLater(plugin, new Runnable() {
public void run() {
try {
BarAPI.getInstance().setStatus(p, "" + ChatColor.RED + ChatColor.BOLD + health + "/" + max, (float) (((p.getHealth() + 1) / p.getMaxHealth())) * 100, true);
} catch (IllegalArgumentException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (SecurityException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (InstantiationException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (IllegalAccessException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (InvocationTargetException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
} catch (NoSuchMethodException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
}
}
}, 0);
}
}
 

 

• You don't need to cast an int to an int. The int is already an int so there is no need to cast the int to an int. (if you think that was repetitive, look at your code.)
• Haven't used the BarAPI but,
  Code:java

  BarAPI.getInstance().setStatus(p, "" + ChatColor.RED + ChatColor.BOLD + health + "/" + max, (float) (((p.getHealth() + 1) / p.getMaxHealth())) * 100, true);

  Does that method really require all those catch statements?

Just to let you know, player's health are doubles, not integers. You'll lose information by casting a double to an int.

 

I knew I would loose information lol, thanks for the tip.

So is there anyway I could produce a health number that can be displayed?

Anyone? lol

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

p.getHealth().length;

 

JiNJaProductionz
Since when has Integer (or Double) had a variable called "length"?

 

Your whole code is just to messy..
- 1st: do what teej107 said
- 2nd: why do you use the scheduler to schedule the task? Why do you use #runTaskLater, when you want your Runnable to run in 0 ticks?
- 3rd: Please make one method like update(Player) that does your logic, and then call this method from your events. Otherwise you have your code duplicated three times
- 4th: remove all unnecessary code for debug purposes, like the ChatColors
- 5th: if you still can't find your mistake, post your updated code

 

shohouku It would be nice if you could post the output of your code. You know, show us what happens and why that's wrong? For instance. It may just be a number full of decimal places, and not a crazy and random number as you put it.