mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-14 14:11:20 +00:00
To simplify code, remove old business logic
This commit is contained in:
parent
df68a125cb
commit
dcde3a22a2
@ -32,27 +32,15 @@ final class GildedRose
|
||||
continue;
|
||||
}
|
||||
|
||||
if ($item->name !== 'Aged Brie') {
|
||||
if ($item->quality > 0) {
|
||||
$item->quality = $item->quality - 1;
|
||||
}
|
||||
} else {
|
||||
if ($item->quality < 50) {
|
||||
$item->quality = $item->quality + 1;
|
||||
}
|
||||
if ($item->quality > 0) {
|
||||
$item->quality = $item->quality - 1;
|
||||
}
|
||||
|
||||
$item->sellIn = $item->sellIn - 1;
|
||||
|
||||
if ($item->sellIn < 0) {
|
||||
if ($item->name !== 'Aged Brie') {
|
||||
if ($item->quality > 0) {
|
||||
$item->quality = $item->quality - 1;
|
||||
}
|
||||
} else {
|
||||
if ($item->quality < 50) {
|
||||
$item->quality = $item->quality + 1;
|
||||
}
|
||||
if ($item->quality > 0) {
|
||||
$item->quality = $item->quality - 1;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Loading…
Reference in New Issue
Block a user