0

Есть у меня много методов, которые имеют такой вид

try {
   .....
} catch (const MyException & e) {
  return HandleError(e);
} catch (const std::exception & e) {
  return HandleError(e);
}

И функции обработки

HRESULT HandleError(const std::exception & ex, HRESULT code) {
    SetErrorInfo(ex.what());
    return code;
}

HRESULT HandleError(const std::exception & ex) {
    return HandleError(ex, E_UNEXPECTED);
}

HRESULT HandleError(const MyException & ex) {
    return HandleError(ex, ex.code);
}

Вопрос: а нельзя ли сократить catch блок до такого вида

try {
   .....
} catch (const std::exception & e) {
  return HandleError(e);
}

А в самой функции уже принимать решение о том, что это за исключение и что с ним делать

HRESULT HandleError(const std::exception & ex) {
    if (ex is MyException)  // нужен оператор проверки класса-наследника
        return HandleError(ex, dynamic_cast<MyException)(ex).code);
    else
        return HandleError(ex, E_UNEXPECTED);
}

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

1
  • dynamic_cast над указателем проверит класс наследника. 5 окт 2021 в 19:05

2 ответа 2

3

Это делается посредством выбрасывания и повторного перехвата текущего исключения в другом контексте:

try
{
   .....
}
catch (...)
{
   return Handle_Error();
}

HRESULT Handle_Error(void) noexcept
{
    try
    {
        throw;
    }
    catch (const MyException & e)
    {
        return e.code;
    }
    catch (const std::exception & e)
    {
        return E_UNEXPECTED;
    } 
    catch (...)
    {
        return E_UNEXPECTED;
    }
}
3
  • 1
    Если какой-то раззява вызовет Handle_Error() без активного исключения, случится std::terminate().
    – gbg
    5 окт 2021 в 19:19
  • Я бы назвал это ружьем Чехова-Мерфи-Голуба - "Если в первом акте программы на C++ на стене висит ружье, оно в неожиданный момент выстрелит по ногам самым необъяснимым и драматичным образом."
    – gbg
    5 окт 2021 в 19:30
  • @gbg Не зачем так драматизировать. Проблем будет не больше, чем с вызовом любой другой функции в неподходящем для этого месте. Скорее даже меньше, так как в таком сценарии программа будет сразу прервана. А тенденция к неправильному использованию такой специфичной функции - тем более меньше. 5 окт 2021 в 19:46
2

Конечно, dynamic_cast в помощь. Имея const std::exception &e, делаете вот так:

if (auto m = dynamic_cast<const MyException *>(&e)) {...}

Еще можно в HandleError ничего не передавать, и в ней делать try {throw;} и лесенку из catch-ей. Тогда в вызывающем коде можно просто писать catch (...) {return HandleError();}.

Если беспокоит возможность получить вылет, если вызвать такую функцию вне catch, то посмотрите в сторону std::current_exception - это замена throw;, но позволяет проверить, есть ли сейчас исключение.

3
  • а причем тут dynamic_cast?... 5 окт 2021 в 19:12
  • @ARHovsepyan Ну. Можно сделать catch (std::exception &e), потом if (MyException *m = dynamic_cast<MyException *>(&e)) {...} else {...}. 5 окт 2021 в 19:20
  • 1
    Ну это же нужно вписать в ответ(я так думаю)... 5 окт 2021 в 19:24

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.