Me and my friend have a item called UraniumIngot and we want it to give us a wither effect when were holding it

 

Items Class

package com.team.ne.init;

import com.team.ne.client.MainVariables;

import net.minecraft.client.Minecraft;
import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.item.Item;
import net.minecraftforge.client.model.ModelLoader;
import net.minecraftforge.fml.common.registry.GameRegistry;

public class NEItems {
public static Item CadiumIngot;
public static Item ElectroliumIngot;
public static Item DialuxIngot;
public static Item UraniumIngot;

public static void init(){
	CadiumIngot = new Item().setCreativeTab(NETabs.eetab).setMaxStackSize(64).setUnlocalizedName("CadiumIngot");
	ElectroliumIngot = new Item().setCreativeTab(NETabs.eetab).setMaxStackSize(64).setUnlocalizedName("ElectroliumIngot");
	DialuxIngot = new Item().setCreativeTab(NETabs.eetab).setMaxStackSize(64).setUnlocalizedName("DialuxIngot");
	UraniumIngot = new Item().setCreativeTab(NETabs.eetab).setMaxStackSize(64).setUnlocalizedName("UraniumIngot");
}

public static void register(){
	GameRegistry.registerItem(CadiumIngot, CadiumIngot.getUnlocalizedName().substring(5));
	GameRegistry.registerItem(ElectroliumIngot, ElectroliumIngot.getUnlocalizedName().substring(5));
	GameRegistry.registerItem(DialuxIngot, DialuxIngot.getUnlocalizedName().substring(5));
	GameRegistry.registerItem(UraniumIngot, UraniumIngot.getUnlocalizedName().substring(5));
}

public static void registerRenders(){
	registerRender(CadiumIngot);
	registerRender(ElectroliumIngot);
	registerRender(DialuxIngot);
	registerRender(UraniumIngot);
}

public static void registerRender(Item item){
	ModelLoader.setCustomModelResourceLocation(item, 0, new ModelResourceLocation(item.getRegistryName(), "inventory"));
}
}

When you are holding it or when it is in your inventory either way use onUpdate(...) in your item class.

Create a class that extends

Item

and overrides

Item#onUpdate

.

 

In your override, check if the

entityIn

argument is an instance of

EntityLivingBase

. If it is, cast it to

EntityLivingBase

and call

EntityLivingBase#isPotionActive

to check if it has the

MobEffects.WITHER

effect active. If it doesn't, create a

PotionEffect

and call

EntityLivingBase#addPotionEffect

to add it.

 

Create and register an instance of this class instead of

Item

.

I dont understand. I made a new class Called UraniumIngotEffect and Overrided onUpdate i dont understand you if statement and casting

Heres what i have so far.

package item;

import net.minecraft.entity.Entity;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.world.World;

public class UraniumIngotEffect extends Item{

@Override
public void onUpdate(ItemStack stack, World worldIn, Entity entityIn, int itemSlot, boolean isSelected) {
	if(entityIn = EntityLivingBase )
	super.onUpdate(stack, worldIn, entityIn, itemSlot, isSelected);
}

}

I dont understand. I made a new class Called UraniumIngotEffect and Overrided onUpdate i dont understand you if statement and casting

 

Then you don't know Java well enough to make a mod. Learn Java properly before trying to make a mod.

I dont understand the way your wording it. Im having a problem. I thought fourms were to help... But forreal me and my friend want to continue. Could you please tell us what were doing wrong.

I dont understand the way your wording it. Im having a problem. I thought fourms were to help... But forreal me and my friend want to continue. Could you please tell us what were doing wrong.

Look up casting in programming and you will see.

So we sorta figured it out were gonna keep following his instuctions but heres our code so far

package item;

import net.minecraft.entity.Entity;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.world.World;

public class UraniumIngotEffect extends Item{


@Override
public void onUpdate(ItemStack stack, World worldIn, Entity entityIn, int itemSlot, boolean isSelected) {
	if(entityIn instanceof EntityLivingBase)
	super.onUpdate(stack, worldIn, entityIn, itemSlot, isSelected);
}

}

If we cast

 

Entity entityIn = EntityLivingBase.this.isPotionActive(MobEffects.WITHER);

We get an error saying

No enclosing instance of the type EntityLivingBase is accessible in scope

Im sorry if this is basic but i rlly need to hurry to get this done but the set time for my friend

Dracos that was a stuped question srry . But forreal this is as far as i could get i tryed mutiple ways with the "Else" Statment and here what i ended up with

 

package item;

import net.minecraft.entity.Entity;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.init.MobEffects;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.potion.PotionEffect;
import net.minecraft.world.World;

public class UraniumIngotEffect extends Item{


@Override
public void onUpdate(ItemStack stack, World worldIn, Entity entityIn, int itemSlot, boolean isSelected) {
	if(entityIn instanceof EntityLivingBase){
		 Object EntityLivingBase = ((EntityLivingBase) entityIn).isPotionActive(MobEffects.WITHER);

	}else{
	    PotionEffect PotionEffect =  ((EntityLivingBase) entityIn).addPotionEffect(MobEffects.WITHER);
	}


	super.onUpdate(stack, worldIn, entityIn, itemSlot, isSelected);
}

}

http://tutorials.jenkov.com/java/index.html

 

Read it all, follow along, do examples. When you get to the end, you will probably understand a lot more.

Ive already read it and followed along. You told me w while ago to do this

What you are doing there makes absolutely no sense and shows us that you dont know much about java...

This forum isnt a java learning forum.

Learn atleast java basics to try modding.

You can't just add an onUpdate into the item. You have to have a code that runs when, and only when the item is equipped.