diff --git a/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java b/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java index f6955e5f..7e8d4a8c 100644 --- a/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java +++ b/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java @@ -28,8 +28,10 @@ class GildedRose { } } } else { - if (!items[i].name.equals(SULFURAS)) { - decreaseQualityByOne(i); + if (items[i].quality > 0) { + if (!items[i].name.equals(SULFURAS)) { + decreaseQualityByOne(i); + } } } @@ -44,8 +46,10 @@ class GildedRose { if (items[i].name.equals(BACKSTAGE_PASSES)) { items[i].quality = 0; } else { - if (!items[i].name.equals(SULFURAS)) { - decreaseQualityByOne(i); + if (items[i].quality > 0) { + if (!items[i].name.equals(SULFURAS)) { + decreaseQualityByOne(i); + } } } } @@ -60,9 +64,7 @@ class GildedRose { } private void decreaseQualityByOne(int i) { - if (items[i].quality > 0) { - items[i].quality = items[i].quality - 1; - } + items[i].quality = items[i].quality - 1; } }