Player Event Question

I want to constantly check what the user is wearing, what would be the best solution for that? Non laggiest solution as well too.
public void CheckItems(WhichEventHandler e) {
player.getInventory().getHelmet().getType() == Material.DIAMOND_BOOTS) {
Do stuff

}
}
Thanks

 

@Eggspurt https://bukkit.org/threads/event-that-happens-when-you-equip-unequip-armor.138690/

 

Follow Java Naming Conventions

 

Really irrelevant to what I was looking for thats not my method I just made it up to explain.

Thanks for actually helping

 

@Eggspurt it is indeed, but pointing out other mistakes in code is useful for the coder to make his code better or learn something new, in the last few years my eyes are trained and go straight to such imperfections xD

 

Anytime ;3

 

I didn't mean to sound rude; as I did. Thanks for pointing it out.

InventoryClickEvent did not work but I did use SneakEvent... But for some reason if I try sneaking again it does not re-add the potion effect I have to wait until it's gone.

 

@Eggspurt Show your code.

 

Code:

    @EventHandler
    public void SpeedAbilities(PlayerToggleSneakEvent e) {
        Player player = e.getPlayer();
        ItemStack boots = player.getInventory().getBoots();
        ItemStack helm = player.getInventory().getHelmet();
        ItemStack leg = player.getInventory().getLeggings();
        ItemStack body = player.getInventory().getChestplate();
      
        ItemStack[] equip = {boots, helm, leg, body};
      
        if (equip != null) {
        if (boots.getType() == Material.GOLD_BOOTS
                && helm.getType() == Material.GOLD_HELMET
                && leg.getType() == Material.IRON_LEGGINGS
                && body.getType() == Material.IRON_CHESTPLATE) {
            player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 300, 1));
                    }
                }
            }
        }

 

@Eggspurt Use some schedulers to test for armor being worn. May be laggier, but that would bypass the need to test for armor equipped, and armor de-equipped.

 

Why do you need the array? It is never used. Also, you don't check to see if boots/helm/leg/body is == null. If someone shifts while not wearing any armour, it will throw a NPE.