Jump to content

[1.10.2] Add a potion effect to an armor


themistik
 Share

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
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.

 Share



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • It looks like the port you are trying to host the server from is already in use (possibly by another server or another instance of Minecraft). You can change the port in server.properties, just make sure you forward that port on your router.
    • Can you send a crash report? (<game_directory>/crash-reports) ¿Puedes enviar un informe de fallas? (<directorio_del_juego>/crash-reports)
    • Never mind, I think finally found the list of Forge tags in the extended libraries of my workspace. I should be all set with that now.
    • This mod does it, so you could try looking at their code to get an idea of how they do it. Or, just use their mod. https://www.curseforge.com/minecraft/mc-mods/shutup-experimental-settings
    • And here is the cmd panel of the server         2023-01-29 20:12:16,231 main WARN Advanced terminal features are not available in this environment [20:12:16] [main/INFO] [cp.mo.mo.Launcher/MODLAUNCHER]: ModLauncher running: args [--launchTarget, forgeserver, --fml.forgeVersion, 44.1.8, --fml.mcVersion, 1.19.3, --fml.forgeGroup, net.minecraftforge, --fml.mcpVersion, 20221207.122022, nogui] [20:12:16] [main/INFO] [cp.mo.mo.Launcher/MODLAUNCHER]: ModLauncher 10.0.8+10.0.8+main.0ef7e830 starting: java version 19.0.2 by Oracle Corporation; OS Windows 10 arch amd64 version 10.0 [20:12:16] [main/INFO] [mixin/]: SpongePowered MIXIN Subsystem Version=0.8.5 Source=union:/C:/Users/Muharrem/Desktop/FTB%20Servers/%7Bcurrent%20directory%202%5D/libraries/org/spongepowered/mixin/0.8.5/mixin-0.8.5.jar%2364!/ Service=ModLauncher Env=SERVER [20:12:16] [main/WARN] [ne.mi.fm.lo.mo.ModFileParser/LOADING]: Mod file C:\Users\Muharrem\Desktop\FTB Servers\{current directory 2]\libraries\net\minecraftforge\fmlcore\1.19.3-44.1.8\fmlcore-1.19.3-44.1.8.jar is missing mods.toml file [20:12:16] [main/WARN] [ne.mi.fm.lo.mo.ModFileParser/LOADING]: Mod file C:\Users\Muharrem\Desktop\FTB Servers\{current directory 2]\libraries\net\minecraftforge\javafmllanguage\1.19.3-44.1.8\javafmllanguage-1.19.3-44.1.8.jar is missing mods.toml file [20:12:16] [main/WARN] [ne.mi.fm.lo.mo.ModFileParser/LOADING]: Mod file C:\Users\Muharrem\Desktop\FTB Servers\{current directory 2]\libraries\net\minecraftforge\lowcodelanguage\1.19.3-44.1.8\lowcodelanguage-1.19.3-44.1.8.jar is missing mods.toml file [20:12:16] [main/WARN] [ne.mi.fm.lo.mo.ModFileParser/LOADING]: Mod file C:\Users\Muharrem\Desktop\FTB Servers\{current directory 2]\libraries\net\minecraftforge\mclanguage\1.19.3-44.1.8\mclanguage-1.19.3-44.1.8.jar is missing mods.toml file [20:12:17] [main/INFO] [ne.mi.fm.lo.mo.JarInJarDependencyLocator/]: No dependencies to load found. Skipping! [20:12:18] [main/INFO] [cp.mo.mo.LaunchServiceHandler/MODLAUNCHER]: Launching target 'forgeserver' with arguments [nogui] [20:12:23] [main/INFO] [minecraft/DataFixers]: Building unoptimized datafixer [20:12:25] [modloading-worker-0/INFO] [ne.mi.co.ForgeMod/FORGEMOD]: Forge mod loading, version 44.1.8, for MC 1.19.3 with MCP 20221207.122022 [20:12:25] [modloading-worker-0/INFO] [ne.mi.co.MinecraftForge/FORGE]: MinecraftForge v44.1.8 Initialized [20:12:25] [Forge Version Check/INFO] [ne.mi.fm.VersionChecker/]: [forge] Starting version check at https://files.minecraftforge.net/net/minecraftforge/forge/promotions_slim.json [20:12:26] [Forge Version Check/INFO] [ne.mi.fm.VersionChecker/]: [forge] Found status: AHEAD Current: 44.1.8 Target: null [20:12:26] [main/INFO] [mojang/YggdrasilAuthenticationService]: Environment: authHost='https://authserver.mojang.com', accountsHost='https://api.mojang.com', sessionHost='https://sessionserver.mojang.com', servicesHost='https://api.minecraftservices.com', name='PROD' [20:12:27] [main/WARN] [minecraft/VanillaPackResourcesBuilder]: Assets URL 'union:/C:/Users/Muharrem/Desktop/FTB%20Servers/%7Bcurrent%20directory%202%5D/libraries/net/minecraft/server/1.19.3-20221207.122022/server-1.19.3-20221207.122022-srg.jar%23101!/assets/.mcassetsroot' uses unexpected schema [20:12:27] [main/WARN] [minecraft/VanillaPackResourcesBuilder]: Assets URL 'union:/C:/Users/Muharrem/Desktop/FTB%20Servers/%7Bcurrent%20directory%202%5D/libraries/net/minecraft/server/1.19.3-20221207.122022/server-1.19.3-20221207.122022-srg.jar%23101!/data/.mcassetsroot' uses unexpected schema [20:12:29] [main/INFO] [minecraft/RecipeManager]: Loaded 7 recipes [20:12:29] [main/INFO] [minecraft/AdvancementList]: Loaded 1179 advancements [20:12:30] [Server thread/INFO] [minecraft/DedicatedServer]: Starting minecraft server version 1.19.3 [20:12:30] [Server thread/INFO] [minecraft/DedicatedServer]: Loading properties [20:12:30] [Server thread/INFO] [minecraft/DedicatedServer]: Default game type: SURVIVAL [20:12:30] [Server thread/INFO] [minecraft/MinecraftServer]: Generating keypair [20:12:30] [Server thread/INFO] [minecraft/DedicatedServer]: Starting Minecraft server on 78.174.226.175:25565 [20:12:30] [Server thread/INFO] [minecraft/ServerConnectionListener]: Using default channel type [20:12:30] [Server thread/WARN] [minecraft/DedicatedServer]: **** FAILED TO BIND TO PORT! [20:12:30] [Server thread/WARN] [minecraft/DedicatedServer]: The exception was: java.net.BindException: Cannot assign requested address: bind [20:12:30] [Server thread/WARN] [minecraft/DedicatedServer]: Perhaps a server is already running on that port? [20:12:30] [Server thread/ERROR] [minecraft/MinecraftServer]: Encountered an unexpected exception java.lang.IllegalStateException: Failed to initialize server         at net.minecraft.server.MinecraftServer.m_130011_(MinecraftServer.java:637) ~[server-1.19.3-20221207.122022-srg.jar%23101!/:?] {re:classloading,pl:accesstransformer:B}         at net.minecraft.server.MinecraftServer.m_206580_(MinecraftServer.java:251) ~[server-1.19.3-20221207.122022-srg.jar%23101!/:?] {re:classloading,pl:accesstransformer:B}         at java.lang.Thread.run(Thread.java:1589) ~[?:?] {} [20:12:30] [Server thread/FATAL] [ne.mi.co.ForgeMod/]: Preparing crash report with UUID 9ad7a420-98e0-4be3-be96-3783b966a471 [20:12:30] [Server thread/ERROR] [minecraft/MinecraftServer]: This crash report has been saved to: C:\Users\Muharrem\Desktop\FTB Servers\{current directory 2]\.\crash-reports\crash-2023-01-29_20.12.30-server.txt [20:12:30] [Server thread/INFO] [minecraft/MinecraftServer]: Stopping server [20:12:30] [Server thread/INFO] [minecraft/MinecraftServer]: Saving worlds [20:12:30] [Server thread/ERROR] [minecraft/MinecraftServer]: Exception stopping the server java.lang.NullPointerException: Cannot invoke "net.minecraft.server.level.ServerLevel.m_6857_()" because "serverlevel2" is null         at net.minecraft.server.MinecraftServer.m_129885_(MinecraftServer.java:513) ~[server-1.19.3-20221207.122022-srg.jar%23101!/:?] {re:classloading,pl:accesstransformer:B}         at net.minecraft.server.MinecraftServer.m_7041_(MinecraftServer.java:586) ~[server-1.19.3-20221207.122022-srg.jar%23101!/:?] {re:classloading,pl:accesstransformer:B}         at net.minecraft.server.dedicated.DedicatedServer.m_7041_(DedicatedServer.java:498) ~[server-1.19.3-20221207.122022-srg.jar%23101!/:?] {re:classloading,pl:accesstransformer:B}         at net.minecraft.server.MinecraftServer.m_130011_(MinecraftServer.java:692) ~[server-1.19.3-20221207.122022-srg.jar%23101!/:?] {re:classloading,pl:accesstransformer:B}         at net.minecraft.server.MinecraftServer.m_206580_(MinecraftServer.java:251) ~[server-1.19.3-20221207.122022-srg.jar%23101!/:?] {re:classloading,pl:accesstransformer:B}         at java.lang.Thread.run(Thread.java:1589) ~[?:?] {} Press any key to continue . . .
  • Topics

×
×
  • Create New...

Important Information

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