added abstract factory pattern to improve the maintainability of the code

python/components/gilded_rose.py

@@ -1,12 +1,17 @@
 from abc import abstractmethod, ABC
-from ..models import Items
 
 
+class Item:
+    def __init__(self, name, sell_in, quality):
+        self.name = name
+        self.sell_in = sell_in
+        self.quality = quality
+
 class GildedRose(ABC):
-    def __init__(self, item: str, sell_in: int, quality: int):
+    def __init__(self, items: Item):
-        self.item = item
+        self.name = items.name
-        self.quality = quality
+        self.quality = items.quality
-        self.sell_in = sell_in
+        self.sell_in = items.sell_in
 
     @abstractmethod
     def update_quality(self):

python/src/aged_brie.py

@@ -0,0 +1,16 @@
+from python.components.gilded_rose import GildedRose, Item
+
+
+class AgedBrieLogic(GildedRose):
+
+    def __init__(self, name, quality, sell_in):
+        super().__init__(Item(name, quality, sell_in))
+
+    def update_quality(self):
+        if self.quality < 50:
+            self.quality += 1
+        self.sell_in -= 1
+        if self.sell_in < 0 and self.quality < 50:
+            self.quality += 1
+
+        return self.quality, self.sell_in

python/src/backstage_passes.py

@@ -0,0 +1,19 @@
+from python.components.gilded_rose import GildedRose, Item
+
+
+class BackstagePassesLogic(GildedRose):
+    def __init__(self, name, quality, sell_in):
+        super().__init__(Item(name, quality, sell_in))
+
+    def update_quality(self):
+        if self.quality < 50:
+            self.quality += 1
+        if self.sell_in < 11 and self.quality < 50:
+            self.quality += 1
+        if self.sell_in < 6 and self.quality < 50:
+            self.quality += 1
+        self.sell_in -= 1
+        if self.sell_in < 0:
+            self.quality = 0
+
+        return self.quality, self.sell_in

python/src/conjured.py

@@ -0,0 +1,17 @@
+from python.components.gilded_rose import GildedRose, Item
+
+
+
+
+class ConjuredItem(GildedRose):
+
+    def __init__(self, name, quality, sell_in):
+        super().__init__(Item(name, quality, sell_in))
+
+    def update_quality(self):
+        if self.quality > 0:
+            self.quality -= 2
+        if self.sell_in <= 0 and self.quality > 0:
+            self.quality -= 2
+        self.sell_in -= 1
+        return self.quality, self.sell_in

python/src/main.py

@@ -0,0 +1,18 @@
+from python.src.aged_brie import AgedBrieLogic
+from python.src.backstage_passes import BackstagePassesLogic
+from python.src.normal_item_logic import NormalItemLogic
+from python.src.sulfuras import SulfurasLogic
+
+
+class UpdateItemLogic:
+    @staticmethod
+    def update_items(items):
+        for item in items:
+            if item.name == "Aged Brie":
+                AgedBrieLogic.update_quality(item)
+            elif item.name == "Backstage passes to a TAFKAL80ETC concert":
+                BackstagePassesLogic.update_quality(item)
+            elif item.name == "Sulfuras, Hand of Ragnaros":
+                SulfurasLogic.update_quality(item)
+            else:
+                NormalItemLogic.update_quality(item)

python/src/normal_item_logic.py

@@ -0,0 +1,15 @@
+from python.components.gilded_rose import GildedRose, Item
+
+
+class NormalItemLogic(GildedRose):
+    def __init__(self, name, quality, sell_in):
+        super().__init__(Item(name, quality, sell_in))
+
+    def update_quality(self):
+        if self.quality > 0:
+            self.quality -= 1
+        self.sell_in -= 1
+        if self.sell_in < 0 and self.quality > 0:
+            self.quality -= 1
+
+        return self.quality, self.sell_in

python/src/sulfuras.py

@@ -0,0 +1,10 @@
+from python.components.gilded_rose import GildedRose, Item
+
+
+class SulfurasLogic(GildedRose):
+    def __init__(self, name, quality, sell_in):
+        super().__init__(Item(name, quality, sell_in))
+
+    def update_quality(self):
+        # "Sulfuras" does not change in quality or sell_in
+        return self.quality, self.sell_in