Checking to see if the player has right-clicked a catcus

I'm building a plugin, however I'll need the line for the right-clicking and making sure it's a catcus. I already got the action set, but I'll need to make sure it checked to see if it's catcus.

How can I achieve this?

 

In your onPlayerInteractEvent method, check to see if the interaction was them right clicking a block using I think (could be wrong) event.getAction() and then check the block Id and compare it to that of cactus using event.getBlock().getTypeId()

 

@Taco
You're not wrong. It's event.getAction()

 

I'm running into problems.

I had defined the action, then the block, now I setted the ID, but I need to set the data type (15) so I can make this functioning properly.

 

I do not understand your new problem.
What do you want to do? Do you want to change the cactus to an Iron Ore block? (15)

 

That should simply be event.getBlock().setData(int data);

 

Actually it's event.getClickedBlock()
I don't know why it's not just event.getBlock()

 

My event is this:

Code:java

 
	public void onPlayerInteract(PlayerInteractEvent event) {
		if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {
			if(event.getClickedBlock() == 81 && event.getItem().getTypeId() == 351) {
			}
		}
	}
}
 

I don't know how I can get the items and find the data. Maybe I should do something else.

 

event.getClickedBlock() == 81 - This compares a block to an int. Not the id of said block. A block never equals an int.

Try this:

Code:java

 
	public void onPlayerInteract(PlayerInteractEvent event) {
		if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {
			if(event.getClickedBlock().getTypeId() == 81 && event.getItem() != null && event.getItem().getTypeId() == 351 && event.getItem().getData == 15) { //I'm assuming you wanted to check the data as well.
			}
		}
	}
}
 

Just to avoid further confusion, event.getItem() returns the item in hand. You need a null check in case they are holding nothing.

 

Generates an error. Please check it.

 

@Jaker232 And what error?

 

I need the error to help you with that.

 

Code:

if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {

This line isn't really needed is it? D: Its not like the right clicked block will ever be anything if nothing was right clicked D:

 

it's needed otherwise it will call on any click.

 

D: But if u left click it its the blockdamage event :/

I fail to see any other ''clicks'' xD

 

onPlayerInteract will be called, along with blockDamageEvent... >.>

 

oh D: I see xD Makes sense Never actually cared wether it was left or right click

 

P.S. I'm already following this thread, so I don't need to be tagged.
@Taco
Error:

"getData cannot be resolved or is not a field."

Line:

Code:java

 
			if(event.getClickedBlock().getTypeId() == 81 && event.getItem() != null && event.getItem().getTypeId() == 351 && event.getItem().getData == 15) {
 

Help?

 

Code:

if(event.hasBlock()){
	Block block = event.getClickedBlock();
	if(block.getType() == Material.CACTUS && event.getAction() == Action.RIGHT_CLICK_BLOCK){

Is that what you need?

 

I has to be

Code:java

 
			if(event.getClickedBlock().getTypeId() == 81 &&  event.getItem() != null && event.getItem().getTypeId() == 351  && event.getItem().getData().getData() == 15) {
 

 

Hehe. I may have made a typo in the code I gave you, this one should work. Thank you Pandemoneus for catching that.