mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-13 21:52:19 +00:00
refactor: helper functions in incrementing and decrementing
This commit is contained in:
parent
671461d440
commit
d700742aa6
@ -4,13 +4,19 @@ class GildedRose(object):
|
|||||||
BACKSTAGE = 'Backstage passes to a TAFKAL80ETC concert'
|
BACKSTAGE = 'Backstage passes to a TAFKAL80ETC concert'
|
||||||
SULFURAS = 'Sulfuras, Hand of Ragnaros'
|
SULFURAS = 'Sulfuras, Hand of Ragnaros'
|
||||||
AGED_BRIE = 'Aged Brie'
|
AGED_BRIE = 'Aged Brie'
|
||||||
|
MAX_QUAL = 50
|
||||||
|
|
||||||
def __init__(self, items):
|
def __init__(self, items):
|
||||||
self.items = items
|
self.items = items
|
||||||
|
|
||||||
|
def __qual_inc(self, item, inc):
|
||||||
|
item.quality = max(50, item.quality + inc)
|
||||||
|
|
||||||
|
def __qual_dec(self, item, dec):
|
||||||
|
item.quality = min(0, item.quality - dec)
|
||||||
|
|
||||||
def update_quality(self):
|
def update_quality(self):
|
||||||
for item in self.items:
|
for item in self.items:
|
||||||
|
|
||||||
match item.name:
|
match item.name:
|
||||||
case self.AGED_BRIE:
|
case self.AGED_BRIE:
|
||||||
pass
|
pass
|
||||||
@ -19,34 +25,36 @@ class GildedRose(object):
|
|||||||
case self.BACKSTAGE:
|
case self.BACKSTAGE:
|
||||||
pass
|
pass
|
||||||
case _:
|
case _:
|
||||||
pass
|
self.__qual_dec(item, 1)
|
||||||
if item.name != "Aged Brie" and item.name != "Backstage passes to a TAFKAL80ETC concert":
|
if not item.name == self.SULFURAS:
|
||||||
if item.quality > 0:
|
|
||||||
if item.name != "Sulfuras, Hand of Ragnaros":
|
|
||||||
item.quality = item.quality - 1
|
|
||||||
else:
|
|
||||||
if item.quality < 50:
|
|
||||||
item.quality = item.quality + 1
|
|
||||||
if item.name == "Backstage passes to a TAFKAL80ETC concert":
|
|
||||||
if item.sell_in < 11:
|
|
||||||
if item.quality < 50:
|
|
||||||
item.quality = item.quality + 1
|
|
||||||
if item.sell_in < 6:
|
|
||||||
if item.quality < 50:
|
|
||||||
item.quality = item.quality + 1
|
|
||||||
if item.name != "Sulfuras, Hand of Ragnaros":
|
|
||||||
item.sell_in = item.sell_in - 1
|
item.sell_in = item.sell_in - 1
|
||||||
if item.sell_in < 0:
|
# if item.name != "Aged Brie" and item.name != "Backstage passes to a TAFKAL80ETC concert":
|
||||||
if item.name != "Aged Brie":
|
# if item.quality > 0:
|
||||||
if item.name != "Backstage passes to a TAFKAL80ETC concert":
|
# if item.name != "Sulfuras, Hand of Ragnaros":
|
||||||
if item.quality > 0:
|
# item.quality = item.quality - 1
|
||||||
if item.name != "Sulfuras, Hand of Ragnaros":
|
# else:
|
||||||
item.quality = item.quality - 1
|
# if item.quality < 50:
|
||||||
else:
|
# item.quality = item.quality + 1
|
||||||
item.quality = item.quality - item.quality
|
# if item.name == "Backstage passes to a TAFKAL80ETC concert":
|
||||||
else:
|
# if item.sell_in < 11:
|
||||||
if item.quality < 50:
|
# if item.quality < 50:
|
||||||
item.quality = item.quality + 1
|
# item.quality = item.quality + 1
|
||||||
|
# if item.sell_in < 6:
|
||||||
|
# if item.quality < 50:
|
||||||
|
# item.quality = item.quality + 1
|
||||||
|
# if item.name != "Sulfuras, Hand of Ragnaros":
|
||||||
|
# item.sell_in = item.sell_in - 1
|
||||||
|
# if item.sell_in < 0:
|
||||||
|
# if item.name != "Aged Brie":
|
||||||
|
# if item.name != "Backstage passes to a TAFKAL80ETC concert":
|
||||||
|
# if item.quality > 0:
|
||||||
|
# if item.name != "Sulfuras, Hand of Ragnaros":
|
||||||
|
# item.quality = item.quality - 1
|
||||||
|
# else:
|
||||||
|
# item.quality = item.quality - item.quality
|
||||||
|
# else:
|
||||||
|
# if item.quality < 50:
|
||||||
|
# item.quality = item.quality + 1
|
||||||
|
|
||||||
|
|
||||||
class Item:
|
class Item:
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user