mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2025-12-12 12:22:12 +00:00
added encoding, fixed som PEP-8 issues. Thanks to apieum for these changes.
This commit is contained in:
parent
ca6f1a5b55
commit
5fb9f52b48
@ -1,41 +1,43 @@
|
|||||||
|
# -*- coding: utf-8 -*-
|
||||||
|
|
||||||
|
|
||||||
def update_quality(items):
|
def update_quality(items):
|
||||||
for item in items:
|
for item in items:
|
||||||
if item.name != "Aged Brie" and item.name != "Backstage passes to a TAFKAL80ETC concert":
|
if item.name != "Aged Brie" and item.name != "Backstage passes to a TAFKAL80ETC concert":
|
||||||
if item.quality > 0:
|
if item.quality > 0:
|
||||||
if item.name != "Sulfuras, Hand of Ragnaros":
|
if item.name != "Sulfuras, Hand of Ragnaros":
|
||||||
item.quality = item.quality - 1;
|
item.quality = item.quality - 1
|
||||||
else:
|
else:
|
||||||
if item.quality < 50:
|
if item.quality < 50:
|
||||||
item.quality = item.quality + 1;
|
item.quality = item.quality + 1
|
||||||
if item.name == "Backstage passes to a TAFKAL80ETC concert":
|
if item.name == "Backstage passes to a TAFKAL80ETC concert":
|
||||||
if item.sell_in < 11:
|
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.sell_in < 6:
|
||||||
if item.quality < 50:
|
if item.quality < 50:
|
||||||
item.quality = item.quality + 1;
|
item.quality = item.quality + 1
|
||||||
if item.name != "Sulfuras, Hand of Ragnaros":
|
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.sell_in < 0:
|
||||||
if item.name != "Aged Brie":
|
if item.name != "Aged Brie":
|
||||||
if item.name != "Backstage passes to a TAFKAL80ETC concert":
|
if item.name != "Backstage passes to a TAFKAL80ETC concert":
|
||||||
if item.quality > 0:
|
if item.quality > 0:
|
||||||
if item.name != "Sulfuras, Hand of Ragnaros":
|
if item.name != "Sulfuras, Hand of Ragnaros":
|
||||||
item.quality = item.quality - 1;
|
item.quality = item.quality - 1
|
||||||
else:
|
else:
|
||||||
item.quality = item.quality - item.quality;
|
item.quality = item.quality - item.quality
|
||||||
else:
|
else:
|
||||||
if item.quality < 50:
|
if item.quality < 50:
|
||||||
item.quality = item.quality + 1;
|
item.quality = item.quality + 1
|
||||||
return items
|
return items
|
||||||
|
|
||||||
|
|
||||||
class Item:
|
class Item:
|
||||||
def __init__(self, name, sell_in, quality):
|
def __init__(self, name, sell_in, quality):
|
||||||
self.name = name
|
self.name = name
|
||||||
self.sell_in = sell_in
|
self.sell_in = sell_in
|
||||||
self.quality = quality
|
self.quality = quality
|
||||||
|
|
||||||
def __repr__(self):
|
def __repr__(self):
|
||||||
return "%s, %s, %s" % (self.name, self.sell_in, self.quality)
|
return "%s, %s, %s" % (self.name, self.sell_in, self.quality)
|
||||||
|
|
||||||
|
|||||||
@ -1,7 +1,8 @@
|
|||||||
|
# -*- coding: utf-8 -*-
|
||||||
import unittest
|
import unittest
|
||||||
|
|
||||||
from gilded_rose import Item, update_quality
|
from gilded_rose import Item, update_quality
|
||||||
|
|
||||||
|
|
||||||
class GildedRoseTest(unittest.TestCase):
|
class GildedRoseTest(unittest.TestCase):
|
||||||
def test_foo(self):
|
def test_foo(self):
|
||||||
items = [Item("foo", 0, 0)]
|
items = [Item("foo", 0, 0)]
|
||||||
@ -10,4 +11,4 @@ class GildedRoseTest(unittest.TestCase):
|
|||||||
|
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
unittest.main()
|
unittest.main()
|
||||||
|
|||||||
@ -1,5 +1,5 @@
|
|||||||
|
# -*- coding: utf-8 -*-
|
||||||
from __future__ import print_function
|
from __future__ import print_function
|
||||||
|
|
||||||
from gilded_rose import *
|
from gilded_rose import *
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
@ -13,9 +13,9 @@ if __name__ == "__main__":
|
|||||||
Item(name="Backstage passes to a TAFKAL80ETC concert", sell_in=15, quality=20),
|
Item(name="Backstage passes to a TAFKAL80ETC concert", sell_in=15, quality=20),
|
||||||
Item(name="Backstage passes to a TAFKAL80ETC concert", sell_in=10, quality=49),
|
Item(name="Backstage passes to a TAFKAL80ETC concert", sell_in=10, quality=49),
|
||||||
Item(name="Backstage passes to a TAFKAL80ETC concert", sell_in=5, quality=49),
|
Item(name="Backstage passes to a TAFKAL80ETC concert", sell_in=5, quality=49),
|
||||||
Item(name="Conjured Mana Cake", sell_in=3, quality=6), # <-- :O
|
Item(name="Conjured Mana Cake", sell_in=3, quality=6), # <-- :O
|
||||||
]
|
]
|
||||||
|
|
||||||
days = 2
|
days = 2
|
||||||
import sys
|
import sys
|
||||||
if len(sys.argv) > 1:
|
if len(sys.argv) > 1:
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user