Hunger Depleting Help

I'm working on a plugin that needs hunger to be lost at half speed. I am trying to accomplish this through using a configuration that stores a fake food level. The fake food level is out of 40 so that they lost it twice as slowly. For some reason I don't seem to lose hunger though. Here is the code (When I should lose hunger I get no numbers and nothing happens. When I gain/eat food the chat says 1, and 7):

Code:java

 @EventHandler
public void onJoin(PlayerJoinEvent event){
if (this.getConfig().contains(event.getPlayer().getUniqueId().toString()))return;
this.getConfig().set(event.getPlayer().getUniqueId().toString(), 40);
this.saveConfig();
}
 
 
 
@EventHandler
public void onFood(FoodLevelChangeEvent event){
if (!(event.getEntity() instanceof Player)) return;
 
Player player = (Player) event.getEntity();
player.sendMessage("1");
int oldFoodLevel = player.getFoodLevel();
int newFoodLevel = event.getFoodLevel();
if (oldFoodLevel > newFoodLevel){
player.sendMessage("2");
 
if ( (this.getConfig().getInt(player.getUniqueId().toString()) % 2) == 0 ) {
player.sendMessage("3");
int a = this.getConfig().getInt(player.getUniqueId().toString()) - 1;
this.getConfig().set(player.getUniqueId().toString(), a);
this.saveConfig();
player.setFoodLevel(this.getConfig().getInt(player.getUniqueId().toString()) % 2);
}
else {
player.sendMessage("4");
if (this.getConfig().getInt( player.getUniqueId().toString()) <= 1){
player.sendMessage("5");
player.setFoodLevel(0);
this.getConfig().set(player.getUniqueId().toString(), 0);
this.saveConfig();
}
else{
player.sendMessage("6");
player.setFoodLevel(this.getConfig().getInt(player.getUniqueId().toString()) % 2);
int a = this.getConfig().getInt(player.getUniqueId().toString()) - 1;
this.getConfig().set(player.getUniqueId().toString(), a);
this.saveConfig();
}
}
}
else {
player.sendMessage("7");
player.setFoodLevel(newFoodLevel);
int a = newFoodLevel * 2;
this.getConfig().set(player.getUniqueId().toString(), a);
}
}

Any ideas how to get 2-6 to trigger?

 

I would just cancel every second event for each player (you will need to filter out when a player eats).

 

The Fancy Whale

What is ("" + player.getUniqueId())? Perhaps you should go back and review some Java.

Other than this, why not cancel every other FoodLevelChangeEvent? Wouldn't that make orders of magnitude more sense?

 

("" + player.getUniqueId()) is ok, I wouldn't do it myself, but using the implicit toString() might look nicer to some people, and it makes little difference.

 

hintss

Pretty sure the explicit toString() is what everyone and their mother would recommend due to readability, this way is confusing and frankly silly, and I see way too much of it here.

 

This may not be half the speed but it is my original way of nerfing hunger.

Code:java

@EventHandler
public void onHunger(FoodLevelChangeEvent evt){
if(evt.getFoodLevel() < ((Player)evt.getEntity()).getFoodLevel()){
if(new Random().nextInt(2) == 0)
evt.setCancelled(true);
}
}

It might interest you too... turns your 40 lines of code down to 6.

What it basically does is whenever the player loses hunger, it'll generate a random number between 0 and 1. If that number is 0, the event is cancelled.

 

But that way is also random.
mythbusterma Personally, don't care too much about how pretty the code looks. More about getting the job done. I'd prefer help with the actual problem if that's okay with you.

I'll try doing that. And I'll just use the same oldFood and newFood filer to check if food is eaten.