איך מתמודדים עם ביקורת קוד בתור ג'וניור¶
שלחתם Pull Request ראשון בעבודה, סוף סוף. אחרי כמה שעות, מגיעה תגובה - ולא "מעולה, מאשר", אלא רשימה של שתים עשרה הערות. שינוי שם משתנה, הצעה לגישה אחרת, שאלה למה בחרתם ככה ולא אחרת. הדחף הראשוני של הרבה ג'וניורים הוא להיכנס ללחץ - "אני כישלון, הקוד שלי גרוע, אולי לא מתאים לתפקיד הזה". התגובה הזו נפוצה מאוד, וגם שגויה כמעט תמיד.
ביקורת קוד לא מודדת אתכם כאנשים¶
ההנחה שכל הערה על הקוד היא הערה עליכם היא המקור לרוב הכאב סביב code review. בפועל, ביקורת קוד היא תהליך טכני שמטרתו לשפר את הקוד ולמנוע בעיות בהמשך, לא לדרג את המפתחים. מפתח בכיר עם עשר שנות ניסיון מקבל הערות על הקוד שלו כל שבוע, וזה לא מפחית ממנו. ההערות הן חלק מהתהליך התקין, לא סטייה ממנו.
כמות ההערות לא אומרת מה שאתם חושבים שהיא אומרת¶
הרבה ג'וניורים מפרשים כמות גדולה של הערות כסימן לכישלון גדול. לרוב זה בדיוק ההפך - סוקר קוד רציני שמשקיע זמן בלכתוב שתים עשרה הערות מפורטות עושה זאת כי אכפת לו שתשתפרו, לא כי הוא רוצה להעניש אתכם. תגובה מדאיגה הרבה יותר היא "מאשר" שקט בלי שום הערה על PR ראשון של ג'וניור - לרוב זה סימן שאף אחד לא באמת קרא את הקוד לעומק.
איך לקרוא הערות בלי להיכנס להגנה¶
כשמגיעה הערה, הדחף הטבעי הוא לפעמים להסביר למה עשיתם ככה, כאילו צריך להצדיק את עצמכם. עדיף גישה אחרת: לקרוא את ההערה, לנסות באמת להבין את הנקודה, ורק אז להגיב. אם ההערה נכונה, פשוט מתקנים ומודים. אם משהו לא ברור, שואלים שאלה במקום להתגונן - "לא הבנתי למה זה עדיף, אפשר להסביר" זה משפט טוב הרבה יותר מ"אבל זה עבד ככה".
לפעמים באמת יש לכם נקודה טובה שהסוקר פספס. גם אז, עדיף לשאול ולהציג את נקודת המבט שלכם בנימה של בירור, לא של הגנה, כי לפעמים אתם צודקים ולפעמים מתברר שיש הקשר שלא הכרתם.
לתעד הערות חוזרות כדי ללמוד מהן¶
אם אתם שמים לב שאותה הערה חוזרת על עצמה בכמה ביקורות שונות - למשל שוב ושוב מבקשים מכם לטפל בשגיאות בצורה מסוימת - זה סימן שיש כאן דפוס שכדאי ללמוד אחת ולתמיד, לא רק לתקן כל פעם מחדש. ג'וניור שמתקדם מהר הוא לא מי שלא מקבל הערות, הוא מי שמפסיק לקבל את אותה הערה שוב ושוב, כי הפנים אותה.
מה עושים כשהערה מרגישה לא הוגנת או קשה מדי¶
לפעמים ביקורת קוד באמת נכתבת בנימה לא נעימה, גם אם התוכן שלה נכון. אם זה קורה שוב ושוב מאותו אדם, שווה שיחה ישירה ומכובדת איתו, או עם המנהל שלכם אם זה ממשיך. יש הבדל בין הערה ענייינית שקשה לשמוע לבין יחס לא הוגן שחוזר על עצמו, וחשוב לדעת להבחין ביניהם ולא לספוג בשקט את הסוג השני.
איך להגיע לביקורת הראשונה עם פחות חרדה¶
ככל שהבסיס שלכם בקוד נקי וסטנדרטים מקובלים חזק יותר, פחות הערות בסיסיות תקבלו, ותוכלו להתמקד בהערות שבאמת מלמדות אתכם משהו חדש. קורס התכנות הבסיסי החינמי שלנו עוזר לבנות בסיס כזה, כולל הרגלים של קוד נקי וברור.
ואם קיבלתם ביקורת קוד שהרגישה קשה ואתם רוצים דעה נוספת מבחוץ, יש קהילה שיכולה להסתכל על זה יחד איתכם בדיסקורד.
לסיכום¶
ביקורת קוד היא אחד הכלים הכי יעילים שיש לכם ללמידה מהירה כג'וניור, לא הוכחה לכישלון. תקבלו הערות בפתיחות, תשאלו כשמשהו לא ברור, ותשימו לב לדפוסים חוזרים כדי ללמוד מהם פעם אחת ולתמיד. מפתח שמפחד מביקורת קוד מתקדם לאט יותר ממי שמאמץ אותה כמורה זמין תמיד.