Jump to content

[1.10.2] Add a potion effect to an armor


themistik

Recommended Posts

Hello.

 

A pretty simple question for you maybe, but for me, I just feel PAIN. I've search for hours, and no one technique works...

Like the title of the topic, all I want is to add a Potion effect to an armor.

 

public class ItemModArmor extends ItemArmor{
	
	private PotionEffect[] effects;
	
	public ItemModArmor(ArmorMaterial materialIn, int renderIndexIn, EntityEquipmentSlot equipmentSlotIn, String unlocalizedName) {
		super(materialIn, renderIndexIn, equipmentSlotIn);
		this.setUnlocalizedName(unlocalizedName);
		this.setRegistryName(new ResourceLocation(References.MODID, unlocalizedName));
	}
	
	public ItemModArmor(ArmorMaterial materialIn, int renderIndexIn, EntityEquipmentSlot equipmentSlotIn, String unlocalizedName, PotionEffect...potionEffects) {
		super(materialIn, renderIndexIn, equipmentSlotIn);
		this.setUnlocalizedName(unlocalizedName);
		this.effects = potionEffects;
		this.setRegistryName(new ResourceLocation(References.MODID, unlocalizedName));
	}
	
	protected void onArmorTick(ItemStack stack, World worldIn, EntityPlayer player)
	{
	    if (stack.getItem() == ModArmor.canardChestplate) {
			for(PotionEffect effect : effects) {
				player.addPotionEffect(new PotionEffect(effect));
			}
	    }
	}

 

this is my main class for the armor. Easy to use.

 

First, I wanted to stay in this "easy to use" way. So I was think if the armor works like the food ; You added a potion effect to your constructor, like this :

 

public class ItemModFood extends ItemFood{

	private PotionEffect[] effects;
	
	public ItemModFood(String unlocalizedName, int amount, boolean isWolfFood, PotionEffect...potionEffects) {
		super(amount, isWolfFood);
		this.setUnlocalizedName(unlocalizedName);
		this.setRegistryName(new ResourceLocation(References.MODID, unlocalizedName));
		this.effects = potionEffects;
		// TODO Auto-generated constructor stub
	}
	
	public ItemModFood(String unlocalizedName, int amount, float saturation, boolean isWolfFood, PotionEffect...potionEffects) {
		super(amount, saturation, isWolfFood);
		this.setUnlocalizedName(unlocalizedName);
		this.setRegistryName(new ResourceLocation(References.MODID, unlocalizedName));
		this.effects = potionEffects;
		// TODO Auto-generated constructor stub
	}


	protected void onFoodEaten(ItemStack stack, World worldIn, EntityPlayer player)
	{
		for(PotionEffect effect : effects) {
			player.addPotionEffect(new PotionEffect(effect));
		}
	}

 

And I define the potion effects in my ModFood class, the one who is initalized.

 

	public static void init() {
		weed = new ItemModFood("weed", 1, 1, false, new PotionEffect(Potion.getPotionById(9), 500, 0), new PotionEffect(Potion.getPotionById(1), 500, 0));
	}
	
	public static void register() {
		registerItem(weed);
	}
	
	public static void registerRenders() {
		registerRender(weed);
	}
	
	public static void registerItem(Item item) {
		item.setCreativeTab(MainClass.Tab);
		GameRegistry.register(item);
		Utools.getLogger().info("Register item for" + item.getUnlocalizedName().substring(5));
	}
	
	public static void registerRender(Item item) {
		ModelLoader.setCustomModelResourceLocation(item, 0, new ModelResourceLocation(new ResourceLocation(References.MODID, item.getUnlocalizedName().substring(5)), "inventory"));
		Utools.getLogger().info("Registered render for " + item.getUnlocalizedName().substring(5));
	}
}

 

(Sorry for the weed'joke, It was a request from a friend of mine, and I've seen in this item a good way to learn how to add potion effect to food stuff)

 

So, this works. There is a way to make this for an armor ? Because I've tried, but it dosen't work this way, obviously. So I've tried many things from other tutorials/videos, but I dosen't work.

 

After a decade, I've abandonned this "easy to use" method, and try an "harder" method. BUT ! This one dosen't work too !

 

	protected void onArmorTick(ItemStack stack, World worldIn, EntityPlayer player)
	{
	    if (stack.getItem() == ModArmor.canardChestplate) {
			for(PotionEffect effect : effects) {
				player.addPotionEffect(new PotionEffect(effect));
			}
	    }
	}

 

This is in my main class for armor.

 

I'm so lost I'm ready to throw anything up...

 

All I want, is to add a potion effect to an armor. If this is possibile, making the potion effect easy to add like the food method.

 

Thanks for the help.

 

Ps : Sorry if you don't understand all of this, English is not my native langage. Don't hestitate to ask me something.

Link to comment
Share on other sites

The constructor you're using for your new PotionEffect() is one which assigns a duration of 0 to the effect. Most effects won't actually be visible with a zero duration. You should try using one of the other constructors which takes in a duration and using a non-zero duration with it. If that still doesn't work, try adding a System.out.println() debug output inside that FOR loop to see if it's even running, and what the effect is if so; that should give you some more insight into the problem.

Whatever Minecraft needs, it is most likely not yet another tool tier.

Link to comment
Share on other sites

Are you sure ? Beacause I have this :

 

595760d983b4c_Sanstiqzdqzdqzdqzdtre.png.040c227680621912f96a2a091fb31a5f.png

 

I'll give a try for the PrintIn.

 

EDIT :

It seems its my For who dosen't work.

 

I have tried that :

 

Captqzdqzdure.PNG.ff86f29e2b0d8ef915a935a9f70e1c47.PNG

 

The console dosent show me anything about it.

Edited by themistik
added screenshot
Link to comment
Share on other sites

2 minutes ago, IceMetalPunk said:

The constructor you're using for your new PotionEffect() is one which assigns a duration of 0 to the effect.

No, he isn't. The constructor for PotionEffect takes in a Potion, duration and then amplifier so OP is passing 500 as the duration.

 

Getting potions by ID though... 

19 minutes ago, themistik said:

Potion.getPotionById(9)

Should not be used. I personaly have no idea what is potion with an ID of 9. And what if it changes later? Potions should be referenced from the MobEffects class instead of their IDs.

 

22 minutes ago, themistik said:

this.setRegistryName(new ResourceLocation(References.MODID, unlocalizedName));

Unlocalized names have nothing to do with registry names.

 

26 minutes ago, themistik said:

onArmorTick(ItemStack stack, World worldIn, EntityPlayer player)

This signature doesn't look right to me. I think that the method params have a different order. Add an Override annotation to the method. If eclipse reports an error then I am correct. You should not manually write overrides, let your IDE do that for you.

