diff --git a/Java/src/main/java/com/gildedrose/GildedRose.java b/Java/src/main/java/com/gildedrose/GildedRose.java index 6865df66..50f51204 100644 --- a/Java/src/main/java/com/gildedrose/GildedRose.java +++ b/Java/src/main/java/com/gildedrose/GildedRose.java @@ -13,31 +13,32 @@ class GildedRose { for (int i = 0; i < items.length; i++) { // ----- Block 1 ----- Reduce quality of item by 1 for a day if not AB, BptaTc or SHoR - if (!items[i].name.equals("Aged Brie") - && !items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) { - if (items[i].quality > 0) { - if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) { - items[i].quality = items[i].quality - 1; + Item item = items[i]; + if (!item.name.equals("Aged Brie") + && !item.name.equals("Backstage passes to a TAFKAL80ETC concert")) { + if (item.quality > 0) { + if (!item.name.equals("Sulfuras, Hand of Ragnaros")) { + item.quality = item.quality - 1; } } // If Aged Brie or BptaTc } else { // Increase quality of item by + 1 - if (items[i].quality < 50) { - items[i].quality = items[i].quality + 1; + if (item.quality < 50) { + item.quality = item.quality + 1; // If pass has less than 11 days to sell, quality increases by + 1 - if (items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) { - if (items[i].sellIn < 11) { - if (items[i].quality < 50) { - items[i].quality = items[i].quality + 1; + if (item.name.equals("Backstage passes to a TAFKAL80ETC concert")) { + if (item.sellIn < 11) { + if (item.quality < 50) { + item.quality = item.quality + 1; } } // If pass has 6 days to sell, quality increases by + 2 - if (items[i].sellIn < 6) { - if (items[i].quality < 50) { - items[i].quality = items[i].quality + 1; + if (item.sellIn < 6) { + if (item.quality < 50) { + item.quality = item.quality + 1; } } } @@ -45,30 +46,30 @@ class GildedRose { } // ----- Block 2 ------ Item reduces by a day - 1 (unless SHoR) - if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) { - items[i].sellIn = items[i].sellIn - 1; + if (!item.name.equals("Sulfuras, Hand of Ragnaros")) { + item.sellIn = item.sellIn - 1; } // ----- Block 3 ----- Item sell date is less than 0 - if (items[i].sellIn < 0) { + if (item.sellIn < 0) { // Reduce quality of items by - 1 if greater than 0 and not AB, BptaTc or SHoR - if (!items[i].name.equals("Aged Brie")) { - if (!items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) { - if (items[i].quality > 0) { - if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) { - items[i].quality = items[i].quality - 1; + if (!item.name.equals("Aged Brie")) { + if (!item.name.equals("Backstage passes to a TAFKAL80ETC concert")) { + if (item.quality > 0) { + if (!item.name.equals("Sulfuras, Hand of Ragnaros")) { + item.quality = item.quality - 1; } } } else { // Item quality goes to 0 of BptaTc as item sell date is less than 0 - items[i].quality = items[i].quality - items[i].quality; + item.quality = item.quality - item.quality; } // Increase quality of item if AB } else { - if (items[i].quality < 50) { - items[i].quality = items[i].quality + 1; + if (item.quality < 50) { + item.quality = item.quality + 1; } } }