בעיה בהידור תוכנה בגאווה

[Dror Arasheven]

http://pastebin.com/mRY2dDyf

המרצה נתן תרגיל לכתוב שיטה סטטית isPermutation להשוואת 2 מחרוזות . (מותר העמסה) השתמשתי בתוכנה שמצאתי לסידור המחרוזת כדי שהוכל להשתמש רק בלולאה 1 ולא מקוננת

המהדר מודיע לי שהוא אינו מזהה את הערך בתוך הסוגרים בפונקציה mergeSort(a) משהוא יכול יכול להגיד לי מה הבעיה ואיך לתקן את זה?

[שניצל]

בדיוק מה שהוא אומר - אתה מנסה להעביר טיפוס []int לתוך פונקציה שמצפה לטיפוס []Comparable. אי אפשר להמיר בין השניים - int אינו Comparable.

אתה יכול לעבוד עם Integer במקום int, כי Integer הוא כן Comparable.

[Dror Arasheven]

יש לי כרגע הודעה הזאת ...

נ.ב . אפשר להתיחס ל Integer כאל int?

[שניצל]

Integer היא גרסה "object oriented" של int. אפשר להמיר באופן חופשי בין השניים (אלא אם אתה מנסה להמיר בין []int ל-[]Integer ולהיפך, ואז אי אפשר).

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

[Dror Arasheven]

השינוי שעשיתי הוא זה

    public static int isPermutation (Integer  a[], Integer  b[])

בשיטה הסטטית שמופיעה השגיאה בהודעה הראשית . והשגיאה היא באותה שורה שצויינה ...

[שניצל]

מה זה "אותה שורה שצויינה"? לא ציינת שום שורה.

בכל מקרה, כמו שאמרתי - השגיאה הזו אומרת שניסית "להשתמש" ב-void. תסתכל על השורה שעליה מצביע הקומפיילר ותנסה להבין מה הבעיה איתה.

אגב, למה הפונקציה מחזירה int ולא boolean?

[Dror Arasheven]

        if (mergeSort(a)==mergeSort(b))

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

boolean

....

[שניצל]

מה הפונקציה mergeSort מחזירה?

[Dror Arasheven]

היא אמורה לסדר את המערך בסדר עולה ......

[שניצל]

לא שאלתי מה היא עושה, שאלתי מה היא מחזירה​.

[Dror Arasheven]

היא אמורה להחזיר את אותו מערך בסדר עולה .... כלומר המס' הכי קטן הוא בתא הראשון וכן הלאה ....

[שניצל]

שוב, זה מה שהפונקציה עושה. השאלה שלי היא מה הפונקציה מחזירה. ה-return שלה.

[Dror Arasheven]

מחרוזת באותו גודל ...http://pastebin.com/hztmifxn זה הקוד המקורי שאני רוצה לשנות .....

[שניצל]

מה הקשר למחרוזת כאן?

טיפוס ההחזרה של הפונקציה שלך הוא void. זה אומר שהיא לא מחזירה כלום. זה אומר שאתה לא יכול לקחת את התוצאה שלה ולנסות להשתמש בה ולהשוות אותה למשהו, כי אין לפונקציה "תוצאה". זה מה שהקומפיילר צועק עליו. אתה מנסה לעשות (mergeSort(a)==mergeSort(b, אבל אין לביטוי (mergeSort(a ערך שהוא יכול להשוות אותו.

[Dror Arasheven]

פתרתי את זה בכך שהוצאתי את הפונ' מהתנאי והכנסתי לולאה . http://pastebin.com/stjhjgeT אני צריך לבדוק שזה עובד . למישהו יש דוגמא אין אני אמור לגשת לזה ?