Posted June 15, 201312 yr How can I get the armor item in the armor slot? Whats wrong? public void onEntityCollidedWithBlock(World par1World, int par2, int par3, int par4, Entity par5Entity) { if(par5Entity instanceof EntityPlayer) { if(((EntityLiving) par5Entity).getCurrentItemOrArmor(1) != new ItemStack(apocalypse.Respirator)) { Debug.log("Entering gas without respirator!"); par5Entity.attackEntityFrom(DamageSource.drown, damage); } } }
June 15, 201312 yr From what I can gather... that piece of code there only makes it so that if you're not wearing a respirator then you get drown damage Sending your full armor file would be more helpful
June 15, 201312 yr How can I get the armor item in the armor slot? Whats wrong? public void onEntityCollidedWithBlock(World par1World, int par2, int par3, int par4, Entity par5Entity) { if(par5Entity instanceof EntityPlayer) { if(((EntityLiving) par5Entity).getCurrentItemOrArmor(1) != new ItemStack(apocalypse.Respirator)) { Debug.log("Entering gas without respirator!"); par5Entity.attackEntityFrom(DamageSource.drown, damage); } } } ((EntityLiving) par5Entity).getCurrentItemOrArmor(1) != new ItemStack(apocalypse.Respirator) This will always be true, because you compare two different instances, although the content of them may be the same. To do it right, use item1.isItemEqual(item2) and for any other object (like strings) use object1.equals(object2) Don't ask for support per PM! They'll get ignored! | If a post helped you, click the "Thank You" button at the top right corner of said post! | mah twitter This thread makes me sad because people just post copy-paste-ready code when it's obvious that the OP has little to no programming experience. This is not how learning works.
June 15, 201312 yr item1.isItemEqual(item2) crashes client(( You need to check before if both are not null, like: item1 != null Don't ask for support per PM! They'll get ignored! | If a post helped you, click the "Thank You" button at the top right corner of said post! | mah twitter This thread makes me sad because people just post copy-paste-ready code when it's obvious that the OP has little to no programming experience. This is not how learning works.
June 15, 201312 yr Author item1.isItemEqual(item2) crashes client(( You need to check before if both are not null, like: item1 != null Thank you! It's working.
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.