Solved Is there a more efficient way to do this?

Hey everyone! So, there's a bit of repetitive code in this method regarding setting a player's armor contents depending on their rank. I was wondering if there is a more efficient way of doing it that doesn't look as ugly Thanks in advance

Code:java

    /**
    * Gives a player a set of armor depending on their rank
    *
    * @param player : The Player who will receive the armor
    */
    public static void giveDefenseArmor(Player player){
        PlayerInventory i = player.getInventory();
 
        if (getArmorTier(player).equals("Diamond")){
            i.setHelmet(new ItemStack(Material.DIAMOND_HELMET));
            i.setChestplate(new ItemStack(Material.DIAMOND_CHESTPLATE));
            i.setLeggings(new ItemStack(Material.DIAMOND_LEGGINGS));
            i.setBoots(new ItemStack(Material.DIAMOND_BOOTS));
        }
 
        if (getArmorTier(player).equals("Iron")){
            i.setHelmet(new ItemStack(Material.IRON_HELMET));
            i.setChestplate(new ItemStack(Material.IRON_CHESTPLATE));
            i.setLeggings(new ItemStack(Material.IRON_LEGGINGS));
            i.setBoots(new ItemStack(Material.IRON_BOOTS));
        }
 
        if (getArmorTier(player).equals("Gold")){
            i.setHelmet(new ItemStack(Material.GOLD_HELMET));
            i.setChestplate(new ItemStack(Material.GOLD_CHESTPLATE));
            i.setLeggings(new ItemStack(Material.GOLD_LEGGINGS));
            i.setBoots(new ItemStack(Material.GOLD_BOOTS));
        }
 
        if (getArmorTier(player).equals("Leather")){
            i.setHelmet(new ItemStack(Material.LEATHER_HELMET));
            i.setChestplate(new ItemStack(Material.LEATHER_CHESTPLATE));
            i.setLeggings(new ItemStack(Material.LEATHER_LEGGINGS));
            i.setBoots(new ItemStack(Material.LEATHER_BOOTS));
        }
    }

 

The Gaming Grunts

Code:java

public void giveDefenseArmor(Player player) {
    ArrayList<ItemStack> contents = new ArrayList<>();
    for (String suffix : Arrays.asList("HELMET", "CHESTPLATE", "LEGGINGS", "BOOTS"))
        contents.add(new ItemStack(Material.valueOf(getArmorTier(player).toUpperCase() + "_" + suffix)));
    player.getInventory.setArmorContents(contents.toArray(new ItemStack[4]));
}

That ought to do it.

 

Maurdekye
Wow thanks. I'll try it out and let you know how it works

Edit:
Maurdekye
Works like a charm. Thanks again