Conversation
Add common films
…orate with conflicts.
Add and realization branch add-recommendations
…orate with conflicts.
# Conflicts: # src/main/java/ru/yandex/practicum/filmorate/storage/film/FilmDbStorage.java
…va-filmorate into add-director # Conflicts: # src/main/java/ru/yandex/practicum/filmorate/storage/film/FilmDbStorage.java
Merge branch 'add-director'
сделал add-most-populars
Merge add-reviews
# Conflicts: # src/main/java/ru/yandex/practicum/filmorate/controller/UserController.java # src/main/java/ru/yandex/practicum/filmorate/service/FilmService.java # src/main/java/ru/yandex/practicum/filmorate/service/UserService.java # src/main/resources/schema.sql
update add review feed
Add remove endpoint
| @GetMapping("/popular") | ||
| public List<Film> getPopularFilms(@RequestParam(defaultValue = "10") int count) { | ||
| return filmService.getMostPopularFilms(count); | ||
| public List<Film> getPopularFilms(@RequestParam(name = "count", defaultValue = "10") Integer count, |
There was a problem hiding this comment.
Если имя параметра реквеста совпадает с именем параметра метода, то писать name = "count" избыточно.
| } | ||
|
|
||
| @ExceptionHandler(ConditionsNotMetException.class) | ||
| @ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR) |
There was a problem hiding this comment.
Здесь лучше BAD_REQUEST, т.к. судя по коду данное исключение выбрасывается при проверке входных данных.
| e.getMessage() | ||
| e.getMessage(),"Ошибка со стороны сервера" | ||
| ); | ||
| } |
There was a problem hiding this comment.
Нет обработки IllegalArgumentException
| e.getMessage(),"Ошибка со стороны сервера" | ||
| ); | ||
| } | ||
| } |
There was a problem hiding this comment.
Нужно также добавить обработку всех оставшихся исключений, которые могут потенциально возникнуть - Throwable в статус INTERNAL_SERVER_ERROR
| @@ -191,6 +350,15 @@ private void saveFilmGenres(Film film) { | |||
| } | |||
There was a problem hiding this comment.
В цикле вставлять по одной записи не оптимально, лучше воспользоваться пакетной операцией.
https://habr.com/ru/articles/703828/
VadimZharkov
left a comment
VadimZharkov
left a comment
There was a problem hiding this comment.
Проделана первая большая совместная работа. В целом все сделано верно. Понравилась схема даннных и структура проекта. Оставил комменты, что можно улучшить.
…ndlers, use JDBC batchUpdate
|
@VadimZharkov Вадим, благодарю за ценную обратную связь. Мы с командой обновили код согласно правкам. |
5 participants
No description provided.