"Random" errors with Soup PvP Plugin

I'm coding a basic kitpvp server with instant soup healing. However, there are a few unresolved variables even though I used craftbukkit and bukkit in my plugin. The bolded and underlined variables are the ones that "cannot be resolved to a variable": food, health, and bowl:

Code:java

package eliteKits.events;
 
import org.bukkit.Material;
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.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
 
public class InventoryListener implements Listener {
    @EventHandler
    public void onPlayerInteract (PlayerInteractEvent event) {
        Player player = event.getPlayer();
        Material mat = player.getItemInHand().getType();
 
        if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
            if (mat == Material.MUSHROOM_SOUP) {
                ItemStack bowl = new ItemStack(Material.BOWL);
                double health = player.getHealth();
                if (health == 20) {
                    int food = player.getFoodLevel();
                    if (food == 20);
                        return;
                    }
                    int nfood = [U][B]food[/B][/U] + 7;
                    if (nfood > 20) {
                        player.setFoodLevel(20);
                    }
                    else {
                        player.setFoodLevel(nfood);
                    }
                    player.getInventory().setItem(player.getInventory().getHeldItemSlot(), bowl);
                    event.setCancelled(true);
                    return;
            } else {
                double nhealth = [U][B]health[/B][/U] + 7;
                if (nhealth > 20) {
                    player.setHealth(20);
                } else {
                    player.setHealth(nhealth);
                }
                event.setCancelled(true);
                player.getInventory().setItem(player.getInventory().getHeldItemSlot(), [U][B]bowl[/B][/U]);
                return;
            }
 
        }
    }
}
 

I'm pretty new to Java, please help! I find it strange that there is an exact same instance of this line of code:

Code:text

player.getInventory().setItem(player.getInventory().getHeldItemSlot(), [U][B]bowl[/B][/U]);

but it has no errors.

Thanks in advance!

 

Initialize your health variable outside of your if statement. Since you are initializing the health variable inside the if statement, your else statement will never have an instance of the health variable. nhealth is trying to access a variable that hasn't been declared or initialized.

 

On line 23 you used a semicolon instead of a opening curly bracket, replace it with one! Also, the code at the end of your if-block and else-block are identical, you could remove them and include one copy behind the else-block. The return statement is excessive by the way

 

Thanks, but I'm really new and am not sure how to fix this, because I used many if statements and I'm not sure where to initialize it.

Nevermind, thank you guys so much!

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