mfp: Writing less code: The "else" statement

[glibs]

Цитата:

Сообщение от kashperuk

...
И если код получается более читабельным
...

А еще иногда код приходиться править (для расширения функциональности). А еще бывает задача переноса на более свежий СП модифицированного кода. Именно поэтому рекомендуется ставить после IF скобки, даже если за ним следует один оператор всего. Чтобы свой кусок кода можно было просто и быстро вставить, нормально при этом откоментировав. Это не я придумал. Просто взял на вооружение. Видел нелепые и обидные ошибки на невнимательность, связанные именно с этим примером при модификации чужого кода. А не ошибается только тот, кто ничего не делает. Значит это просто лишний повод.

Сравните.

...
if (x < 0) throw error ("Error");
if (x == 0) return true;
...

и

...
    
// GLIBS: Some modification -->
// if (x < 0) throw error ("Error");
if (x < 0)
{
    this.doSomething();
    throw error ("Error");
}
// GLIBS: Some modification <--
if (x == 0) return true;
...

И следующие два примера.

...
if (x < 0)
{
    throw error ("Error");
}
if (x == 0)
{
    return true;
}
...

и

...
    
if (x < 0)
{
   // GLIBS: Some modification -->
   this.doSomething();
   // GLIBS: Some modification <--
   throw error ("Error");
}
if (x == 0)
{
   return true;
}
...

Если не комментировать — крышу сорвет при попытке разобраться в изменениях в коде (при том же переходе на новую версию, например, или при изучении чужих модификаций).

Если комментировать, то очень тяжело пользоваться функцией сравнения кода.

Так что mono прав, IMHO. А ВР нужно соблюдать.

[mazzy]

Вот ведь тема эволюционировала...

Цитата:

Сообщение от mazzy

Поэтому без-else-овую оптимизацию, по-моему, рационально делать только в простейших случаях, когда в каждой ветке делается return.

Цитата:

Сообщение от mono

Eсли не нравятся Best Practices - напишите свою Аксапту, с блекджеком и шлюхами.

Цитата:

Сообщение от glibs

А ВР нужно соблюдать.

BP нужно расширять и делать более интеллектуальным.

glibs, я же написал "только в простейших случаях, когда в каждой ветке делается return.". Если вставляется оператор до return'а, а программист не проконтролировал скобки, то... это скорее говорит о неопытности программиста, нежели о Best Practice.

Цитата:

Сообщение от glibs

Если не комментировать — крышу сорвет при попытке разобраться в изменениях в коде (при том же переходе на новую версию, например, или при изучении чужих модификаций).

А чтобы было легче разбираться с проапгрейдженным кодом, то пиши вот так:

...
if (x < 0) { this.doSomething(); throw error ("Error"); } // GLIBS: Some modification --
if (x == 0) return true;
...

Если у тебя длинное выражение, не помещающееся в одну строку, то пиши в несколько строк, как ты показал.
И в этом случае очень хорошо, что сравнение покажет значительные изменения.


Обсуждение обязательности скобок идет еще со времен появления структурного программирования, С и Паскаля. Сколько копий было сломано в свое время. Вот уж не ожидал, что эта "священная война" разгорится снова.

Ок. Перехожу в режим чтения этой ветки.

[glibs]

Цитата:

Сообщение от mazzy

...
я же написал "только в простейших случаях, когда в каждой ветке делается return.".
...

А я писал, что любой изначально задуманный как простейший код может со временем усложниться.

Цитата:

Сообщение от mazzy

...
я же написал
...
Если вставляется оператор до return'а, а программист не проконтролировал скобки, то... это скорее говорит о неопытности программиста, нежели о Best Practice.
...

А я писал, что люди не машины. И даже у очень хорошего программиста могут быть запарки при запуске, недосыпание, проблемы в личной жизни...

Например, ERP системы внедряют для снижения человеческого фактора и зависимости от личностей... я склонен к чему-то подобному в кодировании.

Цитата:

Сообщение от mazzy

...
Перехожу в режим чтения этой ветки.
...

У меня экономическое образование. Так что программирование воспринимаю как умею. Тоже с удовольствием послушаю более опытных коллег, если откликнутся.

[mazzy]

Цитата:

Сообщение от glibs

А я писал, что люди не машины. И даже у очень хорошего программиста могут быть запарки при запуске, недосыпание, проблемы в личной жизни...

тема блога как раз о том, что чем проще написан код, тем проще его читать человеку. Следить за скобками - это должно быть на уровне рефлексов, по-моему.

Цитата:

Сообщение от glibs

А я писал, что любой изначально задуманный как простейший код может со временем усложниться.

ну, дык. Усложни код, когда он усложнится. Заранее то зачем?

[glibs]

Цитата:

Сообщение от mazzy

...
Ок. Перехожу в режим чтения этой ветки.
...

Цитата:

Сообщение от mazzy

...
тема блога как раз о том
...

Я так и знал, что ты опять всех обманешь .