mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-18 07:51:29 +00:00
move quality updates after sell in value updates to quality updates block
This commit is contained in:
parent
9a1ae8ab9d
commit
30a03a10d5
@ -23,23 +23,22 @@ class GildedRose {
|
|||||||
if (isEnhancingItem(item)) {
|
if (isEnhancingItem(item)) {
|
||||||
int qualityOffset = determineQualityOffset(item);
|
int qualityOffset = determineQualityOffset(item);
|
||||||
enhanceItem(item, qualityOffset);
|
enhanceItem(item, qualityOffset);
|
||||||
|
if (item.sellIn <= 0) {
|
||||||
|
if( item.name.equals(AGED_BRIE)) {
|
||||||
|
enhanceItem(item, 1);
|
||||||
|
} else {
|
||||||
|
degradeItem(item, item.quality);
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
||||||
} else {
|
} else {
|
||||||
degradeItem(item,1);
|
degradeItem(item,1);
|
||||||
|
if(item.sellIn <= 0) {
|
||||||
|
degradeItem(item, 1);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
updateSellInValue(item);
|
updateSellInValue(item);
|
||||||
|
|
||||||
if (item.sellIn < 0) {
|
|
||||||
if (item.name.equals(AGED_BRIE)) {
|
|
||||||
enhanceItem(item, 1);
|
|
||||||
} else {
|
|
||||||
if (item.name.equals(BACKSTAGE_PASSES)) {
|
|
||||||
degradeItem(item,item.quality);
|
|
||||||
} else {
|
|
||||||
degradeItem(item,1);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -111,6 +110,6 @@ class GildedRose {
|
|||||||
* @param item the item of which the sell in value is updated
|
* @param item the item of which the sell in value is updated
|
||||||
*/
|
*/
|
||||||
private void updateSellInValue(Item item) {
|
private void updateSellInValue(Item item) {
|
||||||
item.sellIn = item.sellIn - 1;
|
item.sellIn -= 1;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
Loading…
Reference in New Issue
Block a user