O
Совершенно непонятный поток исполнения
Size: a a a
2020 March 18
TS
Если база отвалилась, шлем один тип ошибки, если результат с бд вернул что-то шлем другую ошибку
if (flag)
{
///
return result;
}
return Error;RM
if (flag)
{
///
return result;
}
return Error;Там ниже еще код
O
Что функция возвращает? У тебя там, кажется, UB
TS
Там ниже еще код
разбей на функции
RM
Что функция возвращает? У тебя там, кажется, UB
Какая функция?
TS
Что функция возвращает? У тебя там, кажется, UB
у него там функция на сотню строк кода
O
Какая функция?
O
у него там функция на сотню строк кода
Не, такое я ревьювить не буду
RM
Они bool возвращают
RM
Результат отправки пакета
O
Они bool возвращают
*не возвращают, ты хотел сказать?
BB
Не, такое я ревьювить не буду
Как говорил мой препод по плюсам - больше семи строк - плохая функция
RM
*не возвращают, ты хотел сказать?
В смысле?
O
В смысле?
Я красным показал потенциальный поток исполнения, я не вижу в нём
returnTS
value() делает оверхед, ты уже проверил, что значение есть с помощью has_value
RM
Я красным показал потенциальный поток исполнения, я не вижу в нём
returnаа нее, это просто скоуп
RM
value() делает оверхед, ты уже проверил, что значение есть с помощью has_value
value возвращает объект внутри optional
RM
has_value проверяет вернули мы std::nullopt или объект есть
TS
value возвращает объект внутри optional
и еще раз делает проверку, что optional задан