From 98cb08556b0625a45b1d22d5d547b34f53538e00 Mon Sep 17 00:00:00 2001 From: iTwins Date: Fri, 25 Aug 2023 18:39:31 +0200 Subject: [PATCH] use local variable to avoid accessing array multiple times --- .../slimefun4/implementation/listeners/JoinListener.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/main/java/io/github/thebusybiscuit/slimefun4/implementation/listeners/JoinListener.java b/src/main/java/io/github/thebusybiscuit/slimefun4/implementation/listeners/JoinListener.java index 202fe079c..28d15851b 100644 --- a/src/main/java/io/github/thebusybiscuit/slimefun4/implementation/listeners/JoinListener.java +++ b/src/main/java/io/github/thebusybiscuit/slimefun4/implementation/listeners/JoinListener.java @@ -33,8 +33,9 @@ public class JoinListener implements Listener { final ItemStack[] armorContents = e.getPlayer().getInventory().getArmorContents(); final HashedArmorpiece[] hashedArmorpieces = playerProfile.getArmor(); for (int i = 0; i < 4; i++) { - if (armorContents[i] != null && armorContents[i].getType() != Material.AIR && SlimefunItem.getByItem(armorContents[i]) instanceof SlimefunArmorPiece sfArmorPiece) { - hashedArmorpieces[i].update(armorContents[i], sfArmorPiece); + final ItemStack armorPiece = armorContents[i]; + if (armorPiece != null && armorPiece.getType() != Material.AIR && SlimefunItem.getByItem(armorPiece) instanceof SlimefunArmorPiece sfArmorPiece) { + hashedArmorpieces[i].update(armorPiece, sfArmorPiece); } } });