Обработка "ошибок".

[DIzer]

не правильная постановка вопроса - уместнее использовать фразу "добиться оптимального решения" - оно всегда есть в КОНКРЕТНОМ случае - а уж вопрос насколько оно оптимальное... я вот , например, почитал в этом топике мнение Алексея.... ну и высказался ... насчет "конфетки".

[valexey]

Где логика радостно перемешана с обработкой ошибок.

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

Ну, defer есть :-) Еще паника есть (которую можно поймать - похоже на исключения). Но мне кажется что паниковать по поводу и без - плохой тон. Функции высшего порядка есть :-)

[DIzer]

Вот это

Код: [Выделить]

f1()
f2()
f3()


и то что у вас в шапке топика - СОВЕРШЕННО РАЗЛИЧНЫЕ ВЕЩИ (вы уж разберитесь с тем что вам нужно,..)

Одно и то же. Мне нужно что бы f2 выполнялось когда f1 успешен. и так далее. Но эта связка (условность выполнение f2) это не логика приложения. То есть это не то, что должно выпирать.

Охох... Алексей .. если вам нужно 3 вызова подряд - if по ЛОГИКЕ не нужны... ЕСЛИ НЕТ то НИКУДА ВЫ ОТ НИХ НЕ ДЕНЕТЕСЬ в особенности если вы должны сделать что то ПОСЛЕ  этих  вызовов - это есть ЛОГИКА АЛГОРИТМА. у вас к ЛОГИКЕ ОБРАБОТКИ ОШИБКИ относится вызов функции Log() который я убрал из логики алгоритма... И хватит... на это... достало - вы...это.... довольно неконструктивны....

[valexey]

Охох... Алексей .. если вам нужно 3 вызова подряд - if по ЛОГИКЕ не нужны... ЕСЛИ НЕТ то НИКУДА ВЫ ОТ НИХ НЕ ДЕНЕТЕСЬ в особенности если вы должны сделать что то ПОСЛЕ  этих  вызовов - это есть ЛОГИКА АЛГОРИТМА. у вас к ЛОГИКЕ ОБРАБОТКИ ОШИБКИ относится вызов функции Log() который я убрал из логики алгоритма... И хватит... на это... достало - вы...это.... довольно неконструктивны....

Ну да, ну да. Это все равно что писать вместо вот такого:

Код: [Выделить]

if (a&&b&&c&&d) {...}писать такое:

Код: [Выделить]

if (a) {
    if (b) {
        if (c) {
            if (d) {
                ...
            }
        }
    }
}


[DIzer]

   Ага придуриваемся.... а что, мне тоже весело...

[valexey]

  Ага придуриваемся.... а что, мне тоже весело...

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

[DIzer]

  Если прочитать ВСЕ ваши требования (... по этому топику)... целей у вас... как бы это выразится  помягче .. достаточно много (например, избежание "радостного смешивания") и не все они разумные в  контексте налагаемых вами ограничений.. впрочем, по этому поводу я уже высказался... все.

[vlad]

Ну, defer есть :-)

Тогда я бы точно склонился к варианту с досрочными ретурнами. Дейкстра (точнее его толкование труЪ оберощиками) идет лесом.

[Губанов Сергей Юрьевич]

делать тупо в лоб

Советую делать тупо в лоб с многократно вложенными if, но только диагностику ошибки делать сразу, а не где-то там в else через сто строчек.

Код: [Выделить]

err = f1();
if (err != nil)
{
logError("f1 failed", err);
}
else
{
//...
//...
//...
err = f2();
if (err != nil)
{
logError("f2 failed", err);
}
else
{
//...
//...
//...
err = f3();
if (err != nil)
{
logError("f3 failed", err);
}
else
{
//...
//...
//...
err = f4();
if (err != nil)
{
logError("f4 failed", err);
}
else
{
//...
//...
//...
err = f5();
if (err != nil)
{
logError("f5 failed", err);
}
else
{
//...
//...
//...
}
}
}
}
}


[DIzer]

Ну, defer есть :-)

Тогда я бы точно склонился к варианту с досрочными ретурнами. Дейкстра (точнее его толкование труЪ оберощиками) идет лесом.

Одна проблема в Обероне07(с его ЕДИНСТВЕННЫМ  RETURN -ом ) вам не дадут даже сделать этого...

[DIzer]

делать тупо в лоб

Советую делать тупо в лоб с многократно вложенными if, но только диагностику ошибки делать сразу, а не где-то там в else через сто строчек.

Код: [Выделить]

err = f1();
if (err != nil)
{
logError("f1 failed", err);
}
else
{
//...
//...
//...
err = f2();
if (err != nil)
{
logError("f2 failed", err);
}
else
{
//...
//...
//...
err = f3();
if (err != nil)
{
logError("f3 failed", err);
}
else
{
//...
//...
//...
err = f4();
if (err != nil)
{
logError("f4 failed", err);
}
else
{
//...
//...
//...
err = f5();
if (err != nil)
{
logError("f5 failed", err);
}
else
{
//...
//...
//...
}
}
}
}
}


  А вот по Алексею  это НИЗЗЯ ибо  здесь "радостно смешиваются" две различные логики 

[vlad]

Советую делать тупо в лоб с многократно вложенными if, но только диагностику ошибки делать сразу, а не где-то там в else через сто строчек.

2valexey: я ж говорю, чисто вкусовые вариации - для меня, например, код Сергея абсолютно нечитаем

[vlad]

Одна проблема в Обероне07(с его ЕДИНСТВЕННЫМ  RETURN -ом ) вам не дадут даже сделать этого...

А там и defer нету Так что там, скорее всего, я бы выбрал вариант "с проверками до конца".

[DIzer]

Акаки там есть варианты акромя вашего (лестница)? 

[DIzer]

Кто нибудь знает , есть ли в каком -нибудь Обероне - нечто на вроде  AnyProc -указателя на   произвольную процедуру?