[SOLVED]Switching effect food

[NeoSup2130 1]

I'm having trouble with this code:

 

protected void onFoodEaten(ItemStack item, World world, EntityPlayer player) {

 

player.addPotionEffect(new PotionEffect(Potion.heal.id, 1, 0));

{

if (item == MainClass.CorruptedBlood)

 

player.addPotionEffect(new PotionEffect(Potion.poison.id, 1, 0));

}

}

What I'm trying to code is, when you drink 'CorruptedBlood' you get poison instead of healing

The problem starts with this line : if (item == MainClass.CorruptedBlood) When i hover over the problem it says : Incompatible operand types ItemStack and item

[diesieben07 6692]

You cannot compare ItemStacks and Items. Basic java.

You don't need to check those though, the method will only be called for your Item anyways.

[NeoSup2130 1]

Sorry this may sound stupid but, i don't get what you mean. When i remove item == it still doesn't fix it. The class is used by multiple foods and i want that 'Corruptedblood' has a different effect in the class than healing the player. And i don't want to make a second class just for one food.

[NeoSup2130 1]

My code: https://github.com/NeoSup2130/Neo/blob/master/BottleOfBloodClass

[diesieben07 6692]

Check

this == <someItem>

.

What I meant was that you cannot check if an ItemStack is equal to an Item, because it never is. Same as an Apple is never a Banana.

[NeoSup2130 1]

Thanks for helping but, i still don't get it. I'm just gonna make a second class that will be easier.

[diesieben07 6692]

...

[daafganggdg 3]

Thanks for helping but, i still don't get it. I'm just gonna make a second class that will be easier.

Alright look, its like saying "2" == 2, for our human eye its 2 and 2 and thats ofc the same but for a computer its a string and an integer and you can never ever compare these types like that.

Same as an itemStack is not an item.

you can use item.getItem tho..

Then you have the item of the itemStack and you can compare it with a item

[larsgerrits 510]

...

Then you have the item of the itemStack and you can compare it with a item

You don't need to check those though, the method will only be called for your Item anyways.