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("Conjured")){ items[i].setQuality(items[i].getQuality() *2); } if (!items[i].getName().equals("Aged Brie") && !items[i].getName().equals("Sulfuras, Hand of Ragnaros")) { if (items[i].getQuality() > 0 && items[i].getQuality() < 50) { items[i].setQuality(items[i].getQuality() - 1); items[i].setSellIn(items[i].getSellIn() - 1); } } else { if (items[i].getQuality() < 50) { items[i].setQuality(items[i].getQuality() + 1); } } if (items[i].getSellIn() < 0 && !items[i].getName().equals("Aged Brie")) { items[i].setQuality(items[i].getQuality() - 1); } if (items[i].getName().equals("Aged Brie") && items[i].getQuality() < 50 && items[i].getSellIn() < 0 ){ items[i].setQuality(items[i].getQuality() + 1); } } }