כיצד אתם מבצעים את ההזחה בקוד שלכם?
בעזרת רווחים, או עם התו Tab, או שלא אכפת לכם בכלל?

התפלאתי לקרוא ב-PEP8 שמחברי Python מאמצים הזחה של ארבעה תווי רווח:

ציטוט:

Use 4 spaces per indentation level. ... The most popular way of indenting Python is with spaces only.

למרות שבעבר תמיד המליצו על שימוש בתו tab.

כיום וגם בעבר עורכי הטקסט הקלו בעריכת קוד המוזח גם בעזרת רווחים וגם בעזרת Tab, אבל כרגע השימוש ב-Tab נראה לי הגיוני יותר.

אני משתמש רק ב TAB.
אם במקרה אני עושה Copy paste מקוד שראיתי באינטרנט, והוא מופיע לי עם רווחים ב IDE, אני אתאמץ במיוחד למחוק את הרווחים ולשים שם טאבים במקום

בברכה, מתן.
www.MatanNarkiss.com

Tabs for life!

אם כבר אנחנו בדיון של "איך אתם עושים X", איך אתם משאירים הערות בקוד שלכם?
אחרי שורת הקוד אם זה משהו ספציפי, לפני שורת הקוד כדי להסביר על התהליך שהולך לקרות, לפני התנאי, באותה שורה אם התנאי, הבנתם את הכיוון...

לפני התהליך, שיסביר על מה הוא אמור לעשות, בשורה מעל הפונקציה.

בברכה, מתן.
www.MatanNarkiss.com

ולפעמים אפשר בקצרה באותה השורה (אבל כדאי להימנע)

מסתבר שגם Zend @ PHP תומכים ב4 רווחים ובלי TAB, ואפילו הגדירו את TAB כאסור לשימוש....

Zend Framework Documentation: PHP File Formatting

כן, אבל בכל התיעודים האלה לא מסבירים מדוע.
באשכול Tabs versus spaces in Python programming @ stackoverflow.com, אחת התגובות מסבירה למה לא לערבב רווחים ו-Tabs, ואפשר להסיק מזה למה קובעים שימוש רק ברווחים בשביל ההזחות:

ציטוט:

Editor-to-editor mistake occurs when you have mixed indentation within a file. This arises as follows: a block of code is indented with 4 spaces, and then one indentation level "in", it is indented with tabs. Now the heathen who did this (mixing tabs and spaces) had it so his tabs are also 4 spaces, so he sees no problems, and python sees no problems. Now our victim comes along later, and he has his tabs set to 8 spaces. Now our victims thinks the code looks all whacked, and fixes it by removing one level of indentation, which now makes the code look like it is still 2 levels of indentation, but is really one level. At this point all hell breaks loose. The lesson here is that you should never, ever, mix tabs and spaces. If you keep to this, then it is easy to reindent your code into spaces or tabs, regardless of which you personally use. The best way to ensure you don't mix tabs and spaces is to always run python with -tt, which will produce an error when tabs and spaces are mixed. As for tabs and spaces, I personally use tabs so separate indentation from appearance - it is much easier to change the appearance of code when it is indented with tabs than it is with spaces. I know this runs contrary to what 99% of python programmers do, but that is my personal preference, and it is easy in any case to convert a tabbed file to a spaced one. The reverse is not always true, since you can accidentally whack out 4 spaces in strings etc.

כלומר כשהקוד מועבר למישהו אחר, ייתכן שהעורך שלו יגרום לקוד להיראות אחרת (ולא מסודר כמו המקור), מה שיצריך סידור מחדש של הקוד. בשפה כמו python זה יכול לגרום לבעיות, לכן זה עניין קריטי יותר משפות אחרות.
אבל אם משתמשים רק ב-Tab בשביל ההזחות, ומקפידים לא לערבב עם רווחים - אז לא נראה שיש בעיה...

