Address PR feedback

Author: zohrehKazemianpour

exercises-sprint5/enums.py

@@ -73,26 +73,28 @@ def find_possible_laptops(laptops: List[Laptop], person: Person) -> List[Laptop]
 # Get user input as strings first (raw input)
 name = input("Enter your name: ")
 age_str = input("Enter your age: ")
-os_str = input("Enter preferred OS: ")
+print("Available operating systems: Ubuntu, macOS, Arch Linux")
+os_str = input("Enter your preferred OS (Ubuntu, macOS, or Arch Linux): ")
 
 # --------------------------------------------------------------------
 # INPUT VALIDATION AND CONVERSION
 # --------------------------------------------------------------------
 
 # Convert age from string to integer with error handling
 # Output to stderr as per requirements, exit with non-zero code
-try:
-    age = int(age_str)
-except ValueError:
-         
-        print(f"Error: {age_str} is not a valid age", file=sys.stderr)
-        sys.exit(1)
+while True:
+    age_str = input("Enter your age: ")
+    try:
+        age = int(age_str)
+        break
+    except ValueError:
+        print(f"Error: {age_str} is not a valid age. Please enter a number.")
 
 # Convert OS string to enum with error handling
 try:
     os = OperatingSystem(os_str) 
 except ValueError:
-    print(f"Error: '{os_str}' is not a valid OS. Choose: Ubuntu, macOS, Arch Linux", file=sys.stderr)
+    print(f"Error: '{os_str}' is not a valid OS. Valid options are: Ubuntu, macOS, Arch Linux", file=sys.stderr)
     sys.exit(1)
 
 

exercises-sprint5/generics.py

@@ -16,10 +16,12 @@ class Person:
 
 imran = Person(name="Imran",age=50, children=[fatma, aisha])
 
-def print_family_tree(person: Person) -> None:
-    print(person.name)
+def print_family_tree(person: Person, level: int = 0) -> None:
+    indent = "  " * level
+    print(f"{indent}{person.name} ({person.age})")
     for child in person.children:
-        print(f"- {child.name} ({child.age})")
+        print_family_tree(child, level + 1)
+
 
 print_family_tree(imran)
 

exercises-sprint5/person2.py

@@ -10,8 +10,7 @@ class Person:
     name: str
     date_of_birth: date 
     preferred_operating_system: str
-
-
+    
     def is_adult(self) -> bool:
         today_date = date.today().year
         birth_year = self.date_of_birth.year