ABA


"עזרה עם code review למשהו שכתבתי"
גירסת הדפסה        
קבוצות דיון פיתוח, תיכנות ובניית אתרים נושא #22324 מנהל    סגן המנהל    מפקח   Winner    צל"ש   מומחה  
אשכול מספר 22324
כמה עוד לחץ כאן להצגת דירוג המשתמש
חבר מתאריך 18.11.09
178 הודעות, 11 פידבק
   10:04   15.05.19   
אל הפורום  
  עזרה עם code review למשהו שכתבתי  
 
   אני כרגע עושה פרויקט שאני המפתח היחיד בצוות, אז אין לי את מי לשאול כל כך.
רציתי לבקש את עזרתכם איך לשפר את הקטע קוד הבא-
הבנתי שעדיף לא לעשות return בתוך קטע של catch, ובכלל לא לתת ל try catch להכתיב את ה- flow של התוכנית.
מה שקורה בקוד זה שאני בודק האם משהו נמצא במנגון cache שלי, במידה ולא - אני קורא אותו מהכונן הקשיח ועושה השמה ב cache.
הסיבה שאני משתמש ב try ו- catch זה בגלל פעולת הקריאה מהכונן הקשיח.

איך הייתם משנים את הקוד? או שאולי הוא כן בסדר לפי עקרונות best practice?
תודה אלון



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

  האשכול     מחבר     תאריך כתיבה     מספר  
  מבחינת best practice לא נהוג לתפוס exception כללי Mr Andersson 15.05.19 10:14 1
     תודה על התגובה. אם תאורתית יש סיכוי שווה שיחזור לי אחד מ3 סוגי אקספשן? כמה עוד 15.05.19 10:28 2
         אתה יכול לתפוס את שלושתם בשורה אחת או לתפוס כל אחד בנפרד Mr Andersson 15.05.19 10:49 3
         ואיך אתה יודע את זה לפי ההודעת שגיאה שאתה מחזיר? Zeet3x  15.05.19 22:00 4

       
Mr Andersson
חבר מתאריך 19.3.02
6183 הודעות, 1 פידבק, -2 נקודות
   10:14   15.05.19   
אל הפורום  
  1. מבחינת best practice לא נהוג לתפוס exception כללי  
בתגובה להודעה מספר 0
 
   אתה יודע מה אתה עושה בקוד, ולכן אתה יודע לאיזה שגיאה לצפות, אז עדיף לתפוס את הexception הספציפי שייזרק

http://i60.tinypic.com/35aowg8.jpg


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
כמה עוד לחץ כאן להצגת דירוג המשתמש
חבר מתאריך 18.11.09
178 הודעות, 11 פידבק, 19 נקודות
   10:28   15.05.19   
אל הפורום  
  2. תודה על התגובה. אם תאורתית יש סיכוי שווה שיחזור לי אחד מ3 סוגי אקספשן?  
בתגובה להודעה מספר 1
 
   נגיד אחת מאלה יכולה לקפוץ לי:
בעיה בנתיב של הקובץ
בעיה עם הטקסט בתוך הקובץ
בעיה עם הקידוד (UTF או משהו כזה בקובץ)
?


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Mr Andersson
חבר מתאריך 19.3.02
6183 הודעות, 1 פידבק, -2 נקודות
   10:49   15.05.19   
אל הפורום  
  3. אתה יכול לתפוס את שלושתם בשורה אחת או לתפוס כל אחד בנפרד  
בתגובה להודעה מספר 2
 
   אם הטיפול הוא זהה בכל השגיאות אז תתפוס את שלושתם בשורה אחת

חוץ מזה הקוד נראה בסדר גמור לדעתי

http://i60.tinypic.com/35aowg8.jpg


                                                         (ניהול: מחק תגובה)
מכתב זה והנלווה אליו, על אחריות ועל דעת הכותב בלבד
Zeet3x  לחץ כאן להצגת דירוג המשתמש
חבר מתאריך 14.2.12
9858 הודעות, 24 פידבק, 18 נקודות
   22:00   15.05.19   
אל הפורום  
  4. ואיך אתה יודע את זה לפי ההודעת שגיאה שאתה מחזיר?  
בתגובה להודעה מספר 2
 
   אתה מחזיר הודעת שגיאה שתואמת לבעיה הראשונה.. אם מה שאתה אומר נכון ונזרקת שגיאה בגלל בעיה בטקסט אז אי אפשר לדעת.
הייתי גם עושה משתנה csv_suffix = '.csv' אבל זה באמת קטנוני.


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

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

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



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