Всем привет. Подскажите, пожалуйста, как можно улучшить данный код? studentJPARepository.findAll() -

тут достаю все значения из бд. По идее это плохая практика.
На названия вроде XXX не смотрите, они были изменены

ArrayList<StudentType> studentType = type.getStudentType();

List<Student> studentList= studentJPARepository.findAll()
.stream()
.filter(student-> {
for (StudentType : studentType ) {
switch (type.getValue()) {
case "XXX":
if (student.worksAsXXX()) return true;
break;
case "YYY":
if (student.worksAsYYY()) return true;
break;
case "ZZZ":
if (student.worksAsZZZ()) return true;
break;
case "HHH":
if (Boolean.TRUE.equals(student.getIsHHH())) return true;
break;
}
}
return false;
})
.distinct()
.toList();

какая исходная задача отфильтровать данные?

В эндпоинте приходит массив со значениями: XXX, YYY, ZZZ (пример). Этих значений нет в бд, но они проверяются с помощью методов worksAsXXX() и т.д. Массив может быть пустым, может содержать все значения

А ну, по сути отфильтровать да.

как минимум вынести фильтр в отдельный метод или предикат.

это позволит меньше запросов в бд послать?

так запрос и так один вроде

Ну да, точно. Просто база огромная и мне кажется, что тянуть из базы через findAll() не очень хорошо

ну может можно как-то еще отфильтровать, косвенно из этих worksAsXXX получить что-то для фильтрации на стороне бд

А @Query запросы прописывать в интерфейсе Repository плохая практика?

Нормальная. Где их ещё писать, если не в репозитории?

Так получится, что большую часть логики можно реализовать через @Query - ну именно для работы с бд? А какие подводные камни? К примеру код используется и для веб приложения и для мобильной версии. Я просто в Java и бд не разбираюсь особо, просто помогаю разобраться в легаси коде без документации

Подводный камень - перфоманс, что касается любой орм

И нет разницы, где ты потом эти данные будешь отображать - в браузере или на мобилке. Бэк-то один

Используйте спецификации и пагинацию

Блин, я думала на счет спецификаций, но это очень сложно для меня

Они на самом деле очень легкие

Можешь доставть не все значения а только те что нужны. Делать запрос нативный в базе, и собирать в дто

там с составлением фильтра проблема)

Фильтр есть фильтр у него дело малое ) но важное !

А нужны те, что в фильтре. Но я уже разобралась. Получилось с помощью Query

А что это за спецификации такие?

Спецификации больше накостыль похожи, так ещё и не полностью интегрируемые в data jpa