גם ככה חלק מהעורכים, כמו שנאמר גם באחת התגובות שם, תומכים באפשרות של TAB = 4 spaces ואז לא אמורה להיות שום בעיה, חוץ ממחיקה ידנית של רווחים בשביל לשנות את ההזחה אבל את זה אפשר לעשות בעזרת shift+TAB (ומן הסתם ברוב העורכים אפשר לעשות הזחה לבלוקים שלמים של קוד שמסומנים... כיף טהור)

פייתון זה לא דוגמה טובה.. בפייתון ורק אלוהים יודע למה ויתרו על ה }{ / begin end ומשתמשים בהזחה בישביל להגדיר בלוקים.

דווקא שפה יפה... אני שמחתי שהורידו את הסוגריים המסולסלות האלו סוף סוף! יותר בלי "רגשות האשמה" שמא אני שם את הסוגריים המסולסלות במיקום לא "נכון"...

מה זה מיקום לא נכון?
בהודעה שלך למעלה יש הסבר למה סוגריים מסולסלות זה טוב....

"מיקום לא נכון" מבחינת סגנון כתיבת הקוד. הכוונה:

קוד:

if (cond) {
 //...
}
/*OR*/
if (cond)
{
 //...
}

וכד'...
תאמין או לא אבל לי זה מפריע :/

ציטוט:

בהודעה שלך למעלה יש הסבר למה סוגריים מסולסלות זה טוב....

הציטוט מסביר למה לא כדאי לשים באותה השורה רווחים ו-Tabs. מי שלא ישים אותם באותה השורה - לא אמורה להיות בעיה. אני לא זוכר מתי עירבבתי תוים וטאבים באותה השורה, ואני מניח שרוב האנשים לא יעשו ככה, אז זה בסדר להשתמש בשפות המצייתות לחוק ה-off-side. הרי בכל מקרה מתכנת משתמש בהזחה כדי לסדר את הקוד, אז למה צריך את הסוגריים המסולסלות המיותרות האלה? שההזחה תדריך גם את המנתח התחבירי.

אני אוהב את הסוגריים המסולסלים, ולפעמים יש דברים שאפשר לכתוב ככה בשורה אחת בצורה ערבית.
לדעתי זה הרבה יותר ברור מהזחה... אבל זה כי אני רגיל...

ציטוט:

במקור נכתב על ידי hellfrost ולפעמים יש דברים שאפשר לכתוב ככה בשורה אחת בצורה ערבית.

גם בפייתון אפשר לכתוב בסגנון ערבי

קוד:

if foo == 'blah': do_blah_thing()

ממש לא התכונתי לתנאים...

זה לא ערבי זה רגיל דווקא.. ?: זה נחמד....

חייב חייב חייב להזדהות איתך!
כל פעם שאני מתחיל { } (ו-Aptana משלים אותן, שאגב ממש אהבתי ) אני מרגיש רע עם עצמי שהסוגריים שורה מתחת, אבל החלטתי להתרגל לסטנדרט שראיתי בכל מיני מקומות ובעיקר בZEND - כנראה שהם יודעים מה הם אומרים.
חוץ מזה שהרבה יותר סיכויים שעוד בנאדם יקרא את הקוד שלי וכותב גם לפי הסטנדרט שלהם מאשר לפי הצורה שאני כותב...

אגב, שאלה:
עד לא מזמן, ממש ממש אהבתי להשתמש ב if(!$var) a ובזמן האחרון אני מרגיש לא בנוח עם זה, כי הסימן קריאה לא תמיד כזה ברור שהוא נמצא שם, אז התחלתי להשוות לערכים FALSE וTRUE.
אני מגזים?
אותו דבר בנוגע ל:

קוד:

$var = isset($_POST['var']) ? $_POST['var'] : 'default';

במקום

קוד:

if (isset($_POST['var']))
$var = $_POST['var'];
else $var = 'default';

אני צריך לצאת מנקודת הנחה שהבא אחרי שיקרא את הקוד אולי לא יכיר צורות מיוחדות של השפה ובגלל זה לפגוע בשימוש ב"טריקים" של השפה או שפשוט לעשות מה שבא לי העיקר שיהיה כתוב בצורה טובה ויעילה??

קוד PHP:

 vehicle = arg == 'B' ? bus :
          arg == 'A' ? airplane :
          arg == 'T' ? train :
          arg == 'C' ? car :
          arg == 'H' ? horse :
          feet; 


ציטוט:

במקור נכתב על ידי fealls עד לא מזמן, ממש ממש אהבתי להשתמש ב if(!$var) a ובזמן האחרון אני מרגיש לא בנוח עם זה, כי הסימן קריאה לא תמיד כזה ברור שהוא נמצא שם, אז התחלתי להשוות לערכים FALSE וTRUE. אני מגזים?

לדעתי כן.
בסופו של דבר צריך להגיע למצב שאתה כותב את הקוד באותו הסיגנון בעיקביות, וזה לא אמור לפגוע ביעילות של הקוד.

אגב לדעתי דברים בסגנון:

קוד:

if (cond) func();

גם לא טובים, עדיף פשוט להישאר עם:

קוד:

if (cond)
    func();

ציטוט:

אולי לא יכיר צורות מיוחדות של השפה

Ternary operator זה אחד הדברים המוכרים ביותר
אבל לא אמור להיות דבר כזה שהוא לא מכיר דברים מיוחדים של השפה, ואם יש משהו שהוא לא מכיר - אז סימן שהוא רק התחיל ללמוד אותה לא מזמן...

מסכים איתך לגבי IF עם שורת קוד אחת, הרבה יותר ברור ונוח

אגב גם בחלק של הHTML אני מקפיד (או לפחות התחלתי להקפיד) בזמן האחרון על ירידת שורות בנוסף להזחה מסודרת שכבר הייתה קיימת - אני ממש ממש שונא לעשות סקרול ימינה ושמאלה. זה מעצבן.

אני תמיד משתמש באפשרות השנייה שהצגת בציטוט מהסיבה הפשוטה שהאפשרות הראשונה ממש מציקה לעין למרות שרבים וטובים משתמשים בה.

https://www.youtube.com/watch?v=0HRGczvZINQ&noembed

כן, אבל זה יכול להגיע למצב כזה שהסוגריים תופסות לך את כל המסך, לדוגמא:

קוד:

while(cond)
{
    func();
}
if (cond2)
{
    func2();
}
else (cond3)
{
    func3();
}

במיוחד אם ב-IDE אתה פותח למטה\מעלה חלונות נוספים..

נכון, זה יכול להגיע למצב הזה... הוא בהחלט יותר טוב ממצב אחר ושכיח יותר בו קשה לראות בצורה ברורה (למרות שזה תאכלס עניין של הרגל) איפה מתחילה פונקציה, ואיפה התוכן שלה וכו'.
אל תחפש איך הקוד יכיל 'פחות שורות קוד' כדי 'לייעל' אותו. בכל מה שקשור לעיצוב והערות - תשקיע מקסימום שורות קוד. את האלגוריתם תכתוב בצורה שהיא פשוט קריאה (גם אם זה כולל תנאים כמו השאווה לערך בוליאני במפורש: if (isLogged == true).
פעם גם אנחנו כל הזמן ניסינו 'לכתוב את הקוד הכי קצר' לכל בעיה. היום זו ממש לא הגישה שלנו. במיוחד לאחר שלמדנו שקוד קצר הוא קודם כל - לא בהכרח היעיל ביותר, ושנית בגלל שכשהיינו צריכים לעבוד זה עם קודו של זה - לא תמיד הקוד היה אינטואיטיבי וברור. ריבוי הערות וסוגריים מקלים על העבודה.

בברכה, מתן.
www.MatanNarkiss.com

ציטוט:

במקור נכתב על ידי Narxx if (isLogged == true).

חשבתי שמישהו התכוון לזה באחת ההודעות ממקודם. בכל מקרה זה עילה לבעיטה. ועל חוסר אלגנטיות.
יש יתרון לקוד קצר, בהירות ופשטות, כל עוד זה לא בא על חשבון קריאות זה מצוין.

לא אומר לכתוב קוד הכי קצר, אבל זה שסוגריים מסולסלות תופסות 50% מהמסך - גם יכול להקשות על הבנת התוכנה... וגם debugging (מגיע ל-breakpoint המיוחל, ממשיך כמה steps, ואז אתה רואה קוד שמתייחס לקוד שנמצא עמוד לפני, אז צריך לגלול למעלה, ואז שוב למטה ו\או למעלה כדי להגיע לקוד אחר..)

כן, שנינו יכולים להקצין הרבה... בפועל אני לא כותב כל כך הרבה בלוקים שמכילם רק שורה אחת.
אם מדובר בשורה אחת בודדת, אפשר גם בלי סוגריים בכלל - כל עוד יש הזחה של TAB אחד פנימה... זה די קריא (אבל שורה רווח בין הפקודה לבין הפונקציה החדשה).

בנוגע לבעיטה, שמע, אני לא יודע... מצד אחד אנחנו משתמשים בשמות משמעותיים בדיוק כדי שיהיה ברור מה isLogged אומר, גם מבלי שעקבנו מילה מילה בקוד (וגם מבלי לקרוא בכלל קוד, אפשר להסיק כי isLogged בודק האם אנחנו מחוברים או לא למערכת). מצד שני, אי אפשר להיות 'יותר מדי ברורים' ואם במקום לכתוב
(if (isLogged, אתה תכתוב (if (isLogged == true - לא יקרה שום אסון... במקרה הכי גרוע, מישהו שכבר היה ברור לו מראש מה התנאי אומר מבלי ההשוואה המפורשת, יבין אותו הדבר גם עם ההשוואה המפורשת. הפוך, לא תמיד

בברכה, מתן.
www.MatanNarkiss.com

קוד PHP:

 bool bShouldLearnProgram = humYou.DoWantToBProgrammer()  ^  humYou.DoKnowProgramming();

try
{
    if ( ! bShouldLearnProgram )
    {
        LookUpCourse("Programming", "Professional").SendToCourse( humYou );
    }
}
catch(Exception ex)
{
    System.Console.WriteLn("cant send: " + humYou.Name + "to programming. \n" +" Because: " + ex.Message);
} 


אהבתי

בברכה, מתן.
www.MatanNarkiss.com

כאשר יש לי רק שורה אחת לכתוב בלולאה או ב-IF אז אני פשוט כותב אותו בלי הבלוק. זו הצורה הכי טבעית וברורה.
כאשר יש 2 שורות ומעלה, אני משתמש באפשרות שהזכרתי קודם.

https://www.youtube.com/watch?v=0HRGczvZINQ&noembed

אני מסכים איתך, אבל כשאני רק עובד על קוד חדש אני מקפיד על בלוקים תקינים, כדי שאם אוסיף קוד לא אצטרך לחשוב יותר מדי לאיפה ולהוסיף סוגריים או לא.
בסוף, כשאני עובר על הקוד בשביל תיקונים אסטטיים ולהוסיף עוד קצת הסברים, אני גם מוריד סוגריים ובלוקים מיותרים....

אני משתמש ב2 רווחים כטאב

הכי קל לעבור ככה בין עורכים שונים עם הגדרות שונות.
וחוץ מזה, זה גם חלק מהסטנדרט של דרופל, שיוצא לי לפתח לזה יחסית הרבה

!!אזהרה!!

ההודעה עלולה להכיל שברי אגוזים ו/או איברי דגים כלשהם

!!אזהרה!!

מוזר אני חייב להודות... אבל כל אחד ומה שנוח לו