Jump to content

[solved][1.7.10] Destroying an armor


scoob27

Recommended Posts

Hi everyone,

 

I'm trying to make an armor (chest plate) which destroys itself when the player wears it and goes over a certain distance from the ground.

 

To do so i create a fonction which calculates the distance between the player and the ground (i don't think there is a in-game method() that already exists. Am I right ?)

 

Then I tried to make the chest plate to destroy itself and that is where i got some difficulties. First I look for a method which destroys the armor but didn't find one. Then i tried to use the setDamage method from ItemStack :

 

 


@Override
public void onArmorTick(World world, EntityPlayer player, ItemStack itemStack) 
{
	int distanceToGround = 0;

	if(player.inventory.armorItemInSlot(2) != null)
	{
		if (player.inventory.armorItemInSlot(2).getItem() == ModItems.icarusWings)
		{
			...

			if( distanceToGround > 20)
			{
			        player.inventory.armorItemInSlot(2).setItemDamage(this.getMaxDamage());
			}
		}
	}
}

 

 

So when I tried and got too high the armor got fully damaged (its bar is completely empty) but the armor was still in its slot. I also tried to use this.getMaxDamage()+1 but the result was the same.

 

Then I tried the hard way and replace

 player.inventory.armorItemInSlot(2).setItemDamage(this.getMaxDamage()); 

with

player.inventory.armorInventory[2] = null;

 

It actually worked and the armor disappeared when I went over the limit but when I tried to disconnect and reconnect the armor was back in its slot...

 

I thought that the armor = null was only working in the client side and tried to send a packet to the server (using ByteBufUtils.writeItemStack() with the new value of the slot but I realized it was stupid because the server would only receive "null" and won't know what it refers to. So unless i can send the player with it it won't work. I saw in the ByteBufUtils that there is a method to convert NBTT tag. So if I use a tag to store the player variable and send this tag in that packet with the new value do you think it would work ?

 

But with all that I feel like using a sledgehammer to crack a nut... Is there any ways to do it more easily ?

 

Thank you :)

Link to comment
Share on other sites

this works for me,

player.inventory.armorInventory[2] = (ItemStack) null;
player.inventory.armorInventory[2].stackSize = 0;

The second line will cause a NPE.

Don't PM me with questions. They will be ignored! Make a thread on the appropriate board for support.

 

1.12 -> 1.13 primer by williewillus.

 

1.7.10 and older versions of Minecraft are no longer supported due to it's age! Update to the latest version for support.

 

http://www.howoldisminecraft1710.today/

Link to comment
Share on other sites

this works for me,

player.inventory.armorInventory[2] = (ItemStack) null;
player.inventory.armorInventory[2].stackSize = 0;

The second line will cause a NPE.

 

I haven't try it but i also think it would cause a NPE since null is not an itemStack even if you cast it.

 

Well since it worked for crackedEgg I will try it but I am not very hopeful :) 

 

EDIT : As expected i got a nullPointerExeption... I guess you were lucky somehow

Link to comment
Share on other sites

larsgerrits is correct. This code causes a silent (at least for me) NPE. It does not crash my game. Sorry for the confusion.

Well, back to the drawing board...

 

EDIT:

THIS code does work without NPE.

player.inventory.armorInventory[2] = (ItemStack) null;

forget setting stacksize to zero. If you find a better way to do this please post back.

Link to comment
Share on other sites

larsgerrits is correct. This code causes a silent (at least for me) NPE. It does not crash my game. Sorry for the confusion.

Well, back to the drawing board...

 

EDIT:

THIS code does work without NPE.

player.inventory.armorInventory[2] = (ItemStack) null;

forget setting stacksize to zero. If you find a better way to do this please post back.

That's the simplest way to do that.

Don't PM me with questions. They will be ignored! Make a thread on the appropriate board for support.

 

1.12 -> 1.13 primer by williewillus.

 

1.7.10 and older versions of Minecraft are no longer supported due to it's age! Update to the latest version for support.

 

http://www.howoldisminecraft1710.today/

Link to comment
Share on other sites

Yes and it works but it is only temporary since when I quit the game and reconnect the armor is still in the slot as if it was never destroyed.

 

In fact it looks like it's a little bit more complex. Here it's what happened :

1) My inventory is completely empty. I take one piece of the armor in creative and go back to survival

2) I right-click the armor to equip it so it gets in the armor slot and disappears from the hotter

3) I go over the authorized limit and the armor breaks

4) I fall on the ground but don't take any fall damage (which is strange)

5) I log out and reconnect -> The armor is both in the armor slot and the hot bar as if i've never used it...

 

Do you have an idea of what can cause that ?

Link to comment
Share on other sites

I am almost 100% sure your code (armour - break) is run on CLIENT. Anything that edits anything (entities, inventory, itemStacks) should be done always on SERVER. You can use world.isRemote to check sides.

 

Also - use Syso to check if your code is even called.

1.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

Thanks Ernio, that was the problem. Everything is working perfectly now.

 

Just a question though : According to your suggestion i add an if(!world.isRemote) statement around my armor = null so that will be executed on the server side but before i did that there was nothing to tell where the code should be executed. In that case shouldn't it been executed on both side and so worked like i wanted ?

 

Anyway thanks :)

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.



×
×
  • Create New...

Important Information

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