Solved Send a message when a player has no armour equipped.

Code:

        @EventHandler
        public void eventInventoryClose(final InventoryCloseEvent event){
            final Player player = (Player) event.getPlayer();
            final PlayerInventory inventory = event.getPlayer().getInventory();
            if(inventory.getHelmet() == null
                    || inventory.getChestplate() == null
                    || inventory.getLeggings() == null
                    || inventory.getBoots() == null){
                            ((Player)event.getPlayer()).sendMessage("Armor Set: None");
                        }
                    }

This is not working. I can't seem to find a reason why.

 

Looks fine to me, show where you are registering it.

 

What do you mean?

 

Show the top part of the class file, and the top part of the main class file too.

 

Don't use null for Armor, use Material.AIR.

 

@racoonsru1e Your checking if one is null, do && not ||

&& = and
|| = or

 

I want it to send a message when they have an incomplete set on as well...

Here is the full class: http://pastebin.com/zdb2ypPH

 

Why not just

if(p.getInventory().getArmourContents() == null){

}

 

If I did that, then if the player takes only one, two or three pieces of their armour off, it will not send them a message.

EDIT: I did try it but it did not work for some reason...

Created a separate event handler with the exact same code that I started with in it and tested it. For some reason it worked. Thanks for all the help anyway!

Solved.

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