mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-15 06:21:29 +00:00
move specific logic to backstage item
This commit is contained in:
parent
fa1749ec17
commit
0cb4b53cb4
@ -28,7 +28,7 @@
|
|||||||
-[x] long method
|
-[x] long method
|
||||||
-[ ] plain logic
|
-[ ] plain logic
|
||||||
-[ ] complex operations
|
-[ ] complex operations
|
||||||
-[ ] Item properties are public
|
-[x] Item properties are public
|
||||||
-[x] multiple access by index
|
-[x] multiple access by index
|
||||||
|
|
||||||
## Refactoring actions
|
## Refactoring actions
|
||||||
@ -39,5 +39,5 @@
|
|||||||
-[x] extract boolean methods
|
-[x] extract boolean methods
|
||||||
-[x] encapsulate methods
|
-[x] encapsulate methods
|
||||||
-[ ] isolate specific logics
|
-[ ] isolate specific logics
|
||||||
-[ ] move specific logic to backstage
|
-[x] move specific logic to backstage
|
||||||
-[x] replace items by specific objects in tests
|
-[x] replace items by specific objects in tests
|
||||||
|
|||||||
@ -55,9 +55,6 @@ class GildedRose {
|
|||||||
|
|
||||||
private void increaseQuality(Item item) {
|
private void increaseQuality(Item item) {
|
||||||
item.increaseQuality();
|
item.increaseQuality();
|
||||||
if (item.name.equals(Backstage.BACKSTAGE)) {
|
|
||||||
item.increaseBackstageQuality();
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private void updateSellIn(Item item) {
|
private void updateSellIn(Item item) {
|
||||||
|
|||||||
@ -7,4 +7,17 @@ public class Backstage extends Item {
|
|||||||
public Backstage(int sellIn, int quality) {
|
public Backstage(int sellIn, int quality) {
|
||||||
super(BACKSTAGE, sellIn, quality);
|
super(BACKSTAGE, sellIn, quality);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void increaseQuality() {
|
||||||
|
super.increaseQuality();
|
||||||
|
|
||||||
|
if (this.sellIn < 11 && this.quality < 50) {
|
||||||
|
super.increaseQuality();
|
||||||
|
}
|
||||||
|
|
||||||
|
if (this.sellIn < 6 && this.quality < 50) {
|
||||||
|
super.increaseQuality();
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -26,14 +26,4 @@ public class Item {
|
|||||||
public void increaseQuality() {
|
public void increaseQuality() {
|
||||||
this.quality = this.quality + 1;
|
this.quality = this.quality + 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
public void increaseBackstageQuality() {
|
|
||||||
if (this.sellIn < 11 && this.quality < 50) {
|
|
||||||
this.increaseQuality();
|
|
||||||
}
|
|
||||||
|
|
||||||
if (this.sellIn < 6 && this.quality < 50) {
|
|
||||||
this.increaseQuality();
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|||||||
@ -7,10 +7,4 @@ public class Sulfuras extends Item {
|
|||||||
public Sulfuras(int sellIn, int quality) {
|
public Sulfuras(int sellIn, int quality) {
|
||||||
super(SULFURAS, sellIn, quality);
|
super(SULFURAS, sellIn, quality);
|
||||||
}
|
}
|
||||||
|
|
||||||
/*@Override
|
|
||||||
public void increaseQuality() {
|
|
||||||
|
|
||||||
super.increaseQuality();
|
|
||||||
}*/
|
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user