diff --git a/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java b/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java index 82b1e1bd..5fe3440e 100644 --- a/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java +++ b/Java-Approvals/src/main/java/com/gildedrose/GildedRose.java @@ -10,9 +10,9 @@ class GildedRose { public void updateQuality() { for (Item item : items) { - if (item.name.equals(Item.AGED_BRIE)) { + if (isAgedBride(item)) { item.increaseQualityByOne(); - } else if (item.name.equals(Item.BACKSTAGE_PASSES)) { + } else if (isBackstagePasses(item)) { item.increaseQualityBackstage(); } else { item.decreaseQualityByOne(); @@ -21,9 +21,9 @@ class GildedRose { item.decreaseSellInEachDay(); if (item.sellIn < 0) { - if (item.name.equals(Item.AGED_BRIE)) { + if (isAgedBride(item)) { item.increaseQualityByOne(); - } else if (item.name.equals(Item.BACKSTAGE_PASSES)) { + } else if (isBackstagePasses(item)) { item.quality = 0; } else { item.decreaseQualityByOne(); @@ -32,4 +32,12 @@ class GildedRose { } } + private static boolean isBackstagePasses(Item item) { + return item.name.equals(Item.BACKSTAGE_PASSES); + } + + private static boolean isAgedBride(Item item) { + return item.name.equals(Item.AGED_BRIE); + } + }