0

Это весь код функции. Может кто-нибудь мне объяснить, при каких, гипотетически, условиях переменной response может быть не присвоено значение?

Или же это просто причуды vs?

string method = ExtractMethod(context.Request.RawUrl ?? string.Empty);
IHandlerFactory handlerFactory = GetHandlerFactory(method);
IHandler handler;
Response response;
try
{
    handler = handlerFactory.Create(context);
    response = handler.ProcessRequest();
}
catch (Exception e)
{
    Logger.Log(LogSeverity.Error, nameof(HttpHandler), "Error", e);
    response = new()
    {
        Exception = CoreException.UnknownException
    };
}
finally
{
    JsonSerializer.Serialize(context.Response.OutputStream, response);
    context.Response.OutputStream.Close();
}

Скриншот из студии

5
  • например, если у вас вылетает эксепшон тут handler = handlerFactory.Create(context);, а потом тут Logger.Log(LogSeverity.Error, nameof(HttpHandler), "Error", e);, то в блоке finally непонятно какое значение будет у response. Чтобы всем было всё понятно, положите туда сразу что нибудь, хотя бы просто null Response response = null;
    – tym32167
    16 мая 2023 в 18:50
  • а разве если в блоке catch вылетит исключение, оно не остановит выполнение всей программы?
    – KLGR
    16 мая 2023 в 18:55
  • finally уберите вообще, смысл здесь в нем какой, если catch ловит всё что можно?
    – aepot
    16 мая 2023 в 18:58
  • резонно вообще. ну вопрос я решил инициализацией response изначально, как и посоветовал tym32167. просто интересная ситуация, с которой я раннее ни разу не сталкивался, поэтому и стало интересно. видимо try-catch немного не так обрабатывается анализатором, как то же if-else, например, вот и чудит.
    – KLGR
    16 мая 2023 в 19:02
  • Если response не присвоено значение, то нет смысла сериализовать его. Добавить в finally проверку if (response != null)? 16 мая 2023 в 22:07

1 ответ 1

1

Если у вас в блоке try и catch вылетают исключения, то finally все равно выполнится. Чтобы было понятно что делать с response в этом случае, положите туда сразу что нибудь, хотя бы просто Response response = null;

Пример

    try
    {           
        throw new Exception("oops");
    }
    catch (Exception ex)
    {
        throw new Exception("oops again");      
    }
    finally
    {
        Console.WriteLine("Im finally block!");
    }

Результат

введите сюда описание изображения

4
  • кстати, у меня почему-то все равно падает ислючение, и просто все останавливается. и finally в том числе. и в дебаге и в релизе... просто печатает сообщение об ошибке и виснет.
    – KLGR
    16 мая 2023 в 19:13
  • ну так второе исключение же не ловится. Заверните всю конструкцию в еще один большой try-catch i.stack.imgur.com/5YxGg.png
    – tym32167
    16 мая 2023 в 19:16
  • хм... ну это уже другой совершенно разговор. впрочем, спасибо за ответ)
    – KLGR
    16 мая 2023 в 19:18
  • в моем примере из ответа первое исключение ловится конструкцией catch. А второе не ловится ничем, отчего у вас программа падает. А finally вызывается в любом случае, не важно где ловится или не ловится исключение, не важно было ли исключение вообще, finally вызовится все равно.
    – tym32167
    16 мая 2023 в 19:25

Ваш ответ

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

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