diff --git a/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java b/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java index a523e131..a7d55daf 100644 --- a/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java +++ b/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java @@ -15,18 +15,18 @@ class GildedRose { if (items[i].name.equals(AGED_BRIE) || items[i].name.equals(BACKSTAGE_PASSES)) { if (items[i].quality < 50) { - IncrementQuality(i); + incrementQualityByOne(i); if (items[i].name.equals(BACKSTAGE_PASSES)) { if (items[i].sellIn < 11) { if (items[i].quality < 50) { - IncrementQuality(i); + incrementQualityByOne(i); } } if (items[i].sellIn < 6) { if (items[i].quality < 50) { - IncrementQuality(i); + incrementQualityByOne(i); } } } @@ -34,7 +34,7 @@ class GildedRose { } else { if (items[i].quality > 0) { if (!items[i].name.equals(SULFURAS)) { - items[i].quality = items[i].quality - 1; + decreaseQualityByOne(i); } } } @@ -48,22 +48,26 @@ class GildedRose { if (!items[i].name.equals(BACKSTAGE_PASSES)) { if (items[i].quality > 0) { if (!items[i].name.equals(SULFURAS)) { - items[i].quality = items[i].quality - 1; + decreaseQualityByOne(i); } } } else { - items[i].quality = items[i].quality - items[i].quality; + items[i].quality = 0; } } else { if (items[i].quality < 50) { - IncrementQuality(i); + incrementQualityByOne(i); } } } } } - private void IncrementQuality(int i) { + private void decreaseQualityByOne(int i) { + items[i].quality = items[i].quality - 1; + } + + private void incrementQualityByOne(int i) { items[i].quality = items[i].quality + 1; } }