Fixed attributes attached to class instead of instance.

Author: Arjan Egges

3 - strategy pattern/strategy-after-fn.py

@@ -10,9 +10,6 @@ def generate_id(length=8):
 
 
 class SupportTicket:
-    id: str
-    customer: str
-    issue: str
 
     def __init__(self, customer, issue):
         self.id = generate_id()
@@ -42,7 +39,8 @@ def blackHoleOrdering(list: List[SupportTicket]) -> List[SupportTicket]:
 
 class CustomerSupport:
 
-    tickets: List[SupportTicket] = []
+    def __init__(self):
+        self.tickets = []
 
     def create_ticket(self, customer, issue):
         self.tickets.append(SupportTicket(customer, issue))

3 - strategy pattern/strategy-after.py

@@ -10,9 +10,6 @@ def generate_id(length=8):
 
 
 class SupportTicket:
-    id: str
-    customer: str
-    issue: str
 
     def __init__(self, customer, issue):
         self.id = generate_id()
@@ -52,10 +49,8 @@ def create_ordering(self, list: List[SupportTicket]) -> List[SupportTicket]:
 
 class CustomerSupport:
 
-    tickets: List[SupportTicket] = []
-    processing_strategy: TicketOrderingStrategy = None
-
     def __init__(self, processing_strategy: TicketOrderingStrategy):
+        self.tickets = []
         self.processing_strategy = processing_strategy
 
     def create_ticket(self, customer, issue):

3 - strategy pattern/strategy-before.py

@@ -9,9 +9,6 @@ def generate_id(length=8):
 
 
 class SupportTicket:
-    id: str
-    customer: str
-    issue: str
 
     def __init__(self, customer, issue):
         self.id = generate_id()
@@ -21,9 +18,6 @@ def __init__(self, customer, issue):
 
 class CustomerSupport:
 
-    tickets: List[SupportTicket]
-    processing_strategy: str
-
     def __init__(self, processing_strategy: str = "fifo"):
         self.tickets = []
         self.processing_strategy = processing_strategy