sprint13/develop

README.md

@@ -1,14 +1,21 @@
-# java-filmorate (Java Filmorate)
-### Описание (Description)  
-The Filmorate project — это приложение для работы с базой данных фильмов.
+# java-filmorate
+The Filmorate project — это приложение для работы с базой данных фильмов. 
+А также площадка для общения и взаимодействия пользователей.
+
 ### Основные возможности
-- Хранение данных о фильмах
+- Хранение данных о фильмах  
 - Хранение данных о пользователях
 - Возможность пользователям ставить лайки фильмам
 - Возможность пользователям добавлять друг друга в друзья
 - Возможность получать рейтинг TOP_N фильмов по количеству лайков
+- Возможность получать общих с другим пользователем фильмов с сортировкой по их популярности
+- Возможность заводить друзей
+- Возможность оставлять отзывы о фильмах
 - Фильмы содержат возрастные ограничения по стандарту MPPA
+- Реализован поиск по названию и режиссерам
 - Реализована система жанров для фильмов
+- Есть лента событий
+
 ### Требования (Environmental Requirements) 
 Для запуска проекта требуется: - Java версии 21  
 Приложение работает на порту 8080
@@ -48,7 +55,7 @@ The Filmorate project — это приложение для работы с б
 - logback-classic 
 - logbook-spring-boot-starter
 ### Схема базы данных (Database map)
