ABA


"מה דעתכם על כתיבת הערות בקוד?"
גירסת הדפסה        
קבוצות דיון פיתוח, תיכנות ובניית אתרים נושא #10960 מנהל    סגן המנהל    מפקח   Winner    צל"ש   מומחה  
אשכול מספר 10960
יוחאי
חבר מתאריך 30.12.15
163 הודעות
   15:17   20.10.12   
אל הפורום  
  מה דעתכם על כתיבת הערות בקוד?  
 
   בעד/נגד + נימוקים!


                                שתף        
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד

  האשכול     מחבר     תאריך כתיבה     מספר  
  בעד בונבנציה הכי הכי מפורטת, רמה ממש מפרטת afroman50 20.10.12 16:43 1
  בעיקרון נגד D-KinG 20.10.12 17:11 2
     קריא הוא לא יהיה ב-100% Nesher  20.10.12 17:24 3
  ברור.. ועדיף כמה שיותר... אבל כמובן תלוי איזה קוד.... TheKid 20.10.12 17:33 4
  רק במקרי קצה..שעושים דברים לא שגרתיים או מסובכים.. VeNom  20.10.12 18:11 5
  כן, בהחלט Deuce  20.10.12 19:27 6
  בעד, כמובן. Dr_69 20.10.12 19:34 7
  צריך, אבל לא כמו ראש בקיר. Zippo  20.10.12 19:35 8
     חחח אדיר Nesher  20.10.12 20:19 10
     ענק, ישבתי היום במהלך שיעור 3 שעות C# וקראתי את זה sza  23.10.12 01:20 14
  אני בגישה היותר זריזה - לכתוב קוד שמסביר את עצמו... Zvikadori 20.10.12 20:09 9
     יש מן כלל כזה שאומר: Zippo  20.10.12 23:47 11
         חח תאכלס.. asco88  21.10.12 00:07 12
  טוב אני גם אגיד את דעתי!:) יוחאי 21.10.12 00:21 13

       
afroman50
חבר מתאריך 16.8.04
12555 הודעות, 1 פידבק
   16:43   20.10.12   
אל הפורום  
  1. בעד בונבנציה הכי הכי מפורטת, רמה ממש מפרטת  
בתגובה להודעה מספר 0
 


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
D-KinG
חבר מתאריך 8.6.02
3490 הודעות
   17:11   20.10.12   
אל הפורום  
  2. בעיקרון נגד  
בתגובה להודעה מספר 0
 
   אלא אם כן צריך לפרט קצת ברמת באלגוריתם
פשוט כי הקוד אמור להיות קריא בלי הערות, סתם הערות מיותרות עושות כאב ראש.


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Nesher  לחץ כאן להצגת דירוג המשתמש
חבר מתאריך 2.7.02
2 הודעות, 24 פידבק
   17:24   20.10.12   
אל הפורום  
  3. קריא הוא לא יהיה ב-100%  
בתגובה להודעה מספר 2
 
תמיד מומלץ לשים הערות

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


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
TheKid לחץ כאן להצגת דירוג המשתמש
חבר מתאריך 5.10.07
17978 הודעות, 1 פידבק
   17:33   20.10.12   
אל הפורום  
  4. ברור.. ועדיף כמה שיותר... אבל כמובן תלוי איזה קוד....  
בתגובה להודעה מספר 0
 
   אם זה קוד של GUI פשוט ואין שם תנאים ודברים שעשית לשם משהו מסויים שיש סיכוי שאם משהו אחר יקרא הוא לא יבין או אתה לא תבין עוד שנתיים למה עשית את זה אז יכול להיות שאפשר לווותר


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
VeNom  לחץ כאן להצגת דירוג המשתמש
חבר מתאריך 7.6.02
7922 הודעות, 1 פידבק
   18:11   20.10.12   
אל הפורום  
  5. רק במקרי קצה..שעושים דברים לא שגרתיים או מסובכים..  
בתגובה להודעה מספר 0
 
   אני הגעתי למצב שגם אם יש תיעוד על מחלקה, אני לא קורא אותה..ישר קורא קוד..אם זה היה תיעוד של שורה\2 הייתי משקיע..


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Deuce 
חבר מתאריך 1.9.08
6225 הודעות
   19:27   20.10.12   
אל הפורום  
  6. כן, בהחלט  
בתגובה להודעה מספר 0
 
תיעוד של חתימות של פונקציות וכן תיעוד על קטעים לא טריוויאלים שמבוצעים במהלך הקוד.






                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Dr_69
חבר מתאריך 24.3.02
1275 הודעות
   19:34   20.10.12   
אל הפורום  
  7. בעד, כמובן.  
בתגובה להודעה מספר 0
 
   אני תמיד כותב הערות אם יש לי קוד של 1000 שורות. לדוגמא אפליקציה,
