diff --git a/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java b/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java index a7d55daf..cc3696ab 100644 --- a/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java +++ b/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java @@ -15,19 +15,15 @@ class GildedRose { if (items[i].name.equals(AGED_BRIE) || items[i].name.equals(BACKSTAGE_PASSES)) { if (items[i].quality < 50) { - incrementQualityByOne(i); + items[i].quality = items[i].quality + 1; if (items[i].name.equals(BACKSTAGE_PASSES)) { if (items[i].sellIn < 11) { - if (items[i].quality < 50) { - incrementQualityByOne(i); - } + incrementQualityByOne(i); } if (items[i].sellIn < 6) { - if (items[i].quality < 50) { - incrementQualityByOne(i); - } + incrementQualityByOne(i); } } } @@ -55,19 +51,20 @@ class GildedRose { items[i].quality = 0; } } else { - if (items[i].quality < 50) { - incrementQualityByOne(i); - } + incrementQualityByOne(i); } } } } + private void incrementQualityByOne(int i) { + if (items[i].quality < 50) { + items[i].quality = items[i].quality + 1; + } + } + private void decreaseQualityByOne(int i) { items[i].quality = items[i].quality - 1; } - private void incrementQualityByOne(int i) { - items[i].quality = items[i].quality + 1; - } }