[פוסט זה התפרסם במקור בפייסבוק.]
יאללה, code review לציוץ של צה"ל.
- בשורה 1 רצוי שה-
ifיהיהwhileכדי שהתגובה תקרה תמיד ולא רק במתקפה הראשונה. - בשורות 3-5 עדיף להשתמש ב-consts במקום במחרוזות קסם בגוף הקוד.
- בשורות 3-5 עדיף להחליף את הקוד בקוד יותר גנרי שיאפשר לתמוך בעוד מתקפות בעתיד בלי צורך לשנות את הקוד ולקמפל מחדש.
- זה לא “פיצוח של הקוד” אלא פשוט חשיפת הקוד אם כביכול זה הקוד שצה"ל מריץ ואתם צה"ל.
- נראה שהקוד כתוב בסי-שארפ שהיא שפה שרצה ברוב המקרים על Windows ולעיתים רחוקות מאוד על גנרלים צבאיים ולכן רוב הסיכויים שסי-שארפ אינה השפה המתאימה ביותר למשימה.
- הקונפליקט הישראלי-פלסטיני די מסובך וככל הנראה לא ניתן לייצג אותו בצורה נאמנה באמצעות 12 שורות קוד.
- יתר על כן, יתכן שאם אתם צבא מאורגן של מדינה מסודרת אז להתבדח עם בדיחת קוד גרועה על קונפליקט שבו אנשים פאקינג מתים במשך שנים לצורך איסוף לייקים בטוויטר זה איפשהו בין מביך למחליא.
- שימוש באופרטור
&&במקום||בשורות 3-4.