themistik Posted July 1, 2017 Share Posted July 1, 2017 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. Quote Link to comment Share on other sites More sharing options...
IceMetalPunk Posted July 1, 2017 Share Posted July 1, 2017 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. Quote Whatever Minecraft needs, it is most likely not yet another tool tier. Link to comment Share on other sites More sharing options...
themistik Posted July 1, 2017 Author Share Posted July 1, 2017 (edited) Are you sure ? Beacause I have this : I'll give a try for the PrintIn. EDIT : It seems its my For who dosen't work. I have tried that : The console dosent show me anything about it. Edited July 1, 2017 by themistik added screenshot Quote Link to comment Share on other sites More sharing options...
V0idWa1k3r Posted July 1, 2017 Share Posted July 1, 2017 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. 1 Quote Link to comment Share on other sites More sharing options...
themistik Posted July 1, 2017 Author Share Posted July 1, 2017 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. Quote Link to comment Share on other sites More sharing options...
IceMetalPunk Posted July 1, 2017 Share Posted July 1, 2017 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. 1 Quote Whatever Minecraft needs, it is most likely not yet another tool tier. Link to comment Share on other sites More sharing options...
themistik Posted July 1, 2017 Author Share Posted July 1, 2017 (edited) Thanks for all of you. Now its works. The problem was the ItemStack. Edited July 1, 2017 by themistik Quote Link to comment Share on other sites More sharing options...
Recommended Posts
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.