מה הבעיה בקוד הבא?

avr416

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

DateTime birthday;//תאריך לידה
public override string ToString()
        {
            return base.ToString();
            string str = "Birthday:" + Birthday.ToString();
            return str;
        }

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

אשמח לעזרה!
תודה רבה מראש.

פורסם במקור בפורום CODE613 ב14/05/2015 23:39 (+03:00)

שמואל

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

DateTime birthday;//תאריך לידה
public override string ToString()
        {
            return  "Birthday:" + Birthday.ToString();
        }

בהצלחה

פורסם במקור בפורום CODE613 ב15/05/2015 00:35 (+03:00)

avr416

@שמואל

יתכן שהתכוונת לקוד כזה

DateTime birthday;//תאריך לידה
public override string ToString()
        {
            return  "Birthday:" + Birthday.ToString();
        }

בהצלחה

צודק! הבעיה הייתה הreturn..
התכוונתי לקוד כזה:

base.ToString();
           ()string str = "Birthday:" + birthday.ToString
                          return str;

הוא יורש את המתודה הנ"ל ומוסיף לה נתון.
תודה רבה!

פורסם במקור בפורום CODE613 ב15/05/2015 11:30 (+03:00)

dovid

אני חושב שהתכונת לזה.

string str = base.ToString() + "Birthday:" + birthday.ToString();
return str;

פורסם במקור בפורום CODE613 ב17/05/2015 14:32 (+03:00)

avr416

צודק!!
תודה רבה רבה!

פורסם במקור בפורום CODE613 ב17/05/2015 19:04 (+03:00)

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

return  base.ToString() + "Birthday:" + birthday.ToString();

פורסם במקור בפורום CODE613 ב18/05/2015 17:23 (+03:00)

avr416

תודה על ההארה!

פורסם במקור בפורום CODE613 ב18/05/2015 21:38 (+03:00)

softs

@Pesia

אגב, יותר יפה להכניס לשורת קוד אחת, וכדאי להתרגל לזה, זה נראה יותר מקצועי:
return  base.ToString() + "Birthday:" + birthday.ToString();

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

פורסם במקור בפורום CODE613 ב19/05/2015 18:56 (+03:00)

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

פורסם במקור בפורום CODE613 ב19/05/2015 19:22 (+03:00)

dovid

@Pesia

איזה התקפה...

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

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

פורסם במקור בפורום CODE613 ב19/05/2015 19:32 (+03:00)

softs

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

@דוד ל.ט.

@Pesia
איזה התקפה...

לא ראיתי התקפה, הוא רק אמר את דעתו.

אתה יכול לייצג אותי מול הלקוחות? אתה עושה עבודה מצוינת . . .
@דוד ל.ט.

והוא מתכנת מקצועי לפי מה שנראה בפורום.

אני עובד על זה, אולי בקרוב . . .
@דוד ל.ט.

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

I couldn't agree more

פורסם במקור בפורום CODE613 ב20/05/2015 01:59 (+03:00)