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 ответов

19 просмотров

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

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

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

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

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

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

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

Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
И никого не интересует какие пакеты кто использует. ((% Заходишь на сайт симфони и видишь поддержку Украины - по законам РФ это ж экстремизм. Только никто не отказывается от с...
Am Ambrion
11
лучше скажите, причём тут паскаль?
Alexey Kulakov
36
Чтобы перехватить все нажимания буков на форме, надо хук ставить? Пробовал на форме ОнКейДаун, оно ловит клаву если фокус не на компоненте с вводом текста
Serjone
15
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
7
Народ! Впервые клиенту пришло письмо от РКН, у вас, дескать, есть яндекс метрика, а нигде не написано, что вы ее юзаете. Никто не сталкивался?
Sasha Beep
14
Всем привет! вывожу на общей стр дочерние ресурсыв каждом ресурсе галерея, и первая фотка должна выводиться на общей [!DocLister? &prepare=photo !]
Alekso
12
Я правильно понимаю что нет способов получить список ожидающих заявок на вступление в группу с помощью бота из mtproto?
Шамиль Прилов
7
А можно вопрос? Мне сегодня сказали что у меня функция (которая просто заполняет массив значениями) не правильная void Full(double * arr, int n) { for (int i = 0; i < n; i...
† C E †
7
Добрый вечер. Хочу чтобы у меня в классе поле было функцией, которая возвращает строку. Делаю так: interface ... TGetOutPath = function : String of object; ... protec...
Kirill Filippenok
12
Карта сайта