diff --git a/Java/pom.xml b/Java/pom.xml index 318af002..da2ab7b2 100644 --- a/Java/pom.xml +++ b/Java/pom.xml @@ -33,7 +33,7 @@ org.projectlombok lombok ${lombok.version} - provided + test org.assertj diff --git a/Java/src/main/java/com/gildedrose/GildedRose.java b/Java/src/main/java/com/gildedrose/GildedRose.java index 8729787b..960b8876 100644 --- a/Java/src/main/java/com/gildedrose/GildedRose.java +++ b/Java/src/main/java/com/gildedrose/GildedRose.java @@ -7,7 +7,6 @@ import com.gildedrose.rules.DefaultQualityRule; import com.gildedrose.rules.QualityRule; import com.gildedrose.rules.Result; import com.gildedrose.rules.SulfurasQualityRule; -import lombok.val; import java.util.Arrays; import java.util.List; @@ -35,21 +34,19 @@ class GildedRose { private void processItem(final Item item) { - val newSellIn = calculateSellIn(item.name, item.sellIn); + item.sellIn = calculateSellIn(item.name, item.sellIn); - val newQuality = rules.stream() + item.quality = rules.stream() .reduce(new Result(item.quality, false), (q, rule) -> { if (q.isFinalValue() || !rule.shouldApply(item.name)) { return q; } else { - return rule.calculateQuality(q.getQuality(), newSellIn); + return rule.calculateQuality(q.getQuality(), item.sellIn); } }, (a, b) -> b).getQuality(); - item.quality = newQuality; - item.sellIn = newSellIn; } private int calculateSellIn(final String name, final int originalSellIn) {