שלא נכונה, עד לשימוש בשדות פרייבט כשהם לא אמורים להיות.אני אישית ממליץ לך להבא לממש כל חלק בנפרד (ולדאבג אותו עם main) לפני שאתה ממשיך לצעד הבא.
בכל מקרה, הינה משהו מתקמפל.
יש הרבה הרבה תיקונים אז עדיף שתעשה diff ותראה מה שיניתי מאשר אפרט.
אבל בגדול:
1. קח בחשבון שלמשל Update לא ממומש (אתה חייב לתת הצהרה אם אתה משתמש במתודה מתישהו, גם אם היא ריקה).
2. תשאיר את ה-namespace לסוף, מה שגם הוא דיי מיותר כי כולם (אולי חוץ מ-modifstream) משתפים את ה-namespace הזה, אז הוא דיי מיותר.
3. שים לב מתי שדה/מתודה יכולה להיות private/protected ומתי לא.
4. אין סיבה שמתודה (לא פונקציה!) תהיה סטטית אם לא השתמש בקישור דינמי באף שלב (virtual, למרות שזה לא מזיק, אבל שתדע).
5. שים לב שהבעיה הראשית שהציקה לך זה בגלל ש-locations מסוג Array<Location>* וגם מימשת ל-Array את אופרטור הסוגריים [], לכן אתה צריך להשתמש ב-
(*locations)[theNumber] התהפך קצת בפורום אבל מאמין שתבין
|
ולא איך שאתה ניגשת (לא לשכוח שזה פויינטר)
בכל מקרה, זה מה שאני זוכר בין המשחקים.
יכול להיות ששיניתי על הדרך דברים מיותרים, אבל אלה הכי בולטים.
בברכה,
עידן