יש המון דברים שאני צריך לזכור מה עושה כל אחד ואני לא רוצה לקרוא את כל הקוד ולהבין מה קורה, אני פשוט שם הערה ועובר על הקוד ויודע.

גם יותר קל לדבג ככה. אני רושם בהערות שלי מה מתחבר למה ולדוגמא יש בעיה בשורה 8 אז אני יודע שאני צריך ללכת לשורות 10-22 מפני ששם יכולה להיות הבעיה.

לרוב אני משתמש בהערות רק בmodel ולא בController מפני שאין צורך.

זה הצורך שלי בעיקרון.



                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Zippo 
חבר מתאריך 26.5.02
7921 הודעות
   19:35   20.10.12   
אל הפורום  
  8. צריך, אבל לא כמו ראש בקיר.  
בתגובה להודעה מספר 0
 
כבר ראיתי קוד כזה:
//this methods retrieves the command and execute it.
int getCommandAndExecute()

ברור שהערות כאלה לא ממש תורמות, וסתם מזהמות את המסך.
מה שצריך, זה הסבר טוב.
למשל, מה זה ה-return value, יש int, מה הוא מציין.
מתי המתודה עלולה להיכשל, אם בכלל וכו'...

אגב, בטוח שכולכם מכירים, כי זה עתיק כמו הגלות, ובכ"ז... מצחיק לאללה:
http://stackoverflow.com/questions/184618/what-is-the-best-comment-in-source-code-you-have-ever-encountered


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Nesher  לחץ כאן להצגת דירוג המשתמש
חבר מתאריך 2.7.02
2 הודעות, 24 פידבק
   20:19   20.10.12   
אל הפורום  
  10. חחח אדיר  
בתגובה להודעה מספר 8
 


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
sza  לחץ כאן להצגת דירוג המשתמש
חבר מתאריך 26.4.02
12357 הודעות, 22 פידבק
   01:20   23.10.12   
אל הפורום  
  14. ענק, ישבתי היום במהלך שיעור 3 שעות C# וקראתי את זה  
בתגובה להודעה מספר 8
 

ולא הפסקתי לצחוק חצי מהשיעור
כמובן שיש לי עכשיו הרבה חומר להשלים


--
צחי.


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Zvikadori
חבר מתאריך 3.8.02
5369 הודעות
   20:09   20.10.12   
אל הפורום  
  9. אני בגישה היותר זריזה - לכתוב קוד שמסביר את עצמו...  
בתגובה להודעה מספר 0
 
   ובעיקר להשתמש ב-refactoring, כזה שיגרום לקוד להיות עוד יותר מובן(לא מתאים לסביבות שממש ממש דורשות יעילות גבוהה, או מוגבלות בכוח עיבוד/זיכרון).

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

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

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

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


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Zippo 
חבר מתאריך 26.5.02
7921 הודעות
   23:47   20.10.12   
אל הפורום  
  11. יש מן כלל כזה שאומר:  
בתגובה להודעה מספר 9
 
כל פעולה שאתה רוצה לעשות, אם אתה מצליח למצוא לה שם טוב שמתאר אותה, כנראה שפיסת הקוד הזאת צריכה להיות מתודה. אם אתה לא מצליח למצוא לה שם טוב, אז לא.


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
asco88 
חבר מתאריך 17.6.04
26757 הודעות
   00:07   21.10.12   
אל הפורום  
  12. חח תאכלס..  
בתגובה להודעה מספר 11
 


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
יוחאי
חבר מתאריך 30.12.15
163 הודעות
   00:21   21.10.12   
אל הפורום  
  13. טוב אני גם אגיד את דעתי!:)  
בתגובה להודעה מספר 0
 
   קודם כל אני חושב שיש 2 סיבות עיקריות לכתוב הערות:

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

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

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

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

אני נגד תיאור של אלגוריתמים, כי בדרך כלל אף אחד לא ממיצא את הגלגל מחדש כי אין באמת צורך, אם מדובר בsorting algorithms או כל פעולה מורכבת אחרת, היא תמיד תהיה מבוססת על רעיון קיים (אז אולי כן שווה להוסיף שורה אחת שאומרת This code is base on x algorithm: http://wiki/algorithm

אבל ברגע שאני רואה הערה מהסוג הזה זה מטריף אותי:

//now we checking if x is equal to y

if (x === y)

לתאר חלק ממבני הבקרה של השפה זה מגוחך בעיני.

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


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד

תגובה מהירה  למכתב מספר: 
 
___________________________________________________________________

___________________________________________________________________
למנהלים:  נעל | תייק בארכיון | מחק | העבר לפורום אחר | מחק תגובות | עגן אשכול
       



© כל הזכויות שמורות ל-רוטר.נט בע"מ rotter.net