Исключения

[_Bers]

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

В показанном примере исключение выкидывает send_packet, а в send_packets нет никакой информации.

send_packet для красоты был использован?
или программист, который писал его вызов,
знал для чего он это делал?

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

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

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

разумеется виднее.

одному персонажику из этой самой темы было очень удобно
ловить эксепшен вылетевший из уровня Z, на уровне A.
а потом разводить руками:
"а что я тут ещё могу сделать? только заллогировать"

наверное, ему тоже было "виднее".

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

плохой код:

void send_packets(const vector<data> &packets)
{
   for(const auto &packet: packets)
      send_packet(packet);
}

хороший код:

void send_packets(const vector<data> &packets)
{
   for(const auto &packet: packets)
      if(!send_packet(packet))
          on_error(packet);
}

[waleri]

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

одному персонажику из этой самой темы было очень удобно

А нельзя ли повежливей?

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

Речь шла о возвращении кода ошибки vs исключение. В вашем примере "хорошего" кода возвращения кода ошибки я не наблюдаю - ну и в чем разница? Ну да, вы обрабатываете ошибку внутри, ай маладец. А что делаем если ошибка возникла в on_error? При вашем подходе, верхних слой вообще не может узнать что есть ошибка - функция объявлена void.

[Alex11223]

Кстати, раз уж тут обсуждали все варианты использования исключений, то еще есть такой (не знаю популярен ли в С++, я использовал в одном проекте, брат и кот пока живы): OperationCancelledException для прерывания работы потока/операции.

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

void throwIfCancelled() const
{
    if (cancelled_)
        throw OperationCancelledException();
}

и вызывать где надо

void doWork()
{
    ...
    loadInfo(...);
    ...
    downloadFile(...);
    ...
    uploadFile(...);
}

void loadInfo(...)
{
    login();

    throwIfCancelled();

    ...
    const auto items = loadItems();
    for (const auto item : items)
    {
        throwIfCancelled();

        loadItemInfo(item);
        ...
    }
}

void downloadFile(...)
{
    ...
    downloadFileHttp(...);
}

void downloadFileHttp(...)
{
    ...
    while (localSize < remoteSize)
    {
        throwIfCancelled();

        read...(..., BUF_SIZE);
    }
}

Ну и естественно ловить где-то наверху.

void threadFunc()
{
    try
    {
        doWork();

        onSuccess();
    }
    catch(OperationCancelledException&)
    {
        onCancelled();
    }
    catch(Exception& ex)
    {
        onError(ex.message());
    }
    catch(...)
    {
        onError("Uncaught exception ......");
    }
    onFinished();
}

Что великие гуру думают про это? Есть ли варианты лучше?

[Croessmah]

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

хороший код:

Смысл от проверки, если send_packet кидает исключение? )))