Зачем лишний селект? В эксепшн нужен ид ентити иначе он бесполезен
Селект я сделал, чтобы проверить - есть ли такая ссылка в базе. Или лишнее все равно?
базе и библиотекам обычно пофиг, если ты пытаешься удалить несуществующее Тебе не пофиг? Если тебе важно среагировать как-то на попытку удалить несуществующее, то не лишнее
если селект всё-таки нужен, то я бы вынес ....getId() в переменную Потому что вложенность скобок и чейнинг читать нужно в разные стороны. Удобно, когда они не пересекаются в одном выражении. Одна строчка — чейнинг, другая — вложенные скобочки
У меня обработка ошибок настроена. То есть если что-то выкинет какой-то явно неотлавливаемый мной экспекшн, то я все равно его поймаю и кину чайника. Но в твоих словах есть смысл. Попробую удалить несуществующее и посмотрю как себя поведет приложение
в спринге, например, deleteAllById() не кинет эксепшен
А просто deleteById, полагаю, тоже не выкинет
Обсуждают сегодня