169 похожих чатов

@Service Public DeconstructResponse destroyBuilding(DeconstructRequest request) throws NoSuchBuildingException {

var building = constructedBuildingsRepository.findById(request.getConstructedBuildingId());
if(building.isEmpty()) throw new NoSuchBuildingException("No building with specified id!");
else
{
constructedBuildingsRepository.delete(building.get());
return new DeconstructResponse(request.getConstructedBuildingId());
}

}
----------------------------------
@DeleteMapping("/deconstruct")
private ResponseEntity<?> deconstructBuilding(DeconstructRequest request) {
try
{
var response = buildingService.destroyBuilding(request);
return new ResponseEntity<>(response, HttpStatus.OK);
}
catch (NoSuchBuildingException exception) {
return new ResponseEntity<>(exception.getMessage(), HttpStatus.BAD_REQUEST);
}
}
---------------
Всем привет! У меня вопрос по поводу оформления кода. Это ок в сервисах кидать ошибки на некоторые ситуации (неправильный пароль, неизвестное имя и тд) и потом отлавливать их в контроллере и кидать соответствующее ответы? Просто это все-таки замедляет программу. Какой подход обычно юзается в продакшене?

7 ответов

24 просмотра

Использовать isEmpty() и get() антипаттерн, лучше использовать ifPresentOrElse() или в твоём случае orElseThrow()

Alexandr ∨∧‾ Emelyanov
Использовать isEmpty() и get() антипаттерн, лучше ...

А есть какие-нибудь ресурсы (статьи или видео, на русском или английском), чтоб почитать про текущие антипаттерны?

Alexandr ∨∧‾ Emelyanov
Так там другой метод использовался

Подскажите пожалуйста, почему нельзя использовать !isEmpty()?

Похожие вопросы

Обсуждают сегодня

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта