מהו Code Review? ולמה חשוב לבצע אותו לפני עליה לאויר?
פיתוח תוכנה או אפליקציה מתבצע בתהליך לוגי רציף ומסודר.
זהו תהליך שבו יש להשתמש בקוד בצורה תקנית, איכותית ויעילה, לאור המטרה שרוצים להשיג.
אחד השלבים הקריטיים ביותר לפני עליה לאוויר נקרא code review (קוד ריוויו).
נסביר מה זה אומר ולמה זה כל כך חשוב לפני שהפיתוח מוגמר והתוכנה, האלגוריתם או האפליקציה עולים לאוויר.
Code Review יעיל הרבה יותר מבדיקות אוטומטיות
לפני עליה לאוויר, בכל סוג של תוכנה או אפליקציה ניתן לבצע שלל בדיקות אוטומטיות.
אלה בדיקות בעלות יעילות מוכחת, אך הן אינן מספיקות וצריך לבצע עוד בדיקות על ידי מתכנתים ומומחי פיתוח.
זו עובדה, כל מי שעוסק בפיתוח של תוכנה או אפליקציה מקפיד לאורך התהליך לבצע ניפוי שגיאות או תיקון טעויות.
אבטחת איכות היא חלק בלתי נפרד מכל כתיבה של קוד או אפילו פיסה של תוכנה.
אולם מי שביצע את הפיתוח בעצמו יתקשה לבצע את כל הבדיקות, דווקא מכיוון שהוא זה שביצע את כתיבת הקוד.
לכן, מקובל כי הפעולה שנקראת לעיתים גם סקר קור, תבוצע על ידי מומחה פיתוח אפליקציה, אך לא מי שכתב את הקוד בעצמו.
זוג נוסף של עיניים, כך מקובל להעריך, מזהה בצורה טובה את הפגמים, טעויות ההקלדה או בעיות אחרות,
בשלב הסופי והאחרון שלפני העלייה לאוויר (או בנקודות מפתח אפילו קודם לכן).
כך ניתן לשחרר את המוצר האיכותי, שעומד בכל כללי הלוגיקה, הניסוחים הפורמליים והפונקציות הביצועיות הרצויות.
קוד ריוויו מאפשר שיפור ביצועים ותוצאה טובה יותר
ביצוע סקר קוד במובן הפורמלי, נועד כפי שאמרנו לבדוק שאין תקלות ואין טעויות.
אך תיקוני שגיאות הן רק צד אחד של המטבע. יש עוד דבר לא פחות חשוב, שאליו צריך לחתור.
כפי שיודע כל מי שעוסק בפיתוח, יש הרבה יותר מדרך אחת לפתור בעיה.
יש המון דרכים להגיע לאותה תוצאה, אך לא רק התוצאה היא החשובה.
לפני השלמה של פיתוח תוכנה, הדבר הרצוי הנוסף הוא לוודא שהאפליקציה פועלת בצורה אלגנטית,
חלקה ומהירה כמה שרק אפשר.
לא פעם ולא פעמים הסקירה של הקוד מגלה שניתן לבצע כמה שינויים, כאלה או אחרים, ולשפר את הביצועים.
זמן ריצה קצר יותר וחיסכון במשאבים בהחלט ישפרו כל תוכנה או אפליקציה באשר הם.
סילוק באגים ובקרת איכות על ידי Code Review איכותי
לא ניתן לומר מראש האם סקר קוד יסתיים בתוך זמן קצר או שיהיה צורך בתהליך יותר ארוך.
זה מאוד תלוי בסוג הפרויקט שעל הפרק, במספר המפתחים שהיו מעורבים בפרויקט.
גם ביצוע שינויים לאורך הכתיבה והפיתוח משפיעים על איכות התוצאה שלפני הסקירה.
לפני העלייה לאוויר, תמיד צריך לוודא כי שלב הסקירה יתבצע ברמה גבוהה, בלי שום וויתור ובלי שום פשרות.
זהו רגע האמת שבו צריכים לזהות את כל הבאגים שאולי עוד נשארו.
זה הרגע שבו מסלקים את הבאגים ומנפים את השגיאות ומייעלים את הקוד.
בקיצור, זה לב ליבו של תהליך בקרת האיכות.
למה הליך code review והליך QA כ”כ חשובים?
כאשר יודעים שסקירת הקוד איכותית וניתן לסמוך עליה, זה יכול להאיץ פרויקטים ולהביא לתוצאות משופרות.
בקרת האיכות יכולה להשפיע על שביעות הרצון של המשתמשים ולחסוך כסף רב,
שכן צריך פחות עדכונים ותיקוני טלאים חוזרים ונשנים.
זו הדרך להשיק את המוצר הטוב ביותר, את האפליקציה האיכותית שתפעל לרווחת המשתמשים,
עם מינימום צורך בתמיכה טכנית שוטפת.
בשל החשיבות הרבה, לרוב ביצוע הבדיקות הסופיות יושלם על ידי אנשי פיתוח מנוסים ובכירים.
מה כלול בדוח קוד ריוויו?
סקר קוד – נבדוק את הקוד שורה אחר שורה כדי לזהות בעיות קוד ומבנה אפשריות כגון באגים, שגיאות תחביר,
לוגיקה שגויה או פרצות אבטחה.
סגנון קוד ועיצוב – נבדוק אם הקוד עומד בתקני הקידוד וההנחיות שנקבעו עבור הפרויקט ושנכתבה דוקומנטציה איכותית וברורה.
פונקציונליות קוד – נבדוק את הקוד כדי לוודא שהוא עומד בדרישות הפונקציונליות ומתפקד לפי האפיון.
דוקומנטציה – נבדוק את תיעוד הקוד כדי לוודא שהוא מדויק, מעודכן ומספיק ברור כדי שיאפשר למפתחים אחרים
להבין ולתחזק את קוד המקור.
משוב והצעות – נספק משוב והצעות למפתחי התוכנה כיצד לשפר את איכות הקוד, הביצועים ויכולות הגמישות.
דוח מסכם – לאחר סיום סקירת הקוד, נגיש דוח מפורט עם הממצאים והמלצות לצוות הפיתוח.
