Check if a player is op?

Is it possible to do an if statement like if (player != op) {whatever} ? If not how could i check for if a player is an op so they can do stuff others cant. Ty for any help

 

I am not really good at these kinds of things, but I think it is somewhere along the lines of:

Correction.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Im not sure if im not using the code right or if im just missing something but it dosent seem to work for me :/

 

Code:java

Player player = event.getPlayer;
if (player.isOp()) {
//Do the things you wanna do
}

 

Code:java

 
 
what if im checking for if the player is not op?

 

Then you either do, if (!player.isOp()) { or if (player.isOp() == false) {

Either of those should work, but the first one is probably your best option

 

You'll need to import Player variable then do the following operation.

Code:java

 
if(player.isOp()) {
// Execute a body
}
 
// If not..
if(!player.isOp()){
// Execute a body
}
 

 

So exactly what I said

Where exactly are you trying to do this? In a listener or a command oorrr?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

I keep getting an error for this. Do i have to have some kind of listener? Something is wrong with the "player" part for me.

ya im trying to put it inside a listener for whenever a player interacts with flint and steel.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Have you imported from this way:

Listener

Code:java

 
Player player = event.getPlayer();
 

 

Alright ... If you don't mind, would you be able to post your code here? To see how you're doing it? I have to actually see what you're doing to be able to help

 

public class MyPlayerListener extends PlayerListener {
@Override
public void onPlayerInteract(PlayerInteractEvent event) {
if (event.getItem().getType() == Material.FLINT_AND_STEEL) {
if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
Player player = event.getPlayer();
player.getInventory().remove(new ItemStack(Material.FLINT_AND_STEEL,1));
event.setCancelled(true);
getServer().broadcastMessage(ChatColor.GREEN + player.getName() + "'s Fire Was Suppressed!");
}
}
}
}

 

If you want to place a block somewhere in the middle of the code, you have to build it on top of the other blocks then add a new ending curly bracket for it. I can place the OP checking block. Where do you want me to place it?

 

Do you have the listener in a new class or in the main class, along with your onEnable and stuff

 

There is actually another part. I wanted this whole part to only happen if the player is not op so im not sure if you would have to do it once for each class

public class MyBlockListener extends BlockListener {
@Override
public void onBlockIgnite(BlockIgniteEvent event)
{
event.setCancelled(true);
}
}

public class MyPlayerListener extends PlayerListener {
@Override
public void onPlayerInteract(PlayerInteractEvent event) {
if (event.getItem().getType() == Material.FLINT_AND_STEEL) {
if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
Player player = event.getPlayer();
player.getInventory().remove(new ItemStack(Material.FLINT_AND_STEEL,1));
event.setCancelled(true);
getServer().broadcastMessage(ChatColor.GREEN + player.getName() + "'s Fire Was Suppressed!");
}
}
}
}
}

Ya the other part that says which event too look for is under onEnable

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Try replacing your onInteract with this one:

Code:java

public void onPlayerInteract(PlayerInteractEvent event) {
		Player player = event.getPlayer();
		ItemStack held = player.getItemInHand();
		Action action = event.getAction();
		if (held.getType() == Material.FLINT_AND_STEEL) {
			if (action == Action.RIGHT_CLICK_BLOCK) {
				player.getInventory().remove(new ItemStack(Material.FLINT_AND_STEEL,1));
				event.setCancelled(true);
				this.getServer().broadcastMessage(ChatColor.GREEN + player.getName() + "'s Fire Was Suppressed!");
			}
		}
	}

 

The way i have it now stops all fire. That one only prevents flint and steel from being used.

 

That's just moving your code around and cleaning it, you can still keep the onBlockIgnite method

 

You would cancel it.

 

Cancel what?

 

So what about the op issue? Any ideas what im not doing right when trying to use it?

 

What isn't working?

 

Explain in-depth what you're trying to do?

 

Never mind i fixed it.

 

Good to hear What was the issue?

 

Just forgot () in a section lol