From 1f1275f4db93e639175c817ffa9158dcb387c36f Mon Sep 17 00:00:00 2001 From: pierre Date: Sun, 19 Feb 2023 15:45:20 +0100 Subject: [PATCH] refactor: improving the placement of the SellIn Counter to make the placement more intuitive --- Java/src/main/java/com/gildedrose/GildedRose.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/Java/src/main/java/com/gildedrose/GildedRose.java b/Java/src/main/java/com/gildedrose/GildedRose.java index 0c285a8f..cfd0cdac 100644 --- a/Java/src/main/java/com/gildedrose/GildedRose.java +++ b/Java/src/main/java/com/gildedrose/GildedRose.java @@ -34,9 +34,7 @@ class GildedRose { updateNormalAndConjuredItems(items[i]); } - items[i].sellIn = items[i].sellIn - 1; - - if (items[i].sellIn < 0) { + if (items[i].sellIn <= 0) { if (items[i].name.equals("Aged Brie") && items[i].quality < 50) { items[i].quality = items[i].quality + 1; @@ -50,6 +48,7 @@ class GildedRose { items[i].quality = items[i].quality - items[i].quality; } } + items[i].sellIn = items[i].sellIn - 1; } }