CleanCode-GildedRoseKata/Inventory.java

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") && !items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
            if (items[i].getQuality() > 0 && items[i].getQuality() < 50) {    //cp1
                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);
            } 
        }
        //new brie
        if (items[i].getName().equals("Aged Brie") && items[i].getQuality() < 50){
            items[i].setQuality(items[i].getQuality() + 1);

        }
    }
}