How to run onItemRightClick one time?

[progamergalil]

I' ve a problem with onItemRightClick function...

It' s a loop, and when I right click the text is repeated several times...

How can I print only one time every time that I right click?

Here is the code:

    @Override
    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player) {
        
        player.addChatMessage(new ChatComponentText("Hello world"));
        
        return stack;
    }

[progamergalil]

How this method works?

[progamergalil]

How can I call another function and stop onItemRightClick function?

If I find a way to stop onItemRightClick function I 've find a solution...

Is there any way to do this?

(Can you make an easy example?)

Edited September 5, 2017 by progamergalil

[progamergalil]

I mean that the function must write "hello world" only 1 time evry right click...

I've think that I could make a function that "lock" onItemRightClick function if right click is pressed.

 

int count = 1;

    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player) {

        if (count == 1) {

             player.addChatMessage(new ChatComponentText("Right clicked"));

        }

     count = 0;

    }

 

public void control() {

      if ("here test if mouse is NOT right clicked") {   // <-- LOOK DOWN *

             count = 1;

       }

}

 

 

*This is the "if" that I don't know. Is possible to test in "if" if mouse is not right clicked?

 

With this way the onItemRightClick function is executed ONE time (because the next time COUNT is equal to 0).

Edited September 5, 2017 by progamergalil

[That_Martin_Guy]

Have you tried checking !world.isRemote before doing any action?

[progamergalil]

Can I have an example?

I haven't found any example of this function...

[progamergalil]

Sorry, I mean once per side / per hand...

Can you give me an easy example?

[Draco18s]

11 minutes ago, progamergalil said:

Sorry, I mean once per side / per hand...

Can you give me an easy example?

The function is called four times. Once on each side for each hand. Two sides, two hands, four calls.

 

You need to check what side and which hand and decide based on that information what to do. You want to pick one combination and only send the chat message on that combination.

[jabelar]

What they mean is that the function isn't really on a loop, rather it is being called on both the client and the server and for each of those called passing in ItemStack in each of the two hands.

 

So you don't stop it from running, instead when it runs you need to check which side is running and which whether the itemstack is passed in a hand that you care about and then act accordingly.

 

[progamergalil]

I want to send message "hello" every time I right click with my custom sword...

How can I make in this case?

[progamergalil]

2 minutes ago, jabelar said:

What they mean is that the function isn't really on a loop, rather it is being called on both the client and the server and for each of those called passing in ItemStack in each of the two hands.

 

So you don't stop it from running, instead when it runs you need to check which side is running and which whether the itemstack is passed in a hand that you care about and then act accordingly.

 

Yes, but I don't know how to make it

[Draco18s]

if(the active side == side I care about && active hand == hand I care about) {
  //do something
}

 

[progamergalil]

6 minutes ago, Draco18s said:

if(the active side == side I care about && active hand == hand I care about) {
  //do something
}

 

Ok, but what do I write in "the active side", "side I care about && active hand", "hand I care about".

Sorry, I 'm very ignorant in this program.

[Draco18s]

Well...one of the parameters passed the function is a World object. World objects can either be client side or server side and you can tell by querying World#isRemote.

Edited September 5, 2017 by Draco18s

[progamergalil]

ok...and the other parameter?

[Jay Avery]

If you use an up-to-date version of Forge, there will be an EnumHand parameter passed to the method. 

[progamergalil]

Unluckly there isn' t EnumHand

[progamergalil]

How can't be a command similar to EnumHand in minecraft 1.8???

[progamergalil]

Then why onItemRightClick works as a loop?