Add remove endpoint

.gitignore

@@ -14,6 +14,9 @@ target/
 !**/src/main/**/target/
 !**/src/test/**/target/
 
+### macOS ###
+.DS_Store
+
 ### STS ###
 .apt_generated
 .classpath

pom.xml

@@ -43,6 +43,7 @@
         <dependency>
             <groupId>org.projectlombok</groupId>
             <artifactId>lombok</artifactId>
+            <version>1.18.30</version>
             <optional>true</optional>
         </dependency>
         <dependency>
@@ -81,6 +82,7 @@
                         <path>
                             <groupId>org.projectlombok</groupId>
                             <artifactId>lombok</artifactId>
+                            <version>1.18.30</version>
                         </path>
                     </annotationProcessorPaths>
                 </configuration>

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

@@ -0,0 +1,55 @@
+package ru.yandex.practicum.filmorate.controller;
+
+import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
+import org.springframework.http.HttpStatus;
+import org.springframework.validation.annotation.Validated;
+import org.springframework.web.bind.annotation.*;
+import ru.yandex.practicum.filmorate.dto.DirectorDto;
+import ru.yandex.practicum.filmorate.model.Director;
+import ru.yandex.practicum.filmorate.service.director.DirectorService;
+
+import java.util.Collection;
+
+@Slf4j
+@RestController
+@RequestMapping("/directors")
+@RequiredArgsConstructor
+@Validated
+public class DirectorController {
+
+    private final DirectorService directorService;
+
+    @GetMapping
+    public Collection<DirectorDto> getDirectors() {
+        log.info("Received GET /directors request");
+        return directorService.getDirectors();
+    }
+
+    @GetMapping("/{directorId}")
+    public DirectorDto getDirectorById(@PathVariable long directorId) {
+        log.info("Received GET /directors/{} request", directorId);
+        return directorService.getDirectorById(directorId);
+    }
+
+    @PostMapping
+    @ResponseStatus(HttpStatus.CREATED)
+    public DirectorDto addDirector(@RequestBody Director director) {
+        log.info("Received POST /directors request with body: {}", director);
+        return directorService.addDirector(director);
+    }
+
+    @PutMapping
+    public DirectorDto updateDirector(@RequestBody Director director) {
+        log.info("Received PUT /directors request with body: {}", director);
+        return directorService.updateDirector(director);
+    }
+
+    @DeleteMapping("/{directorId}")
+    public void deleteDirector(@PathVariable long directorId) {
+        log.info("Received DELETE /directors/{} request", directorId);
+        directorService.deleteDirector(directorId);
+
+    }
+
+}

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

@@ -38,4 +38,11 @@ public ErrorResponse handleConstraintViolation(final ConstraintViolationExceptio
         log.warn("constraint violation");
         return new ErrorResponse(e.getMessage());
     }
+
+    @ExceptionHandler(InternalServerException.class)
+    @ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR)
+    public ErrorResponse handleException(Exception e) {
+        log.error("Ошибка сервера: {}", e.getMessage());
+        return new ErrorResponse("Внутренняя ошибка сервера: " + e.getMessage());
+    }
 }

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

@@ -63,8 +63,48 @@ public void deleteLikeToFilm(@PathVariable long filmId, @PathVariable long userI
     }
 
     @GetMapping("/popular")
-    public List<FilmDto> getTopFilms(@RequestParam(defaultValue = "10") @Positive(message = "Count must be positive") int count) {
-        log.info("Received GET /films/popular?count={} request", count);
-        return filmService.getTopFilms(count);
+    public List<FilmDto> getTopFilms(@RequestParam(defaultValue = "10") @Positive(message = "Count must be positive") int count,
+                                     @RequestParam(defaultValue = "-1") int genreId,
+                                     @RequestParam(defaultValue = "-1") int year) {
+        log.info("Received GET /films/popular?count={}&genreId={}&year={} request", count, genreId, year);
+        return filmService.getTopFilms(count, genreId, year);
     }
+
+    @DeleteMapping("/{filmId}")
+    @ResponseStatus(HttpStatus.NO_CONTENT)
+    public void deleteFilm(@PathVariable long filmId) {
+        log.info("Received DELETE /films/{} request", filmId);
+        filmService.deleteFilm(filmId);
+    }
+
+    @GetMapping("/common")
+    public List<FilmDto> getCommonFilms(
+            @RequestParam long userId,
+            @RequestParam long friendId) {
+        log.info("GET common films for userId={} and friendId={}", userId, friendId);
+        return filmService.getCommonFilms(userId, friendId);
+    }
+
+    @DeleteMapping("/{filmId}")
+    @ResponseStatus(HttpStatus.NO_CONTENT)
+    public void deleteFilm(@PathVariable long filmId) {
+        log.info("Received DELETE /films/{} request", filmId);
+        filmService.deleteFilm(filmId);
+    }
+
+    @GetMapping("/common")
+    public List<FilmDto> getCommonFilms(
+            @RequestParam long userId,
+            @RequestParam long friendId) {
+        log.info("GET common films for userId={} and friendId={}", userId, friendId);
+        return filmService.getCommonFilms(userId, friendId);
+    }
+
+    @GetMapping("/director/{directorId}")
+    public List<FilmDto> findByDirector(@PathVariable long directorId,
+                                        @RequestParam(name = "sortBy") String sortMode) {
+        log.info("Received GET /films/director/{}?sortBy={} request", directorId, sortMode);
+        return filmService.findByDirector(directorId, sortMode);
+    }
+
 }

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

