public class Inventory { private Item[] items; int i; //removed for loop, defined i in Constructor public Inventory(Item[] items) { super(); this.items = items; i = 0; } public void updateQuality() { if (!items[i].getName().equals("Aged Brie")) { if (items[i].getQuality() > 0) { if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) { items[i].setQuality(items[i].getQuality() - 1); } } } else { if (items[i].getQuality() < 50) { items[i].setQuality(items[i].getQuality() + 1); } } if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) { items[i].setSellIn(items[i].getSellIn() - 1); } if (items[i].getSellIn() < 0) { if (!items[i].getName().equals("Aged Brie")) { items[i].setQuality(items[i].getQuality() - 1); } else { if (items[i].getQuality() < 50) { items[i].setQuality(items[i].getQuality() + 1); } } } } }