gkislin Sep 28 2021 at 13:52

Рекомендации по реализации тестового задания (и не только): Java проект с REST API

6 min

12K

Programming*Java*Studying in ITIT career

Tutorial

Comments 18

dlazerka Sep 28 2021 at 14:04

Рекомендую писать проект современно: на Java это стек: Spring Boot + Spring Data JPA

Эмм, не совсем современно, хоть и популярно. Spring тянет с собой медленный reflection stack (все ключи бинов в итоге приводятся к строке). Современно -- это Dagger 2, который генерирует dependency injection во время компиляции, так что JIT проще, и можно скомпилировать код.

Вообще современно сейчас деплоить приложения в контейнерах легковесных, где довольно важно startup time. Отчасти из-за этого, по моим наблюдениям, Java в последнее время стали всё больше компилировать с помощью GraalVM. А там dependency injection ahead-of-time очень даже помогает.

Для веб сервера Jetty конечно очень популярен, не спорю, но я бы сказал сейчас современно писать на обёртках Netty (из-за async processing), например http4k или ktor для Котлина. Они тоже компилируются с GraalVM.

Для all-around frameworks (по крайней мере в Долине) популярны Dropwizard и Micronaut.

А JPA уже точно устарело. JOOQ самый популярный из современных, я бы сказал. Опять же таки из-за кодогенерации во время билда, как и Dagger.

gkislin Sep 28 2021 at 15:15

Спасибо за комментарий! Поправил на "востребованном на рынке стеке ". Действительно, если современный воспринимать как самый передовой, то это не Spring Boot. А популярность фреймворка легко оценить для России в вакансиях на hh или по обзорам: https://www.jrebel.com/blog/2021-java-technology-report, https://snyk.io/jvm-ecosystem-report-2021/ - Spring Boot - 58-62%, Dropwizard 4%-9%, Micronaut 4%-5%. Dagger отсутствует.
Spring Data - это работа с БД по умолчанию в Spring Boot (причем не только с реляционными). Поэтому что "точно устарело" и JOOQ популярнее- категорически не соглашусь. Ну и еще раз именно про тестовые приложения на вакансии - чем у вас все будет стандартнее для разработки и совпадать со стеком компании, тем выше оценка.

ris58h Sep 28 2021 at 16:23

taluyev Sep 29 2021 at 01:15

6.4: Обновление в базе делается через update - не стоит смешивать JPA и DAO. апдейт делать не надо и сейв загруженного POJO также не надр - достаточно лишь изменить поля на необходимые значения, остальную работу сделает JPA. JPA и есть "паттерн".

gkislin Sep 29 2021 at 08:57

Спасибо за плодовитость комментариев:)
6.4 Я в курсе, что в загруженных entity при изменении полей save не нужен. Речь идет о логике, когда ради экономии пары строчек Java люди объединяют save и update примерно так:

if(updateCondition)
repository.delete(entity) } repository.save(entity)

taluyev Sep 29 2021 at 01:19

7.5: Поля базы case insensitive, не пишите camelStyle - в обратных одинарных кавычках для мариядэбэ или майскл кейс сенситив, для оракла в двойных кавычках уейс сенситив, для мсскл в квадратных скобках кейс сенситив. поля можно указать в маппинге сущьностей. Поля кейс сенситив...

gkislin Sep 29 2021 at 08:58

Я именно об этом- не надо camelCase и двойных кавычек вместе с ними

taluyev Sep 29 2021 at 01:21

7.6: Таблицы я предпочитаю именовать в единственном числе. Исключение - users, т. к. user - зарезервированное слово - order и другие также лучше в множественном.

taluyev Sep 29 2021 at 01:23

8.2: Я предпочитаю четкое разделение ролей на основе URL. Например, для админа URL содержит /admin - спорное утверждение, не имеющее отношения к праыильному выполнению тестового задания.

gkislin Sep 29 2021 at 09:07

Именно поэтому я написал - "Я предпочитаю"

taluyev Sep 29 2021 at 01:29

10.2: Проверяйте входные данные Primary Key при create/update в контроллерах. - этим может заниматься база данных, а Джава (слой данных) возбуждать исключение и возврвщать клиенту, например "бэд реквест". Без блокировки не существующих записей (да, это можно сделать) это не сделать консистентно, либо нужно изменять уровень изоляции транзакций, а это уменьшает масштабируемость приложения. Проще на это не обращать внимание, за вас сделает эту работу база данных и джпа/спринг.

gkislin Sep 29 2021 at 09:03

Это неверно. При POST надо проверять id==nullобъекта, а при PUT соответствие id в URL и в объекте в теле запроса (null допускается). База этого не сделает и проверка должна быть именно в контроллере- он отвечает за проверку входных данных

taluyev Sep 29 2021 at 01:35

11.1: JUnit-тесты очень желательны. Можно не делать 100% покрытие, только основные сценарии - junit тесты надо писать без поднятия контекста. С поднятием контекста - это лучше относить к интеграционным тестам. Почему? Потому, что тесты с контестом работают медленно, лучше мказать медленно поднимается контекст. Тесты дата-лейера тем более это интеграционные тесты. Как писать на спринге и не поднимать его в юнмт тестах? надо использовать библиотеку моков, например мокито.

gkislin Sep 29 2021 at 09:06

В Spring Boot есть большое количество подходов к тестированию: https://javaops.ru/view/bootjava/lesson06#test
Все они имеют право на использование.

kacetal Sep 29 2021 at 14:31

Если тестов много то это не так заметно, так как спринг может кешировать контекст.

gkislin Sep 29 2021 at 23:19

Все так, по умолчанию кэширует. На достаточно скромной машине порядка 100 тестов в минуту через MockMvc меня устраивает.

gkislin Oct 2 2021 at 18:26

Поправка: - на стажировке у нас 133 теста, у меня выполняются за 18 секунд.

taluyev Sep 29 2021 at 01:37

ОБЯЗАТЕЛЬНО: запусти mvn test - ошибок быть не должно - mvn verify тоже ошибки быть не должно при запуске интеграционных тестов.