Помогите Проблема с проверкой брони

Вот таким способом проверяю есть ли на игроке броня.

Код:

                 for(Player pl: Bukkit.getOnlinePlayers()){
                     ItemStack[] armor = pl.getInventory().getArmorContents();
                    
                     if (armor[0].getType() == Material.CHAINMAIL_HELMET) {
                        }
                        if (armor[1].getType() == Material.CHAINMAIL_CHESTPLATE) {
                        }
                        if (armor[2].getType() == Material.CHAINMAIL_LEGGINGS) {
                        }
                        if (armor[3].getType() == Material.CHAINMAIL_BOOTS) {
                        }
                 }

Если на нем нет брони то выдает ошибку. Почему это происходит и как можно это исправить ?

 

Нужно добавить подобные проверки:

Код:

armor[N] != null

 

Эту магию сложно понять. То оно воздух то null.

 

Код:

try{

}catch(NullPointerException e){
p.sendMessage("На тебе нет брони какой-то");
}

 

Фи, как неэффективно. Исключения стоят дорого, так что так можно легко просадить производительность.

 

На 1.17.10 достаточно было на AIR, а потом 1.12 добавилось != null.

 

Пустым catch-ем? Да лан?

 

Он не пустой. Там есть вывод сообщения. В любом случае это гораздо дороже обычного null check.

 

Ты замерял?

 

Конкретно этот случай - нет, но другие - да.
UPD. Замерил. В этом случае разница невелика, но обычная проверка на null всё равно компактнее с точки зрения исходников и быстрее на какие-то микросекунды.

 

Я вот в эсеншелсе видел, что каждая "ошибка" аля игрок не найден, недостаточно аргументов - кидается исключение. Зачем...

 

player.getEquipment().getHelmet(). Ошибка выдается потому, что item null, вызвать getType() нельзя

 

Полагаю, так удобно выбраться в код, запустивший выполнение команды, не пробрасывая данные через кучу return'ов.