mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-04 09:11:39 +00:00
split || if statements, remove unneeded parts
This commit is contained in:
parent
b3c825097a
commit
09d4e84008
@ -12,19 +12,17 @@ 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)) {
|
||||
incrementQualityByOne(i);
|
||||
} else if (items[i].name.equals(BACKSTAGE_PASSES)) {
|
||||
incrementQualityByOne(i);
|
||||
|
||||
if (items[i].name.equals(BACKSTAGE_PASSES)) {
|
||||
if (items[i].sellIn < 11) {
|
||||
incrementQualityByOne(i);
|
||||
}
|
||||
|
||||
if (items[i].sellIn < 6) {
|
||||
incrementQualityByOne(i);
|
||||
}
|
||||
if (items[i].sellIn < 6) {
|
||||
increaseQualityByTwo(i);
|
||||
} else if (items[i].sellIn < 11) {
|
||||
incrementQualityByOne(i);
|
||||
}
|
||||
|
||||
} else {
|
||||
decreaseQualityByOne(i);
|
||||
}
|
||||
@ -47,6 +45,11 @@ class GildedRose {
|
||||
}
|
||||
}
|
||||
|
||||
private void increaseQualityByTwo(int i) {
|
||||
incrementQualityByOne(i);
|
||||
incrementQualityByOne(i);
|
||||
}
|
||||
|
||||
private void decreaseQualityByOne(int i) {
|
||||
if (items[i].quality > 0) {
|
||||
if (!items[i].name.equals(SULFURAS)) {
|
||||
|
||||
Loading…
Reference in New Issue
Block a user