mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-16 06:51:27 +00:00
Refactor arithmetic operations
This commit is contained in:
parent
87a76edda3
commit
9482acd29c
@ -4,14 +4,14 @@ public class AgedBrieUpdateStrategy implements ItemUpdateStrategy {
|
|||||||
@Override
|
@Override
|
||||||
public void update(Item item) {
|
public void update(Item item) {
|
||||||
if (item.quality < 50) {
|
if (item.quality < 50) {
|
||||||
item.quality = item.quality + 1;
|
item.quality += 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
item.sellIn = item.sellIn - 1;
|
item.sellIn -= 1;
|
||||||
|
|
||||||
if (item.sellIn < 0) {
|
if (item.sellIn < 0) {
|
||||||
if (item.quality < 50) {
|
if (item.quality < 50) {
|
||||||
item.quality = item.quality + 1;
|
item.quality += 1;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -4,22 +4,22 @@ public class BackstagePassesUpdateStrategy implements ItemUpdateStrategy {
|
|||||||
@Override
|
@Override
|
||||||
public void update(Item item) {
|
public void update(Item item) {
|
||||||
if (item.quality < 50) {
|
if (item.quality < 50) {
|
||||||
item.quality = item.quality + 1;
|
item.quality += 1;
|
||||||
|
|
||||||
if (item.sellIn < 11) {
|
if (item.sellIn < 11) {
|
||||||
if (item.quality < 50) {
|
if (item.quality < 50) {
|
||||||
item.quality = item.quality + 1;
|
item.quality += 1;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (item.sellIn < 6) {
|
if (item.sellIn < 6) {
|
||||||
if (item.quality < 50) {
|
if (item.quality < 50) {
|
||||||
item.quality = item.quality + 1;
|
item.quality += 1;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
item.sellIn = item.sellIn - 1;
|
item.sellIn -= 1;
|
||||||
|
|
||||||
if (item.sellIn < 0) {
|
if (item.sellIn < 0) {
|
||||||
item.quality = 0;
|
item.quality = 0;
|
||||||
|
|||||||
@ -5,10 +5,10 @@ public class StandardItemUpdateStrategy implements ItemUpdateStrategy {
|
|||||||
public void update(Item item) {
|
public void update(Item item) {
|
||||||
item.sellIn -= 1;
|
item.sellIn -= 1;
|
||||||
if (item.quality > 0) {
|
if (item.quality > 0) {
|
||||||
item.quality = item.quality - 1;
|
item.quality -= 1;
|
||||||
}
|
}
|
||||||
if (item.sellIn < 0 && item.quality > 0) {
|
if (item.sellIn < 0 && item.quality > 0) {
|
||||||
item.quality = item.quality - 1;
|
item.quality -= 1;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user