Inheritance notebook fixes

Author: yammesicka

week8/1_Inheritance.ipynb

@@ -147,9 +147,6 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "import random\n",
-    "\n",
-    "\n",
     "class Song:\n",
     "    \"\"\"Represent a Song in our lyrics site.\n",
     "\n",
@@ -159,7 +156,7 @@
     "        The name of the song.\n",
     "    lyrics : str\n",
     "        The lyrics of the song.\n",
-    "    _artists : list of str or str, optional\n",
+    "    artists : list of str or str, optional\n",
     "        Can be either a list, or a string separated by commas.\n",
     "\n",
     "    Attributes\n",
@@ -1119,7 +1116,7 @@
     "<p style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
     "    כיוון שהטריק הזה נפוץ יחסית, פייתון נותנת בידינו דרך נוחה להתייחס למחלקת־העל, כשהפרמטר הראשון שנעביר לפעולה שבה הוא <var>self</var>.<br>\n",
     "    במקום לכתוב בכל פעם את הביטוי המתיש <code dir=\"ltr\">Song.__init__(self)</code>, נוכל לכתוב <code dir=\"ltr\">super().__init__()</code>.<br>\n",
-    "    הפונקציה <var>super</var> מאפשרת לנו להתייחס להעברת המופע שיצרנו במחלקת־הבת, לתוך פעולה הנמצאת מחלקת־העל.\n",
+    "    הפונקציה <var>super</var> מאפשרת לנו להעביר את המופע שיצרנו בתת־המחלקה, לתוך פעולה הנמצאת במחלקת־העל.\n",
     "</p>"
    ]
   },
@@ -1174,7 +1171,7 @@
    "metadata": {},
    "source": [
     "<p style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
-    "    כידוע לכם בכפר הדרדרסים יש הרבה דרדרסים \"רגילים\", אבל יש גם כמה דרדסים מפורסמים, כמו דרדסבא, דרדסית ודרדשף.<br>\n",
+    "    כידוע לכם בכפר הדרדסים יש הרבה דרדסים \"רגילים\", אבל יש גם כמה דרדסים מפורסמים, כמו דרדסבא, דרדסית ודרדשף.<br>\n",
     "</p>"
    ]
   },
@@ -1554,7 +1551,7 @@
    "outputs": [],
    "source": [
     "print(Rectangle(5, 6))\n",
-    "print(Square(3)"
+    "print(Square(3))"
    ]
   },
   {
@@ -1631,7 +1628,7 @@
    "metadata": {},
    "source": [
     "<p style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
-    "    לאחר עריכת רוחב הריבוע ציפינו ששטח הריבוע יהיה 9, אך ארכיטקטורת הקוד שלנו כשלה בטיפול במקרה הזה.<br>\n",
+    "    לאחר עריכת רוחב הריבוע ציפינו ששטח הריבוע יהיה 25, אך ארכיטקטורת הקוד שלנו כשלה בטיפול במקרה הזה.<br>\n",
     "    הקוד שלנו שינה רק את הרוחב של הריבוע ולא את אורכו, בזמן שבריבוע דבר כזה לא אמור להתאפשר.<br>\n",
     "    בשורה התחתונה – הקוד שלנו נשבר כיוון שלא עקבנו אחרי עקרון ההחלפה.\n",
     "</p>"
@@ -2196,6 +2193,9 @@
    "metadata": {},
    "outputs": [],
    "source": [
+    "import datetime\n",
+    "\n",
+    "\n",
     "class Order:\n",
     "    DELIVERY_PRICE_IN_USD = 5\n",
     "    VAT_IN_PERCENTS = 20\n",
@@ -2212,9 +2212,14 @@
     "        base_price = sum(product.price for product in self.products)\n",
     "        return base_price + self._calculate_extra_price(base_price)\n",
     "\n",
-    "    def _calculate_extra_price(self, base_price, include_vat=True):\n",
+    "    def _calculate_extra_price(\n",
+    "            self, base_price, include_vat=True, include_delivery=True,\n",
+    "    ):\n",
     "        tax = self.VAT_IN_PERCENTS / 100\n",
-    "        return base_price * tax + self.DELIVERY_PRICE_IN_USD\n",
+    "        price = base_price * tax\n",
+    "        if include_delivery:\n",
+    "            price = price + self.DELIVERY_PRICE_IN_USD\n",
+    "        return price\n",
     "\n",
     "    def __str__(self):\n",
     "        return (\n",
@@ -2230,8 +2235,8 @@
     "class PhoneOrder(Order):\n",
     "    PHONE_CALL_TOLL_IN_USD = 1.99\n",
     "\n",
-    "    def _calculate_extra_price(self, base_price, include_vat=True):\n",
-    "        base_price = super()._calculate_extra_price(base_price, include_vat)\n",
+    "    def _calculate_extra_price(self, base_price, **kwargs):\n",
+    "        base_price = super()._calculate_extra_price(base_price, **kwargs)\n",
     "        return base_price + self.PHONE_CALL_TOLL_IN_USD\n",
     "\n",
     "\n",
@@ -2240,9 +2245,14 @@
     "\n",
     "\n",
     "class StoreOrder(Order):\n",
-    "    def __init__(self, *args, **kwargs):\n",
-    "        super().__init__(*args, **kwargs)\n",
-    "        self.delivered = True"
+    "    def __init__(self, **kwargs):\n",
+    "        super().__init__(**kwargs)\n",
+    "        self.delivered = True\n",
+    "\n",
+    "    def _calculate_extra_price(self, base_price, **kwargs):\n",
+    "        return super()._calculate_extra_price(\n",
+    "            base_price, include_delivery=False, **kwargs,\n",
+    "        )"
    ]
   },
   {
@@ -2255,15 +2265,15 @@
     "\n",
     "book1 = Product(1, \"The Fountainhead\", 27.99)\n",
     "book2 = Product(2, \"Thinking, Fast and Slow\", 19.69)\n",
-    "order = PhoneOrder(251, 666, [book1, book2])\n",
+    "order = PhoneOrder(seller_id=251, buyer_id=666, products=[book1, book2])\n",
     "print(order)\n",
     "\n",
     "print('\\n\\n')\n",
-    "order = StoreOrder(251, 666, [book1, book2])\n",
+    "order = StoreOrder(seller_id=251, buyer_id=666, products=[book1, book2])\n",
     "print(order)\n",
     "\n",
     "print('\\n\\n')\n",
-    "order = OnlineOrder(251, 666, [book1, book2])\n",
+    "order = OnlineOrder(seller_id=251, buyer_id=666, products=[book1, book2])\n",
     "print(order)"
    ]
   },