[Refactor] BlockList API to simplify request handling

Replaced BlockReq DTO with direct method parameters for better clarity and reduced complexity. Updated controller methods to use `@AuthenticationPrincipal` for user context, streamlining data flow and enhancing maintainability.

Author: moon-kun-woong

src/main/java/org/dfbf/soundlink/domain/blocklist/controller/BlockListController.java

@@ -3,9 +3,9 @@
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.tags.Tag;
 import lombok.RequiredArgsConstructor;
-import org.dfbf.soundlink.domain.blocklist.dto.BlockReq;
 import org.dfbf.soundlink.domain.blocklist.service.BlockListService;
 import org.dfbf.soundlink.global.exception.ResponseResult;
+import org.springframework.security.core.annotation.AuthenticationPrincipal;
 import org.springframework.web.bind.annotation.*;
 
 @RestController
@@ -20,25 +20,31 @@ public class BlockListController {
             summary = "유저 차단",
             description = "특정 유저를 차단합니다."
     )
-    public ResponseResult blockUser(@RequestBody BlockReq blockReq) {
-        return blockListService.blockUser(blockReq);
+    public ResponseResult blockUser(
+            @AuthenticationPrincipal Long userId,
+            @AuthenticationPrincipal Long blockUserId
+    ) {
+        return blockListService.blockUser(userId, blockUserId);
     }
 
     @DeleteMapping
     @Operation(
             summary = "유저 차단 해제",
             description = "차단한 유저를 해제합니다."
     )
-    public ResponseResult unblockUser(@RequestBody BlockReq blockReq) {
-        return blockListService.unblockUser(blockReq);
+    public ResponseResult unblockUser(
+            @AuthenticationPrincipal Long userId,
+            @AuthenticationPrincipal Long blockUserId
+    ) {
+        return blockListService.unblockUser(userId, blockUserId);
     }
 
-    @GetMapping("/{userId}")
+    @GetMapping
     @Operation(
             summary = "차단 목록 조회",
             description = "해당 유저의 차단 목록을 가져옵니다."
     )
-    public ResponseResult getBlockList(@PathVariable Long userId) {
+    public ResponseResult getBlockList(@AuthenticationPrincipal Long userId) {
         return blockListService.getBlockListByUserId(userId);
     }
 }

src/main/java/org/dfbf/soundlink/domain/blocklist/dto/BlockReq.java

@@ -2,7 +2,6 @@
 
 import jakarta.transaction.Transactional;
 import lombok.RequiredArgsConstructor;
-import org.dfbf.soundlink.domain.blocklist.dto.BlockReq;
 import org.dfbf.soundlink.domain.blocklist.dto.BlockRes;
 import org.dfbf.soundlink.domain.blocklist.entity.Blocklist;
 import org.dfbf.soundlink.domain.blocklist.exception.AlreadyBlockedUser;
@@ -25,18 +24,18 @@ public class BlockListService {
     private final UserRepository userRepository;
 
     @Transactional
-    public ResponseResult blockUser(BlockReq blockReq) {
+    public ResponseResult blockUser(Long userId, Long blockedUserId) {
         try {
-            User user = userRepository.findById(blockReq.userId())
+            User user = userRepository.findById(userId)
                     .orElseThrow(
                             BlockedUserNotFound::new
                     );
-            User blockedUser = userRepository.findById(blockReq.blockedUserId())
+            User blockedUser = userRepository.findById(blockedUserId)
                     .orElseThrow(
                             BlockingUserNotFound::new
                     );
             blockListRepository.findByUserIdAndBlockedUserId(
-                    blockReq.userId(), blockReq.blockedUserId()
+                    userId, blockedUserId
             ).ifPresent(block -> {
                 throw new AlreadyBlockedUser();
             });
@@ -76,10 +75,10 @@ public ResponseResult blockUser(BlockReq blockReq) {
     }
 
     @Transactional
-    public ResponseResult unblockUser(BlockReq blockReq) {
+    public ResponseResult unblockUser(Long userId, Long blockedUserId) {
         try {
             Blocklist block = blockListRepository.findByUserIdAndBlockedUserId(
-                    blockReq.userId(), blockReq.blockedUserId()
+                    userId, blockedUserId
             ).orElseThrow(BlockingUserNotFound::new);
 
             blockListRepository.delete(block);