mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-04 09:11:39 +00:00
split || if condition
This commit is contained in:
parent
f0afa8c7d7
commit
3ab5fb11d1
@ -12,8 +12,21 @@ class GildedRose {
|
||||
|
||||
public void updateQuality() {
|
||||
for (int i = 0; i < items.length; i++) {
|
||||
if (items[i].name.equals(AGED_BRIE)
|
||||
|| items[i].name.equals(BACKSTAGE_PASSES)) {
|
||||
if (items[i].name.equals(AGED_BRIE)) {
|
||||
if (items[i].quality < 50) {
|
||||
items[i].quality = items[i].quality + 1;
|
||||
|
||||
if (items[i].name.equals(BACKSTAGE_PASSES)) {
|
||||
if (items[i].sellIn < 11) {
|
||||
incrementQualityByOne(i);
|
||||
}
|
||||
|
||||
if (items[i].sellIn < 6) {
|
||||
incrementQualityByOne(i);
|
||||
}
|
||||
}
|
||||
}
|
||||
} else if (items[i].name.equals(BACKSTAGE_PASSES)) {
|
||||
if (items[i].quality < 50) {
|
||||
items[i].quality = items[i].quality + 1;
|
||||
|
||||
@ -28,11 +41,7 @@ class GildedRose {
|
||||
}
|
||||
}
|
||||
} else {
|
||||
if (items[i].quality > 0) {
|
||||
if (!items[i].name.equals(SULFURAS)) {
|
||||
items[i].quality = items[i].quality - 1;
|
||||
}
|
||||
}
|
||||
decreaseQualityByOne(i);
|
||||
}
|
||||
|
||||
if (!items[i].name.equals(SULFURAS)) {
|
||||
@ -46,17 +55,21 @@ class GildedRose {
|
||||
if (items[i].name.equals(BACKSTAGE_PASSES)) {
|
||||
items[i].quality = 0;
|
||||
} else {
|
||||
if (items[i].quality > 0) {
|
||||
if (!items[i].name.equals(SULFURAS)) {
|
||||
items[i].quality = items[i].quality - 1;
|
||||
}
|
||||
}
|
||||
decreaseQualityByOne(i);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private void decreaseQualityByOne(int i) {
|
||||
if (items[i].quality > 0) {
|
||||
if (!items[i].name.equals(SULFURAS)) {
|
||||
items[i].quality = items[i].quality - 1;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private void incrementQualityByOne(int i) {
|
||||
if (items[i].quality < 50) {
|
||||
items[i].quality = items[i].quality + 1;
|
||||
|
||||
Loading…
Reference in New Issue
Block a user