Can't get my item to have durability

[Triphion]

I do not seem to understand how im supposed to give my item i've done durability. I've tried the setmaxdamage method aswell as stack.damageitem method, but when i do the stack.damageitem method it just gives me error saying that it isn't existing or something. So then i try to add Itemstack Stack in the upper session where the Player PlayerIn is an all that, but that doesn't work either. I just can't figure out what im supposed to have to succesfully get this to work. 

 

My item class - https://pastebin.com/1b3jg52a

Edited August 4, 2017 by Triphion

[Ugdhar]

public ActionResult<ItemStack> onItemRightClick(Itemstack Stack, World worldIn, EntityPlayer playerIn, EnumHand handIn) {

 

I believe Itemstack should be ItemStack. Not sure if that's specifically the problem, but it certainly can't be helping

[Triphion]

18 minutes ago, diesieben07 said:

This makes no sense, please clarify.

Well, i have been looking all over the internet and on every one of those durability posts there is ItemStack Stack in the onitemrightclick method, but when i did the onitemrightclick method, it wasn't mentioned in the method at all. 

Edited August 4, 2017 by Triphion

[Triphion]

23 minutes ago, diesieben07 said:

You tell the game that the resulting ItemStack is a new, default ItemStack of your item. What did you expect?

I was following a tutorial on item with custom properties, didn't really look into what it actually did. This was the tutorial btw - http://couchdoescode.blogspot.se/2017/02/bonus-material-raytrace-and.html

[Triphion]

But what should i do? When i add ItemStack Stack successfully it says that i need to remove @Override but then it's not working since it needs to be overriding. So i have no idea what i should add or remove. I tried removing that new itemstack after both "SUCCESS" and "FAIL", but it says that it has to have some sort of statement after which i have no idea what sort of statement it should be. 

[Triphion]

And if i want to add durability it should be what exactly? The item? Or null? Because i want to add a durability of like 1000 on this staff and i do not know when this applies. 

Edited August 4, 2017 by Triphion

[Triphion]

22 minutes ago, diesieben07 said:

If you want to add durability, you need to add durability to the stack held by the player and then pass that to the ActionResult.

And the method for that is setmaxdamage? And also, how would i pass it on to the actionresult and register it so that on item rightclick it removes 1 from the durability?

[Triphion]

5 minutes ago, diesieben07 said:

If you do not know how to pass something to a constructor you need to go and learn basic Java.

I do not see that necessary since this seems to be a very simple matter since its only one thing i want to do and i feel like you already know how to do this. It would be much easier if you could just tell me a method which i can implement into the code such as "Stack.damageitem" or something. Or what needs to be replaced with what method. 

5 minutes ago, diesieben07 said:

What do you want to register?

How much damage it removes on itemrightclick and also that it actually removes durability. 

Edited August 4, 2017 by Triphion

[jeffryfisher]

@Override
    public ActionResult<ItemStack> onItemRightClick(Itemstack Stack, World worldIn, EntityPlayer playerIn, EnumHand handIn)

This doesn't look right. Do you get a complaint about the Override? If so, then you need to revise the method profile until the complaint goes away. I know, Eclipse will always suggest that you cut the Override, but that's because Eclipse is too stupid to know if it really belongs there. If it does, then you need to keep it there and make it work, that's what the annotation is for -- to irritate you until you fix the broken method profile.

 

I suggest copying the profile of the method you're overriding.

 

return new ActionResult(EnumActionResult.SUCCESS, new ItemStack(this));

Why are you returning a new stack? What's wrong with the one you were given (and to which you added damage)? You should set some breakpoints around your code and then step through it and into method calls and out to the surrounding vanilla code to see how your result is used. Only then will you see what's happening and why.

Edited August 4, 2017 by jeffryfisher