Vi3arD spring-course-task-2-web-mvc issues

Vi3arD / spring-course-task-2-web-mvc

0 stars 0 forks source link

issues

Newest

Newest Most commented Recently updated Oldest Least commented Least recently updated

Может ли тут быть NULL?

#22 coursar closed 3 years ago
1
Где проверяется, что пользователь работает именно со своим запросом? Который он сам зарегистрировал. Может ли он при текущей реализации посмотреть/отменить любой "заказ"?

#21 coursar closed 3 years ago
0
Тесты лучше писать хотя бы в таком виде, иначе как вы проверите, что всё работает? Заново руками будете запускать?

#20 coursar closed 3 years ago
2
Code duplication, возможно стоит подумать о том, чтобы вынести в утилитные методы и переиспользовать

#19 coursar closed 3 years ago
0
Если выше (на уровне класса) не пишите value, то и тут без лишней необходимости не пишите

#18 coursar closed 3 years ago
0
Лучше не делить на две строки, если всё, что вы делаете с Optional'ом - вытаскиваете значение, либо выкидываете Exception

#17 coursar closed 3 years ago
0
Лучше нормально именовать, userService

#16 coursar closed 3 years ago
0
Зачем этот метод?

#15 coursar closed 3 years ago
0
Почему отмена - это delete? По-хорошему, это именно отмена, а не удаление

#14 coursar closed 3 years ago
0
Bad practice - no {}

#13 coursar closed 3 years ago
0
Эксепшены должны быть с суффиксом Exception

#12 coursar closed 3 years ago
0
Лучше предусмотреть специализированное исключение

#11 coursar closed 3 years ago
0
Не очень понятно, кто всё-таки отвечает за генерацию логических исключений - сервис или репозиторий?

#10 coursar closed 3 years ago
0
Где логика с "Если указать в параметре amount ноль, завершение произойдёт на всю предавторизованную сумму."?

#9 coursar closed 3 years ago
3
В оригинальном сервисе ответы другие

#8 coursar closed 3 years ago
0
url'ы и параметры не совпадают с оригинальным сервисом

#7 coursar closed 3 years ago
2
Если набор статусов фиксирован (не динамически выставляется клиентом), то их надо или в final String, либо в enum

#6 coursar closed 3 years ago
0
Желательно, чтобы методы сервиса маппились на бизнес-логику. Сейчас он больше похож на CRUD-сервис

#5 coursar closed 3 years ago
2
Всё-таки должно быть pay (или, если делать мимику оригинального сервиса, deposit)

#4 coursar closed 3 years ago
0
Насколько логично в контексте задачи давать возможность обновлять всю запись целиком?

#3 coursar closed 3 years ago
1
Добавить лимит, offset и order by

#2 coursar closed 3 years ago
2
Зачем в заказе хранить логин-пароль клиента?

#1 coursar closed 3 years ago
3