liranoz12/dsp — pull request #1 by oryband (2014-04-10 11:06:59+00:00)
Title: pull changes from me

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

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

אני מעדיף להשתמש בשמות קצרים של משתנים ולא לקרוא למשתנה בשלוש מילים (ארוכות) שונות כמו
requestMessageHandler
אלא פשוט לקרוא לזה handler
או משהו כזה

בחלק מהמקומות גם כתבת הערות למרות שברור מההקשר מה קורה. אם נגיד לפונקציה קוראים
handleMessage
אז אין צורך לשים בצד הערה ולכתוב
`// handling messages`
תכתוב הערות על קטעי קוד שלא ברור מההקשר או מהשם שלהם מה הם עושים

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