-![DatabaseMap](QuickDBD-FILMO_RATE.png "Database map:")
+![DatabaseMap](FILMO_RATE_DATABASE_MAP.png "Database map:")
 ### Примеры запросов к базе данных:
 1. Get TOP-10 films:
 ```sql

src/main/java/ru/yandex/practicum/filmorate/FilmorateApplication.java

@@ -2,10 +2,11 @@
 
 import org.springframework.boot.SpringApplication;
 import org.springframework.boot.autoconfigure.SpringBootApplication;
+import org.springframework.boot.autoconfigure.orm.jpa.HibernateJpaAutoConfiguration;
 
-@SpringBootApplication
+@SpringBootApplication(exclude = {HibernateJpaAutoConfiguration.class})
 public class FilmorateApplication {
-    public static void main(String[] args) {
-        SpringApplication.run(FilmorateApplication.class, args);
-    }
+	public static void main(String[] args) {
+		SpringApplication.run(FilmorateApplication.class, args);
+	}
 }

src/main/java/ru/yandex/practicum/filmorate/controller/DirectorController.java

@@ -0,0 +1,43 @@
+package ru.yandex.practicum.filmorate.controller;
+
+import jakarta.validation.Valid;
+import lombok.RequiredArgsConstructor;
+import org.springframework.web.bind.annotation.*;
+import ru.yandex.practicum.filmorate.dto.DirectorDto;
+import ru.yandex.practicum.filmorate.dto.request.create.DirectorCreateRequest;
+import ru.yandex.practicum.filmorate.dto.request.update.DirectorUpdateRequest;
+import ru.yandex.practicum.filmorate.service.DirectorService;
+
+import java.util.Collection;
+
+@RequestMapping("/directors")
+@RestController
+@RequiredArgsConstructor
+public class DirectorController {
+	private final DirectorService directorService;
+
+	@GetMapping
+	public Collection<DirectorDto> getAllDirectors() {
+		return directorService.getAllDirectorDtos();
+	}
+
+	@GetMapping("/{directorId}")
+	public DirectorDto getDirectorById(@PathVariable Integer directorId) {
+		return directorService.getDirectorDtoById(directorId);
+	}
+
+	@PostMapping
+	public DirectorDto createDirector(@RequestBody @Valid DirectorCreateRequest directorRequest) {
+		return directorService.create(directorRequest);
+	}
+
+	@PutMapping
+	public DirectorDto updateDirector(@RequestBody @Valid DirectorUpdateRequest directorRequest) {
+		return directorService.update(directorRequest);
+	}
+
+	@DeleteMapping("/{directorId}")
+	public void deleteDirector(@PathVariable Integer directorId) {
+		directorService.delete(directorId);
+	}
+}

src/main/java/ru/yandex/practicum/filmorate/controller/ErrorHandler.java

@@ -4,14 +4,15 @@
 import org.springframework.http.HttpStatus;
 import org.springframework.http.MediaType;
 import org.springframework.http.ResponseEntity;
+import org.springframework.http.converter.HttpMessageNotReadableException;
 import org.springframework.validation.FieldError;
 import org.springframework.validation.method.MethodValidationException;
 import org.springframework.web.bind.MethodArgumentNotValidException;
 import org.springframework.web.bind.annotation.ExceptionHandler;
 import org.springframework.web.bind.annotation.ResponseStatus;
 import org.springframework.web.bind.annotation.RestControllerAdvice;
-import ru.yandex.practicum.filmorate.model.ErrorResponse;
 import ru.yandex.practicum.filmorate.exception.*;
+import ru.yandex.practicum.filmorate.model.ErrorResponse;
 
 import java.util.HashMap;
 import java.util.Map;
@@ -83,6 +84,12 @@ public ErrorResponse handlerMethodValidationException(final MethodValidationExce
 		return new ErrorResponse(e.getAllErrors().getFirst().getDefaultMessage());
 	}
 
+	@ResponseStatus(HttpStatus.BAD_REQUEST)
+	@ExceptionHandler
+	public ErrorResponse handleEmptyBody(HttpMessageNotReadableException ex) {
+		return new ErrorResponse("Тело запроса отсутствует или некорректно");
+	}
+
 	@ExceptionHandler(MethodArgumentNotValidException.class)
 	public ResponseEntity<Map<String, String>> handleMethodArgumentException(final MethodArgumentNotValidException e) {
 		Map<String, String> errors = new HashMap<>();

src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java

@@ -4,54 +4,80 @@
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.HttpStatus;
 import org.springframework.web.bind.annotation.*;
-import ru.yandex.practicum.filmorate.dto.request.create.FilmCreateRequest;
 import ru.yandex.practicum.filmorate.dto.FilmDto;
+import ru.yandex.practicum.filmorate.dto.request.create.FilmCreateRequest;
 import ru.yandex.practicum.filmorate.dto.request.update.FilmUpdateRequest;
-import ru.yandex.practicum.filmorate.servise.FilmService;
-import ru.yandex.practicum.filmorate.servise.util.LikeAction;
+import ru.yandex.practicum.filmorate.service.FilmService;
+import ru.yandex.practicum.filmorate.service.util.LikeAction;
 
 import java.util.Collection;
 
+@RequestMapping("/films")
 @RestController
 @RequiredArgsConstructor
-@RequestMapping("/films")
 public class FilmController {
 
-    private final FilmService filmService;
-
-    @GetMapping
-    public Collection<FilmDto> findAll() {
-        return filmService.findAll();
-    }
-
-    @GetMapping("/{id}")
-    public FilmDto findById(@PathVariable long id) {
-        return filmService.findById(id);
-    }
-
-    @GetMapping("/popular")
-    public Collection<FilmDto> getTop(@RequestParam (defaultValue = "10", required = false) Integer count) {
-        return filmService.getTopFilms(count);
-    }
-
-    @PostMapping
-    @ResponseStatus(HttpStatus.CREATED)
-    public FilmDto create(@RequestBody @Valid FilmCreateRequest request) {
-        return filmService.create(request);
-    }
-
-    @PutMapping
-    public FilmDto update(@RequestBody FilmUpdateRequest request) {
-        return filmService.update(request);
-    }
-
-    @PutMapping("/{filmId}/like/{userId}")
-    public void setLike(@PathVariable long filmId, @PathVariable long userId) {
-        filmService.changeLike(LikeAction.SET, filmId, userId);
-    }
-
-    @DeleteMapping("/{filmId}/like/{userId}")
-    public void deleteLike(@PathVariable long filmId, @PathVariable long userId) {
-        filmService.changeLike(LikeAction.REMOVE, filmId, userId);
-    }
-}
+	private final FilmService filmService;
+
+	@GetMapping
+	public Collection<FilmDto> findAll() {
+		return filmService.findAll();
+	}
+
+	@GetMapping("/{id}")
+	public FilmDto findById(@PathVariable long id) {
+		return filmService.findById(id);
+	}
+
+	@GetMapping("/popular")
+	public Collection<FilmDto> getTop(@RequestParam(defaultValue = "10", required = false) Integer count,
+	                                  @RequestParam(required = false) Integer genreId,
+	                                  @RequestParam(required = false) String year) {
+		return filmService.getTopFilms(count, genreId, year);
+	}
+
+	@GetMapping("/common")
+	public Collection<FilmDto> getCommonLikedFilms(@RequestParam(required = true) Long userId,
+	                                               @RequestParam(required = true) Long friendId) {
+		return filmService.getCommonLikedFilms(userId, friendId);
+	}
+
+	@GetMapping("/director/{directorId}")
+	public Collection<FilmDto> findSortedFilmsOfDirector(
+			@PathVariable Integer directorId,
+			@RequestParam(required = false) String sortBy
+	) {
+		return filmService.getSortedFilmsOfDirector(directorId, sortBy);
+	}
+
+	@GetMapping("/search")
+	public Collection<FilmDto> search(@RequestParam String query, @RequestParam String by) {
+		return filmService.search(query, by);
+	}
+
+	@ResponseStatus(HttpStatus.CREATED)
+	@PostMapping
+	public FilmDto create(@RequestBody @Valid FilmCreateRequest request) {
+		return filmService.create(request);
+	}
+
+	@PutMapping
+	public FilmDto update(@RequestBody FilmUpdateRequest request) {
+		return filmService.update(request);
+	}
+
+	@PutMapping("/{filmId}/like/{userId}")
+	public void setLike(@PathVariable long filmId, @PathVariable long userId) {
+		filmService.changeLike(LikeAction.SET, filmId, userId);
+	}
+
+	@DeleteMapping("/{filmId}/like/{userId}")
+	public void deleteLike(@PathVariable long filmId, @PathVariable long userId) {
+		filmService.changeLike(LikeAction.REMOVE, filmId, userId);
+	}
+
+	@DeleteMapping("/{filmId}")
+	public void deleteFilm(@PathVariable long filmId) {
+		filmService.remove(filmId);
+	}
+}

src/main/java/ru/yandex/practicum/filmorate/controller/GenreController.java

@@ -6,12 +6,12 @@
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 import ru.yandex.practicum.filmorate.dto.GenreDto;
-import ru.yandex.practicum.filmorate.servise.GenreService;
+import ru.yandex.practicum.filmorate.service.GenreService;
 
 import java.util.Collection;
 
-@RestController
 @RequestMapping("/genres")
+@RestController
 @RequiredArgsConstructor
 public class GenreController {
 	private final GenreService genreService;

src/main/java/ru/yandex/practicum/filmorate/controller/HomeController.java

@@ -4,43 +4,43 @@
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
-@RestController
 @RequestMapping("")
+@RestController
 public class HomeController {
 
-    @GetMapping
-    public String homePage() {
-        return """
-                <p><br></p>
-                <pre>
-                    ███████╗██╗██╗     ███╗   ███╗ ██████╗     ██████╗  █████╗ ████████╗███████╗
-                    ██╔════╝██║██║     ████╗ ████║██╔═══██╗    ██╔══██╗██╔══██╗╚══██╔══╝██╔════╝
-                    █████╗  ██║██║     ██╔████╔██║██║   ██║    ██████╔╝███████║   ██║   █████╗
-                    ██╔══╝  ██║██║     ██║╚██╔╝██║██║   ██║    ██╔══██╗██╔══██║   ██║   ██╔══╝
-                    ██║     ██║███████╗██║ ╚═╝ ██║╚██████╔╝    ██║  ██║██║  ██║   ██║   ███████╗
-                    ╚═╝     ╚═╝╚══════╝╚═╝     ╚═╝ ╚═════╝     ╚═╝  ╚═╝╚═╝  ╚═╝   ╚═╝   ╚══════╝
-                </pre>
-                """;
-    }
+	@GetMapping
+	public String homePage() {
+		return """
+				<p><br></p>
+				<pre>
+				    ███████╗██╗██╗     ███╗   ███╗ ██████╗     ██████╗  █████╗ ████████╗███████╗
+				    ██╔════╝██║██║     ████╗ ████║██╔═══██╗    ██╔══██╗██╔══██╗╚══██╔══╝██╔════╝
+				    █████╗  ██║██║     ██╔████╔██║██║   ██║    ██████╔╝███████║   ██║   █████╗
+				    ██╔══╝  ██║██║     ██║╚██╔╝██║██║   ██║    ██╔══██╗██╔══██║   ██║   ██╔══╝
+				    ██║     ██║███████╗██║ ╚═╝ ██║╚██████╔╝    ██║  ██║██║  ██║   ██║   ███████╗
+				    ╚═╝     ╚═╝╚══════╝╚═╝     ╚═╝ ╚═════╝     ╚═╝  ╚═╝╚═╝  ╚═╝   ╚═╝   ╚══════╝
+				</pre>
+				""";
+	}
 
-    @GetMapping("favicon.ico")
-    public String faviconIco() {
-        return """
-                <p><br></p>
-        """;
-    }
+	@GetMapping("favicon.ico")
+	public String faviconIco() {
+		return """
+				        <p><br></p>
+				""";
+	}
 
-    @GetMapping(".well-known/appspecific/com.chrome.devtools.json")
-    public String devtools() {
-        return """
-                <p><br></p>
-        """;
-    }
+	@GetMapping(".well-known/appspecific/com.chrome.devtools.json")
+	public String devtools() {
+		return """
+				        <p><br></p>
+				""";
+	}
 
-    @GetMapping("sw.js")
-    public String swjs() {
-        return """
-                <p><br></p>
-        """;
-    }
+	@GetMapping("sw.js")
+	public String swjs() {
+		return """
+				        <p><br></p>
+				""";
+	}
 }

src/main/java/ru/yandex/practicum/filmorate/controller/MpaController.java

@@ -6,15 +6,15 @@
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 import ru.yandex.practicum.filmorate.dto.MpaDto;
-import ru.yandex.practicum.filmorate.servise.MpaServise;
+import ru.yandex.practicum.filmorate.service.MpaService;
 
 import java.util.Collection;
 
-@RestController
 @RequestMapping("/mpa")
+@RestController
 @RequiredArgsConstructor
 public class MpaController {
-	private final MpaServise mpaService;
+	private final MpaService mpaService;
 
 	@GetMapping("/{mpaId}")
 	public MpaDto getMpaById(@PathVariable long mpaId) {