How would I go about this?

Hi, I'm wondering how I would go about checking that the player has chainmail armour equipped and integrating it into this:

Code:java

 @EventHandler
public void onClick(PlayerInteractEvent event)
{
Player player = event.getPlayer();
 
if(player.getItemInHand().getType() == Material.GOLD_AXE && (event.getAction() == Action.RIGHT_CLICK_AIR))
{
player.sendMessage(ChatColor.BLUE + "" + ChatColor.BOLD + "WARRIOR: " + ChatColor.YELLOW + "" + ChatColor.BOLD + "USED WAR CRY");
player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 100, 2));
player.addPotionEffect(new PotionEffect(PotionEffectType.INCREASE_DAMAGE, 100, 2));
player.addPotionEffect(new PotionEffect(PotionEffectType.CONFUSION, 220, 1));
player.playSound(player.getLocation(), Sound.ENDERDRAGON_GROWL, 2, 1);
}
}

Thank you!

 

Hm... Why in if parameters you have player.getInventory()? For what?

Code:java

if(event.getAction == Action.RIGHT_CLICK_AIR){
if(event.getPlayer().getItemInHand().getType() == Material.GOLD_AXE){
//your code
}
}

 

Shit oops my bad, was just testing stuff out haha! Will delete.

 

Antigrate

Code:java

 
@EventHandler
public void onClick(PlayerInteractEvent event)
{
Player player = event.getPlayer();
 
if(player.getItemInHand().getType() == Material.GOLD_AXE && (event.getAction() == Action.RIGHT_CLICK_AIR))
&& player.getInventory().getHelmet().getType() == Material.WHATEVER) {
player.sendMessage(ChatColor.BLUE + "" + ChatColor.BOLD + "WARRIOR: " + ChatColor.YELLOW + "" + ChatColor.BOLD + "USED WAR CRY");
player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 100, 2));
player.addPotionEffect(new PotionEffect(PotionEffectType.INCREASE_DAMAGE, 100, 2));
player.addPotionEffect(new PotionEffect(PotionEffectType.CONFUSION, 220, 1));
player.playSound(player.getLocation(), Sound.ENDERDRAGON_GROWL, 2, 1);
}
}
 

Here is the doc on how to get all or only specific parts of the armour equipped: http://jd.bukkit.org/rb/apidocs/org/bukkit/inventory/PlayerInventory.html#getArmorContents()

 

So if a player has chainmail armor on, you want to add all those potion effects / sounds / messages?

Here is how to check for armor:

Code:java

if (player.getInventory().getChestplate().getType() == Material.CHAINMAIL_CHESTPLATE) {
//Add anything you want here
}

 

Hi, thanks for your reply!

I did this:

Code:java

 @EventHandler
public void onClick(PlayerInteractEvent event)
{
Player player = event.getPlayer();
 
if(player.getItemInHand().getType() == Material.GOLD_AXE && event.getAction() == Action.RIGHT_CLICK_AIR)
&& (player.getInventory().getHelmet().getType() == Material.CHAINMAIL_HELMET);
{
player.sendMessage(ChatColor.BLUE + "" + ChatColor.BOLD + "WARRIOR: " + ChatColor.YELLOW + "" + ChatColor.BOLD + "USED WAR CRY");
player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 100, 2));
player.addPotionEffect(new PotionEffect(PotionEffectType.INCREASE_DAMAGE, 100, 2));
player.addPotionEffect(new PotionEffect(PotionEffectType.CONFUSION, 220, 1));
player.playSound(player.getLocation(), Sound.ENDERDRAGON_GROWL, 2, 1);
}
}

But have errors in Eclipse for some reason:

No exception of type boolean can be thrown; an exception type must be a subclass of Throwable
Syntax error on token "&&", throw expected

What's going on here? Thank you!

EDIT: Accidentally added ; at the end for first error but still have the syntax error.

 

Antigrate The error is because you close off your if statement, and then you try to add something else to it.

You did:

Code:java

if(player.getItemInHand().getType() == Material.GOLD_AXE && event.getAction() == Action.RIGHT_CLICK_AIR)
&& (player.getInventory().getHelmet().getType() == Material.CHAINMAIL_HELMET);

But it should be this:

Code:java

if (player.getItemInHand().getType() == Material.GOLD_AXE && event.getAction() == Action.RIGHT_CLICK_AIR && player.getInventory().getHelmet().getType() == Material.CHAINMAIL_HELMET) {
//Stuff goes here
}

PS: When writing an if statement, you shouldn't NORMALLY have a semi colon, but there are exceptions..

 

Antigrate
Remove the ; after the if statement, if doesn't take a ; only in case its an inline statement.
You should also fix the brackets of the if. They seem to be messed up..

EDIT:
Squid_Boss had the better answer

 

Ah! Thank you very much, fixed!

 

Antigrate in what IDE you coding?.. Eclipse should automaticly checks for all errors.

 

The brackets work fine, some choose to press enter then have the brackets, and others keep them on the same line, it's just a preference, not an error ;P

 

@Squid_BossI meant these: >> ) << ^^

 

Ah...I call "{}" brackets, and "()" parentheses ^_^

 

Squid_Boss I think they're all "brackets" but parentheses sounds much better

 

Yeah I'm using Eclipse, but it doesn't help that a) I'm super tired b) 3rd or so day of learning Java and don't know how to fix all errors yet, so I get help on here!