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

Всем привет! Подскажите пожалуйста. Вопрос по обновлениям (и созданиям) сущностей. Такая

ситуация:

Допустим мы хотим обновить машину, у машины есть колеса и фары...
Мы хотим заменить колёса и фары, указываем новые id колес и фар.

Во время обновления сущности (или перед обновлением) необходимо делать валидацию связынных сущностей (фар и колес, может таковых нет уже в БД).

У меня сейчас в проекте есть кастомный валидатор, в котором как раз и проверяются и фары и колёса.
Этот валидатор запускается первой строчкой в сервисе в методе updateCar (но можно его сделать конечно и через AOP, не суть)

в валидаторе метод:
void checkКолесо(Long Id) { repository.findById(1L).orElseThrow () -> new КастомнаяОшибка} (или existsById(1L) )

то есть при таком подходе у нас сервис даже не запустится, если сущность не будет найдена, так-как все проверки будут в валидаторе.

Но дело в том что после успешного прохождения проверок в валидаторе, начинается выполнятся сервис, а там в конвертере (маппере) идут setЫ сущностей, то есть опять repository.findById(1L).orElseThrow() (только уже без кастомной ошибки, т.к. оно отлавливается в валидаторе и сущность уже хранится в кеше хибернейта).

Норм вообще такой подход?)

Варианты:
1) оставить так как есть.
2) убрать поиск сущностей в валидаторе, этого достаточно в конвертерах, пусть ошибка если не нашли падает там, ничего страшного что побежит логика сервиса и где нить потом упадет, мы ведь используем транзакцию
3) оставить проверки сущностей в валидаторе, не делать проверки в конвертерах, просто в конвертерах можно использовать вариант setWheel(new Well.setId(...))

Старался более подробно и понятно описать ситуацию, хочется узнать, как правильней лучше, какой подход чаще используется)

4 ответов

17 просмотров

а можно убрать лези лоадинг чтобы впринципе в конверторах ничо не вызывалось?) Третий вариант самый адекватный

нет смысла валидировать корректность внешних ключей на этапе преобразования или в сервисах, проверки внешних ключей выполняются на этапе коммита транзакции(даже не на этапе flush'а) собирай сущность как есть и пытайся коммитить транзакцию, если не сработало - обрабатывай ошибку уже вне транзакции вот тут есть речь об этом https://youtu.be/YzOTZTt-PR0?t=2351

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

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

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
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
Карта сайта