בלוג התוכנה של שני פדידה

נתקלתי במאמר הזה  שמדבר על code review checklist, אז החלטתי לכתוב אחד משלי.  אמנם יש המון מאמרים בסגנון  code review checklist ברשת, בנתיים קראתי רק את המאמר המקושר. זו פעם ראשונה שאני מפרמלת את תהליך ה code review העצמי בצורת מסמך ואני מתכוונת מרגע זה להשתמש בו לפני code review .לכן  אני מניחה שהפוסט יעבור עוד שינויים. בנוסף אשמח לשמוע מכם הצעות מה עבד לכם ומה פחות. אני מניחה שהרשימה פה מושפעת מסוג העבודה שאני עושה: backend וכתיבה מחדש וגם אולי מהשפה שבה אני כותבת C# . Scope כדאי לבחור scope קטן . כדי שיהיה קל לעבור עליו בפגישה אחת (בהנחה שעושים code review ביחד). כלל אצבע שנראה לי הגיוני :עד שבוע עבודה. כללי לוודא שלא השארתי TODO בקוד  / נקודות פתוחות. שמירה על קונבנציות   - באמצעות כלי אוטומטי. תיעוד החלטות ביצועים: IO ,  מבני נתונים. קוד נקי שימוש בכלי static code analysis ( לדוגמה sonarQube) ותיקון  ההערות שהוא נותן. הפרדה בין command ל query SOLID בדיקה של coupling ע"י ספירת כמות התלויות היוצאות ממחלק...