mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-03-01 13:21:14 +00:00
message
This commit is contained in:
parent
21cb24bf97
commit
72f0a797bc
@ -5,6 +5,8 @@ class GildedRose {
|
|||||||
public static final String AGED_BRIE = "Aged Brie";
|
public static final String AGED_BRIE = "Aged Brie";
|
||||||
public static final String SULFURAS = "Sulfuras, Hand of Ragnaros";
|
public static final String SULFURAS = "Sulfuras, Hand of Ragnaros";
|
||||||
public static final String BACKSTAGE = "Backstage passes to a TAFKAL80ETC concert";
|
public static final String BACKSTAGE = "Backstage passes to a TAFKAL80ETC concert";
|
||||||
|
public static final String CONJURED = "Conjured Mana Cake";
|
||||||
|
private Item[] items;
|
||||||
|
|
||||||
private Item[] items;
|
private Item[] items;
|
||||||
|
|
||||||
@ -19,17 +21,31 @@ class GildedRose {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private void updateItemQuality(Item item) {
|
private void updateItemQuality(Item item) {
|
||||||
if (!item.name.equals(AGED_BRIE)
|
boolean isExpired = item.sellIn < 1;
|
||||||
&& !item.name.equals(BACKSTAGE)) {
|
int degradeRate = determineDegradeRate(item, isExpired);
|
||||||
if (item.quality > 0) {
|
boolean doesDegrade = !item.name.equals(AGED_BRIE) && !item.name.equals(BACKSTAGE) && !item.name.equals(SULFURAS);
|
||||||
if (!item.name.equals(SULFURAS)) {
|
boolean hasSellBy = !item.name.equals(SULFURAS);
|
||||||
adjustQuality(item, -1);
|
|
||||||
|
if (doesDegrade) {
|
||||||
|
adjustQuality(item, degradeRate);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (item.name.equals(AGED_BRIE)) {
|
||||||
|
int adjustment = isExpired ? 2 : 1;
|
||||||
|
adjustQuality(item, adjustment);
|
||||||
}
|
}
|
||||||
} else {
|
|
||||||
adjustQuality(item, 1);
|
|
||||||
|
|
||||||
if (item.name.equals(BACKSTAGE)) {
|
if (item.name.equals(BACKSTAGE)) {
|
||||||
|
updateBackStagePass(item, isExpired);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (hasSellBy) {
|
||||||
|
item.sellIn = item.sellIn - 1;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
private void updateBackStagePass(Item item, boolean isExpired) {
|
||||||
|
adjustQuality(item, 1);
|
||||||
if (item.sellIn < 11) {
|
if (item.sellIn < 11) {
|
||||||
adjustQuality(item, 1);
|
adjustQuality(item, 1);
|
||||||
}
|
}
|
||||||
@ -37,33 +53,20 @@ class GildedRose {
|
|||||||
if (item.sellIn < 6) {
|
if (item.sellIn < 6) {
|
||||||
adjustQuality(item, 1);
|
adjustQuality(item, 1);
|
||||||
}
|
}
|
||||||
}
|
if (isExpired) {
|
||||||
}
|
|
||||||
|
|
||||||
if (!item.name.equals(SULFURAS)) {
|
|
||||||
item.sellIn = item.sellIn - 1;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (item.sellIn < 0) {
|
|
||||||
if (!item.name.equals(AGED_BRIE)) {
|
|
||||||
if (!item.name.equals(BACKSTAGE)) {
|
|
||||||
if (!item.name.equals(SULFURAS)) {
|
|
||||||
adjustQuality(item, -1);
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
item.quality = item.quality - item.quality;
|
item.quality = item.quality - item.quality;
|
||||||
}
|
}
|
||||||
} else {
|
|
||||||
int adjustment = 1;
|
|
||||||
adjustQuality(item, adjustment);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
public void adjustQuality(Item item, int adjustment) {
|
private int determineDegradeRate(Item item, boolean isExpired) {
|
||||||
|
int baseDegradeRate = item.name.equals(CONJURED) ? -2 : -1;
|
||||||
|
return isExpired ? baseDegradeRate * 2 : baseDegradeRate;
|
||||||
|
}
|
||||||
|
|
||||||
|
private void adjustQuality(Item item, int adjustment) {
|
||||||
int newQuality = item.quality + adjustment;
|
int newQuality = item.quality + adjustment;
|
||||||
boolean isValid = newQuality <= 50 && newQuality >= 0;
|
boolean isInvalidRange = newQuality <= 50 && newQuality >= 0;
|
||||||
if (isValid) {
|
if (isInvalidRange) {
|
||||||
item.quality = newQuality;
|
item.quality = newQuality;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user