@@ -5,8 +5,10 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.web.bind.annotation.*;
+import ru.yandex.practicum.filmorate.dto.FilmDto;
 import ru.yandex.practicum.filmorate.dto.UserDto;
 import ru.yandex.practicum.filmorate.model.User;
+import ru.yandex.practicum.filmorate.service.film.FilmService;
 import ru.yandex.practicum.filmorate.service.user.UserService;
 import ru.yandex.practicum.filmorate.storage.user.UserDbStorage;
 
@@ -17,10 +19,12 @@
 @RequestMapping("/users")
 public class UserController {
     private final UserService userService;
+    private final FilmService filmService;
 
     @Autowired
-    public UserController(UserDbStorage userDbStorage) {
+    public UserController(UserDbStorage userDbStorage, FilmService filmService) {
         userService = new UserService(userDbStorage);
+        this.filmService = filmService;
     }
 
     @GetMapping
@@ -72,5 +76,18 @@ public Collection<UserDto> getFriends(@PathVariable int id) {
         log.info("getFriends");
         return userService.getFriends(id);
     }
+
+    @DeleteMapping("/{userId}")
+    @ResponseStatus(HttpStatus.NO_CONTENT)
+    public void deleteUser(@PathVariable int userId) {
+        log.info("Recieved DELETE /users/{} request", userId);
+        userService.deleteUser(userId);
+    }
+
+    @GetMapping("/{id}/recommendations")
+    public Collection<FilmDto> getRecommendations(@PathVariable long id) {
+        log.info("get recommendations");
+        return filmService.getRecommendations(id);
+    }
 }
 

src/main/java/ru/yandex/practicum/filmorate/dal/DirectorRepository.java

@@ -0,0 +1,48 @@
+package ru.yandex.practicum.filmorate.dal;
+
+import org.springframework.jdbc.core.JdbcTemplate;
+import org.springframework.jdbc.core.RowMapper;
+import org.springframework.stereotype.Repository;
+import ru.yandex.practicum.filmorate.model.Director;
+
+import java.util.List;
+import java.util.Optional;
+
+@Repository
+public class DirectorRepository extends BaseRepository<Director> {
+
+    private static final String FIND_ALL_QUERY = "SELECT * FROM DIRECTORS d";
+    private static final String FIND_BY_ID_QUERY = "SELECT * FROM DIRECTORS d WHERE d.director_id = ?";
+    private static final String INSERT_QUERY = "INSERT INTO DIRECTORS (NAME) VALUES(?)";
+    private static final String UPDATE_QUERY = "UPDATE DIRECTORS SET NAME = ? WHERE DIRECTOR_ID = ?";
+    private static final String DELETE_QUERY = "DELETE FROM DIRECTORS WHERE DIRECTOR_ID = ?";
+
+    public DirectorRepository(JdbcTemplate jdbc,
+                              RowMapper<Director> mapper) {
+        super(jdbc, mapper);
+    }
+
+    public List<Director> findAll() {
+        return findMany(FIND_ALL_QUERY);
+    }
+
+    public Optional<Director> findById(long id) {
+        return findOne(FIND_BY_ID_QUERY, id);
+    }
+
+    public Director save(Director director) {
+        Long id = (Long) insert(INSERT_QUERY, director.getName());
+        director.setId(id);
+        return director;
+    }
+
+    public Director update(Director director) {
+        update(UPDATE_QUERY, director.getName(), director.getId());
+        return director;
+    }
+
+    public void delete(long id) {
+        update(DELETE_QUERY, id);
+    }
+
+}