  • Like 1
Link to comment
Share on other sites

1 minute ago, V0idWa1k3r said:

No, he isn't. The constructor for PotionEffect takes in a Potion, duration and then amplifier so OP is passing 500 as the duration.

 

Getting potions by ID though... 

Should not be used. I personaly have no idea what is potion with an ID of 9. And what if it changes later? Potions should be referenced from the MobEffects class instead of their IDs.

 

Unlocalized names have nothing to do with registry names.

 

This signature doesn't look right to me. I think that the method params have a different order. Add an Override annotation to the method. If eclipse reports an error then I am correct. You should not manually write overrides, let your IDE do that for you.

The tutorial i'm using uses PotionEffect with an Id. I don't want to screw up things so I don't try anything this way

Same thing for " this.setRegistryName(new ResourceLocation(References.MODID, unlocalizedName)); "

 

For the signature, yes your are correct.

Link to comment
Share on other sites

1 minute ago, V0idWa1k3r said:

No, he isn't. The constructor for PotionEffect takes in a Potion, duration and then amplifier so OP is passing 500 as the duration.

*Facepalm* Whoops, I was looking at an entirely different line of code when I wrote that... I probably shouldn't try to help people at 4AM without sleep. I guess I'm taking the "Always Be Coding" motto too far...

Sorry, themistik! But, yes, as voidwalker pointed out, you've got your order of parameters mixed up; the ItemStack comes last, not first.

  • Like 1

Whatever Minecraft needs, it is most likely not yet another tool tier.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • They were already updated, and just to double check I even did a cleanup and fresh update from that same page. I'm quite sure drivers are not the problem here. 
    • i tried downloading the drivers but it says no AMD graphics hardware has been detected    
    • Update your AMD/ATI drivers - get the drivers from their website - do not update via system  
    • As the title says i keep on crashing on forge 1.20.1 even without any mods downloaded, i have the latest drivers (nvidia) and vanilla minecraft works perfectly fine for me logs: https://pastebin.com/5UR01yG9
    • Hello everyone, I'm making this post to seek help for my modded block, It's a special block called FrozenBlock supposed to take the place of an old block, then after a set amount of ticks, it's supposed to revert its Block State, Entity, data... to the old block like this :  The problem I have is that the system breaks when handling multi blocks (I tried some fix but none of them worked) :  The bug I have identified is that the function "setOldBlockFields" in the item's "setFrozenBlock" function gets called once for the 1st block of multiblock getting frozen (as it should), but gets called a second time BEFORE creating the first FrozenBlock with the data of the 1st block, hence giving the same data to the two FrozenBlock :   Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=head] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@73681674 BlockEntityData : id:"minecraft:bed",x:3,y:-60,z:-6} Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=3, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=2, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} here is the code inside my custom "freeze" item :    @Override     public @NotNull InteractionResult useOn(@NotNull UseOnContext pContext) {         if (!pContext.getLevel().isClientSide() && pContext.getHand() == InteractionHand.MAIN_HAND) {             BlockPos blockPos = pContext.getClickedPos();             BlockPos secondBlockPos = getMultiblockPos(blockPos, pContext.getLevel().getBlockState(blockPos));             if (secondBlockPos != null) {                 createFrozenBlock(pContext, secondBlockPos);             }             createFrozenBlock(pContext, blockPos);             return InteractionResult.SUCCESS;         }         return super.useOn(pContext);     }     public static void createFrozenBlock(UseOnContext pContext, BlockPos blockPos) {         BlockState oldState = pContext.getLevel().getBlockState(blockPos);         BlockEntity oldBlockEntity = oldState.hasBlockEntity() ? pContext.getLevel().getBlockEntity(blockPos) : null;         CompoundTag oldBlockEntityData = oldState.hasBlockEntity() ? oldBlockEntity.serializeNBT() : null;         if (oldBlockEntity != null) {             pContext.getLevel().removeBlockEntity(blockPos);         }         BlockState FrozenBlock = setFrozenBlock(oldState, oldBlockEntity, oldBlockEntityData);         pContext.getLevel().setBlockAndUpdate(blockPos, FrozenBlock);     }     public static BlockState setFrozenBlock(BlockState blockState, @Nullable BlockEntity blockEntity, @Nullable CompoundTag blockEntityData) {         BlockState FrozenBlock = BlockRegister.FROZEN_BLOCK.get().defaultBlockState();         ((FrozenBlock) FrozenBlock.getBlock()).setOldBlockFields(blockState, blockEntity, blockEntityData);         return FrozenBlock;     }  
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.