תיקונים למחברות – שבוע 8

תגיות:

פעמיים כתוב דרדרסים במקום דרדסים :sweat_smile:

3 לייקים

זה דרדסים שהם קוץ. (תודה, מתקן) :stuck_out_tongue:
אני חושב שזה השלב שהעורכת הלשונית מדלגת על דברים כאלו בקטע של “שוב הוא והשטויות שלו”

6 לייקים

אפשר פשוט לעדכן את הקובץ של המחברת במקום לכתוב הערות כאן? :slight_smile:
ככה הקובץ מקור תקין

לא כרגע :slightly_smiling_face:
זה יחסית הרבה עבודה עבורי לעדכן, לארוז מחדש ולהעלות לגיט.
לעשות את זה עבור כל תיקון זה לא ניצול טוב של הזמן שלי נכון לעכשיו.
אעשה את זה כשתצא המחברת הבאה.

גם בקישור ל-3 דוגמאות נוספות לססטינה לכל אחת חוקיות משלה ב -3 השורות האחרונות.
האם ניתן לקחת את אחת מהדוגמאות בהתאם לחוקיות שבססטינה שנבחרה מהשלושה?

אני רואה שזה מסתבך אז אפשט את זה – אפשר להתעלם מהחוקיות לגבי הבית האחרון בתרגיל.

2 לייקים

חולק תפקיד לכל אחד מהשחקנים: אזרח, שוטר או רוצח
בגרסת המשחק שלנו, יהיה רק איש מאפיה אחד ואיש משטרה אחד.

מאמין שאיש המאפייה והרוצח הם אותו אחד, אבל אין אחידות בשמות. בשאר התרגיל השימוש הוא מאפיה.

לייק 1

[main.Instrumental, main.Song, object]

נשים לב שבראש השרשרת עומדת המחלקה object.

האם לא הכוונה בתחתית? הרי היא נמצאת ב list[2]

Instrumental יורש מ Song שיורש מ object
אז הוא אכן בראש השרשת. ויזואלית זה פשוט לא נראה ככה.

לייק 1

יש סיבה שבתיעוד של Song הפרמטר artists הוא עם קו תחתון לפניו כשהוא ללא קו תחתון בפרמטר?

הבנתי שבגלל זה ששמים קו תחתון כשזה self._artists זה יהיה בתיעוד של זה כattribute אבל בתיעוד של זה כargs זה גם עם קו תחתון למרות שאין קו תחתון במקום של artists כפרמטר (בסוגריים ליד שם המתודה).
על זה שאלתי. האם על זה ענית או שחשבת ששאלתי משהו אחר?

בדוגמה האחרונה במחברת הראשונה:
image
הייתי מוסיף פונקציה _calculate_extra_price שמחסירה מהמחיר את דמי המשלוח, או לחילופן משנה במחלקת האם את החישוב, ומוסיף לשאר המחלקות את דמי המשלוח.
לא נראה לי שמי שמגיע לחנות רוצה לשלם גם דמי משלוח

נאלצתי למחוק את כל השאלות שלא היו קשורות לאשכול.
אני מבקש לשמור על האשכול מאורגן, ולפתוח אשכול חדש אם יש לכם שאלה שקשורה למחברת או לתיקונים פה.
תודה

2 לייקים

בסופו של דבר זה עניין של המשגה ויזואלית :slight_smile:
חשבתי האם לשנות אחרי שהעלית, אבל אני לא מוצא ביטוי טוב יותר. אני חושב שיותר נוח להסתכל על מחלקת־העל (על == למעלה) כראש השרשרת.

לייק 1

לא :slight_smile: דאגתי לתקן, תודה!

הוספתי, תודה!

לייק 1

מסכימה איתך.
הכל תלוי בהקשר של המילה: השרשרת, האם היא בהקשר לMRO או לסדר ירושה.

לייק 1

לא עברתי על כל השרשור ויתכן שמישהו ציין זאת אבל הורדתי את המחברת המתוקנת ולעניות דעתי יש שם שגיאה בשיכתוב של str של class Instrumental. נכתב שם:
class Instrumental(Song):
def init(self, name, artists=None):
Song.init(self, name=name, lyrics="", artists=artists)

def __str__(self):
    Song.__str__(self)
    artists = ', '.join(self.get_artists())
    title = f'"{self.name}" / {artists}'
    separator = "-" * len(title)
    return f"{title}\n{separator}\nSeen: {self._views} time(s)."

אבל כל הרעיון של השימוש ב- str של Song הוא לא לכתוב פעם נוספת את 3 השורות שלפני ה-return

אינה שגיאה. המשתנים הללו לא מוגדרים בפעולה במחלקת־העל.
אם לא הבנת איך זה עובד כדאי לפתוח אשכול חדש (או לחפש את אלו הקיימים) ולהדיין על איך זה עובד :slight_smile:

אם הקובץ הבינארי הוא מסוג תמונה, צרו עבורו הפעולה get_size

הצ"ל: צרו עבורו את הפעולה.
(לא קריטי :see_no_evil:)

לייק 1

כשאתה שובב ולא מקפיד לבצע את הנחיות העורכת הלשונית כלשונן
image

תוקן, תודה!

(טריוויה מעניינת: דוד בן גוריון לא חשב שהמילה “את” הכרחית, יש הרבה קריאות נחמדות בנושא)

6 לייקים