שלום לכולם... אני צריך לעשות תוכנית הבודקת אם המערך הוא ריבוע קסם ב-c#.

עשיתי את התוכנית והיא פועלת אבל רציתי את חוות דעתכם אם התוכנית יעילה עד ככל שאפשר או שישנה אפשות ליעל אותה עוד

הינה הקוד להורדה בנופאד כדי שהיה יותר נוח:

http://uploading.com/files/E1DQ9W35/New Text Document.txt.html

<a href=http://uploading.com/files/E1DQ9W35/New Text Document.txt.html>Download from Uploading.com</a>

תודה לכל העוזרים

תקן את הלינק שלך (עטוף אותו בתג url).

הקוד נראה סבבה, אמור לעבוד. השמות של המשתנים שלך יכולים לבלבל קצת (לפעמים amuda מייצג את מספר העמודה ולפעמים את מספר השורה, וכנ"ל לגבי shura). חוץ מזה, אתה עושה הרבה השוואות מיותרות - אין צורך להשוות כל שורה וכל עמודה גם לאלכסונים, וגם לעמודה ולשורה הראשונות. מספיק רק להשוות לאחד מכל הארבעה האלה, ולוודא בנפרד שהארבעה שווים.

קודם כל תודה על התגובה

תיקנתי את מה שרשמת לי אבל זה נראה לי קצת הרבה משתנים... ישנה דרך לצמצם?

הינה התוכנית שלי המתוקנת

http://rapidshare.com/files/195471693/New_Text_Document.txt.html.url

נראה סבבה. דרך אגב, קטע כזה קצר אין צורך להעלות כקובץ, אתה יכול פשוט להדביק כאן בפורום (רק תשתמש בטג קוד - כפתור # למעלה).

אין ממש דרך לחסוך משתנים משמעותית, אלא אם אתה רוצה לחשב כל סכום בלולאה נפרדת (וזה כבר מיותר).

אפשר לחסוך שני משתנים (אתה יכול להשתמש באותו משתנה עבור SumMainDiagonal ו-SumRow2, לדוגמה) אבל זה מיותר לחלוטין, ורק הופך את הקוד לפחות קריא. תכלס אין היום שום סיבה לנסות לחסוך משתנים בקוד שלך, המחשבים שלנו יודעים היטב להתמודד עם זה.