- This topic has 6 תגובות, 4 משתתפים, and was last updated לפני 10 שנים, 7 חודשים by
.
-
תגובות
-
טוב, אז הנה המנוע החדש יחד עם השינויים שלי ושל עידן:
https://github.com/Yovaayova/corewars8086/releases/tag/v4.0
הרשימה המלאה של השינויים נמצאת בקובץ
Changes.txt
בתיקייה המזופזפת. שימו לב שהוספנו גם את כל שורדי העבר, מקודגורו 1 עד קודגורו 9, מטעמי נוחיות.
אם יש לכם שאלות, בקשות, אמירות או כל דבר אחר אתם מוזמנים
יואבקודם כל, אני מסיר בפניכם את הכובע. שיפרתם את המנוע ונתתם את השיפורים במתנה לקהילה, וזה שם אתכם במקום יותר מכובד ממני לא משנה מה אגיד.
יש כמה דברים שראיתי כשעברתי על הקוד שלכם שאולי יעשו אותו קריא יותר ונוח יותר לעבודה, בעתיד, אז אם בא לכם אתם מוזמנים להשתמש בהצעות שלי ואם לא – תגידו לי לשתוק, כי אחרי הכל אתם עושים עבודה ואני רק מבקר אותה:
* לא לגמרי הבנתי איך או למה עובד כל העניין עם הרנדום בבינום. ממליץ לתעד יותר טוב את הרציונל מאחורי זה בשביל אנשים כמוני. לחלופין, אם הבנתי נכון הכוונה בזה היא לערבב, נכון – אז למה לא פשוט להפריד לגמרי את הערבוב, בתור משהו שקורה רק רגע לפני הרצה של תחרות (להוסיף ערבוב של הרשימה ממש בתוך הפונקציה runWar)? אפשר להשתמש בערבוב רגיל של java, שנקרא Collection.shuffle אם אני זוכר נכון.
* קראתי פעם שכל פעם שמישהו כותב פונקציה עם פרמטר בוליאני מה שהוא מתכוון זה שתי פונקציות שאחת עוטפת את השנייה. נראה לי שבמקרה הזה זה נכון – יכול להיות שאתם רוצים פונקציה אחת
string runCompetition(
ושנייה
void runAndSaveCompetition(
שמריצה את הראשונה ואז שומרת את התוצאות?בכל מקרה, הרבה כבוד! תודה!
(כמובן, באותו נושא, מה דעתכם שtillEnd יהיה פרמטר של הConstructor ולא פרמטר של הפונקציות שהן מעבירות עוד ועוד פנימה?)
אכן, עצות טובות
הנה המנוע עם השינויים המינוריים:
https://github.com/Yovaayova/corewars8086/releases/tag/v4.0.1וכמובן שהקישור הבא ייתן פשוט את הגרסה הכי חדשה
כשאני מנסה לפתוח את קובץ הזיפ של המנוע החדש, הוא נותן לי הודעה:
cannot open file: it does not appear to be a valid archive/
if you downloaded this file, try downloading the file again.אוז פותח לי קובץ זיפ ריק…
מישו יודע למה זה קורה? ואיך אני מתקן את זה?
תודה רבה!!
- יש להתחבר למערכת על מנת להגיב.