ציטוט:
נכתב במקור על ידי IgalSt
אני לא מתכנת PHP, אבל מבחינת פידבק בונה שאפשר לתת בכל מקרה בלי קשר לשפה:
הדבר הראשון שצורם לי בעין זה שאין כל הזחה (indentation) בקוד.
דבר זה מאוד פוגע בקריאות של הקוד וכן בתחזוקה העתידית שלו וקוד מהסוג הזה לעולם לא יעבור code review.
שמתי לב שניהול ה-empty line space לא אחיד ופעם יש כמה שורות ריקות ופעם יש אחת. מומלץ להיות קונסטינטנטי.
ממליץ בחום לעשות סדר.
|
אתה צודק, אפשר לסדר את הקוד קצת מהבחינה הזאת, אבל בעיקרון זה דברים די קטנים.