Merge pull request #13 from Sportize/refactor/post

🐛 Fix: 커뮤니티 게시글 기능관련 버그 수정

Author: imjuyongp

src/main/java/com/be/sportizebe/domain/comment/controller/CommentController.java

@@ -4,6 +4,7 @@
 import com.be.sportizebe.domain.comment.dto.response.CommentResponse;
 import com.be.sportizebe.domain.comment.service.CommentService;
 import com.be.sportizebe.domain.user.entity.User;
+import com.be.sportizebe.domain.user.repository.UserRepository;
 import com.be.sportizebe.global.response.BaseResponse;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.Parameter;
@@ -12,7 +13,6 @@
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
-import org.springframework.security.core.annotation.AuthenticationPrincipal;
 import org.springframework.web.bind.annotation.*;
 
 import java.util.List;
@@ -24,13 +24,16 @@
 public class CommentController {
 
     private final CommentService commentService;
+    private final UserRepository userRepository; // TODO: 인증 로직 개발 후 제거
 
     @PostMapping
     @Operation(summary = "댓글 생성", description = "게시글에 댓글 또는 대댓글 생성")
     public ResponseEntity<BaseResponse<CommentResponse>> createComment(
         @Parameter(description = "게시글 ID") @PathVariable Long postId,
-        @RequestBody @Valid CreateCommentRequest request,
-        @AuthenticationPrincipal User user) {
+        @RequestBody @Valid CreateCommentRequest request) {
+        // TODO: 인증 로직 개발 후 @AuthenticationPrincipal User user로 변경
+        User user = userRepository.findById(1L)
+            .orElseThrow(() -> new RuntimeException("테스트 유저가 없습니다. users 테이블에 id=1인 유저를 추가해주세요."));
         CommentResponse response = commentService.createComment(postId, request, user);
         return ResponseEntity.status(HttpStatus.CREATED)
             .body(BaseResponse.success("댓글 생성 성공", response));
@@ -44,15 +47,17 @@ public ResponseEntity<BaseResponse<List<CommentResponse>>> getComments(
         return ResponseEntity.ok(BaseResponse.success(responses));
     }
 
-//    @DeleteMapping("/{commentId}")
-//    @Operation(summary = "댓글 삭제", description = "댓글 삭제 (대댓글도 함께 삭제)")
-//    public ResponseEntity<BaseResponse<Void>> deleteComment(
-//        @Parameter(description = "게시글 ID") @PathVariable Long postId,
-//        @Parameter(description = "댓글 ID") @PathVariable Long commentId,
-//        @AuthenticationPrincipal User user) {
-//        commentService.deleteComment(commentId, user);
-//        return ResponseEntity.ok(BaseResponse.success("댓글 삭제 성공", null));
-//    }
+    @DeleteMapping("/{commentId}")
+    @Operation(summary = "댓글 삭제", description = "댓글 삭제 (대댓글도 함께 삭제)")
+    public ResponseEntity<BaseResponse<Void>> deleteComment(
+        @Parameter(description = "게시글 ID") @PathVariable Long postId,
+        @Parameter(description = "댓글 ID") @PathVariable Long commentId) {
+        // TODO: 인증 로직 개발 후 @AuthenticationPrincipal User user로 변경
+        User user = userRepository.findById(1L)
+            .orElseThrow(() -> new RuntimeException("테스트 유저가 없습니다. users 테이블에 id=1인 유저를 추가해주세요."));
+        commentService.deleteComment(commentId, user);
+        return ResponseEntity.ok(BaseResponse.success("댓글 삭제 성공", null));
+    }
 
     @GetMapping("/count")
     @Operation(summary = "댓글 수 조회", description = "게시글의 총 댓글 수 조회")

src/main/java/com/be/sportizebe/domain/comment/entity/Comment.java

@@ -36,8 +36,8 @@ public class Comment extends BaseTimeEntity {
     private User user; // 작성자 (프록시 객체)
 
     @ManyToOne(fetch = FetchType.LAZY)
-    @JoinColumn(name = "parent_id")
-    private Comment parent; // 부모 댓글 (null이면 일반 댓글)
+    @JoinColumn(name = "parent_id") // 자기 참조
+    private Comment parent; // 부모 댓글 (null이면 부모 댓글)
 
     @OneToMany(mappedBy = "parent", cascade = CascadeType.ALL, orphanRemoval = true)
     @Builder.Default

src/main/java/com/be/sportizebe/domain/comment/exception/CommentErrorCode.java

@@ -12,7 +12,8 @@ public enum CommentErrorCode implements BaseErrorCode {
   COMMENT_DELETE_DENIED("C002", "댓글 삭제 권한이 없습니다.", HttpStatus.FORBIDDEN),
   COMMENT_UPDATE_DENIED("C003", "댓글 수정 권한이 없습니다.", HttpStatus.FORBIDDEN),
   INVALID_PARENT_COMMENT("C004", "유효하지 않은 부모 댓글입니다.", HttpStatus.BAD_REQUEST),
-  COMMENT_PARENT_POST_MISMATCH("C005", "부모 댓글 다른 게시글에 속해있습니다.", HttpStatus.BAD_REQUEST);
+  COMMENT_PARENT_POST_MISMATCH("C005", "부모 댓글 다른 게시글에 속해있습니다.", HttpStatus.BAD_REQUEST),
+  NESTED_REPLY_NOT_ALLOWED("C006", "대댓글에는 답글을 달 수 없습니다.", HttpStatus.BAD_REQUEST);
 
   private final String code;
   private final String message;

src/main/java/com/be/sportizebe/domain/comment/service/CommentService.java

@@ -15,7 +15,7 @@ public interface CommentService {
   List<CommentResponse> getCommentsByPostId(Long postId);
 
   // 댓글 삭제
-  // void deleteComment(Long commentId, User user);
+  void deleteComment(Long commentId, User user);
 
   // 게시글의 댓글 수 조회
   long getCommentCount(Long postId);

src/main/java/com/be/sportizebe/domain/comment/service/CommentServiceImpl.java

@@ -34,14 +34,19 @@ public CommentResponse createComment(Long postId, CreateCommentRequest request,
 
     // 부모 댓글 조회 (대댓글인 경우)
     Comment parent = null; // null로 초기화
-    if (request.parentId() != null) { // parentId가 null이 아니면 대댓글이기 때문에 부모 댓글 조회가 필요함
+    if (request.parentId() != null) { // parentId가 null이 아니면 대댓글이기 때문에 부모 댓글 조회로 검증 필요함
       parent = commentRepository.findById(request.parentId())
           .orElseThrow(() -> new CustomException(CommentErrorCode.COMMENT_NOT_FOUND));
 
       // 해당 대댓글의 부모 댓글이 같은 게시글에 속하는지 검증
       if(parent.getPost().getId() != post.getId()) {
           throw new CustomException(CommentErrorCode.COMMENT_PARENT_POST_MISMATCH);
       }
+
+      // 부모 댓글이 이미 대댓글인 경우 답글 불가 (댓글 깊이 1단계로 제한)
+      if(parent.getParent() != null) {
+          throw new CustomException(CommentErrorCode.NESTED_REPLY_NOT_ALLOWED);
+      }
     }
 
     // 댓글 생성
@@ -65,19 +70,19 @@ public List<CommentResponse> getCommentsByPostId(Long postId) {
         .toList();
   }
 
-//  @Override
-//  @Transactional
-//  public void deleteComment(Long commentId, User user) {
-//    Comment comment = commentRepository.findById(commentId)
-//        .orElseThrow(() -> new CustomException(CommentErrorCode.COMMENT_NOT_FOUND));
-//
-//    // 작성자 확인
-//    if (!comment.getUser().getId().equals(user.getId())) {
-//      throw new CustomException(CommentErrorCode.COMMENT_DELETE_DENIED);
-//    }
-//
-//    commentRepository.delete(comment);
-//  }
+  @Override
+  @Transactional
+  public void deleteComment(Long commentId, User user) {
+    Comment comment = commentRepository.findById(commentId)
+        .orElseThrow(() -> new CustomException(CommentErrorCode.COMMENT_NOT_FOUND));
+
+    // 작성자 확인
+    if (comment.getUser().getId() != user.getId()) {
+      throw new CustomException(CommentErrorCode.COMMENT_DELETE_DENIED);
+    }
+
+    commentRepository.delete(comment);
+  }
 
   @Override
   public long getCommentCount(Long postId) {

src/main/java/com/be/sportizebe/domain/post/controller/PostController.java

@@ -1,9 +1,12 @@
 package com.be.sportizebe.domain.post.controller;
 
 import com.be.sportizebe.domain.post.dto.request.CreatePostRequest;
-import com.be.sportizebe.domain.post.dto.response.CreatePostResponse;
+import com.be.sportizebe.domain.post.dto.request.UpdatePostRequest;
+import com.be.sportizebe.domain.post.dto.response.PostResponse;
 import com.be.sportizebe.domain.post.entity.PostProperty;
 import com.be.sportizebe.domain.post.service.PostService;
+import com.be.sportizebe.domain.user.entity.User;
+import com.be.sportizebe.domain.user.repository.UserRepository;
 import com.be.sportizebe.global.response.BaseResponse;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.Parameter;
@@ -12,11 +15,7 @@
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
-import org.springframework.web.bind.annotation.PathVariable;
-import org.springframework.web.bind.annotation.PostMapping;
-import org.springframework.web.bind.annotation.RequestBody;
-import org.springframework.web.bind.annotation.RequestMapping;
-import org.springframework.web.bind.annotation.RestController;
+import org.springframework.web.bind.annotation.*;
 
 @RestController
 @RequiredArgsConstructor
@@ -25,14 +24,41 @@
 public class PostController {
 
     private final PostService postService;
+    private final UserRepository userRepository; // TODO: 인증 로직 개발 후 제거
 
     @PostMapping("/posts/{property}")
     @Operation(summary = "게시글 생성", description = "게시판 종류별 게시글 생성")
-    public ResponseEntity<BaseResponse<CreatePostResponse>> createPost(
+    public ResponseEntity<BaseResponse<PostResponse>> createPost(
         @Parameter(description = "게시판 종류 (SOCCER, BASKETBALL, FREE)") @PathVariable PostProperty property,
         @RequestBody @Valid CreatePostRequest request) {
-        CreatePostResponse response = postService.createPost(property, request);
+        // TODO: 인증 로직 개발 후 @AuthenticationPrincipal User user로 변경
+        User user = userRepository.findById(1L)
+            .orElseThrow(() -> new RuntimeException("테스트 유저가 없습니다. users 테이블에 id=1인 유저를 추가해주세요."));
+        PostResponse response = postService.createPost(property, request, user);
         return ResponseEntity.status(HttpStatus.CREATED)
             .body(BaseResponse.success("게시글 생성 성공", response));
     }
+
+    @PutMapping("/posts/{postId}")
+    @Operation(summary = "게시글 수정", description = "게시글 수정 (작성자만 가능)")
+    public ResponseEntity<BaseResponse<PostResponse>> updatePost(
+        @Parameter(description = "게시글 ID") @PathVariable Long postId,
+        @RequestBody @Valid UpdatePostRequest request) {
+        // TODO: 인증 로직 개발 후 @AuthenticationPrincipal User user로 변경
+        User user = userRepository.findById(1L)
+            .orElseThrow(() -> new RuntimeException("테스트 유저가 없습니다. users 테이블에 id=1인 유저를 추가해주세요."));
+        PostResponse response = postService.updatePost(postId, request, user);
+        return ResponseEntity.ok(BaseResponse.success("게시글 수정 성공", response));
+    }
+
+    @DeleteMapping("/posts/{postId}")
+    @Operation(summary = "게시글 삭제", description = "게시글 삭제 (작성자만 가능)")
+    public ResponseEntity<BaseResponse<Void>> deletePost(
+        @Parameter(description = "게시글 ID") @PathVariable Long postId) {
+        // TODO: 인증 로직 개발 후 @AuthenticationPrincipal User user로 변경
+        User user = userRepository.findById(1L)
+            .orElseThrow(() -> new RuntimeException("테스트 유저가 없습니다. users 테이블에 id=1인 유저를 추가해주세요."));
+        postService.deletePost(postId, user);
+        return ResponseEntity.ok(BaseResponse.success("게시글 삭제 성공", null));
+    }
 }

src/main/java/com/be/sportizebe/domain/post/dto/request/CreatePostRequest.java

@@ -2,6 +2,7 @@
 
 import com.be.sportizebe.domain.post.entity.Post;
 import com.be.sportizebe.domain.post.entity.PostProperty;
+import com.be.sportizebe.domain.user.entity.User;
 import jakarta.validation.constraints.NotBlank;
 import lombok.Builder;
 
@@ -12,13 +13,14 @@ public record CreatePostRequest(
     boolean isAnonymous,
     String imgUrl) {
 
-  public Post toEntity(PostProperty property) { // DTO -> Entity 변환
+  public Post toEntity(PostProperty property, User user) { // DTO -> Entity 변환
     return Post.builder()
         .title(title)
         .content(content)
         .isAnonymous(isAnonymous)
         .imgUrl(imgUrl)
         .property(property)
+        .user(user)
         .build();
   }
 }

src/main/java/com/be/sportizebe/domain/post/dto/request/UpdatePostRequest.java

@@ -0,0 +1,9 @@
+package com.be.sportizebe.domain.post.dto.request;
+
+import jakarta.validation.constraints.NotBlank;
+
+public record UpdatePostRequest(
+    @NotBlank(message = "제목은 필수입니다.") String title,
+    @NotBlank(message = "내용은 필수입니다.") String content,
+    String imgUrl) {
+}

…ost/dto/response/CreatePostResponse.java …main/post/dto/response/PostResponse.javasrc/main/java/com/be/sportizebe/domain/post/dto/response/CreatePostResponse.java renamed to src/main/java/com/be/sportizebe/domain/post/dto/response/PostResponse.java src/main/java/com/be/sportizebe/domain/post/dto/response/CreatePostResponse.java renamed to src/main/java/com/be/sportizebe/domain/post/dto/response/PostResponse.java

@@ -5,17 +5,19 @@
 import lombok.Builder;
 
 @Builder
-@Schema(title = "CreatePostResponse DTO", description = "게시글 생성 응답")
-public record CreatePostResponse(
+@Schema(title = "PostResponse DTO", description = "게시글 관련 응답")
+public record PostResponse(
     @Schema(description = "게시글 고유번호", example = "1") Long postId,
+    @Schema(description = "게시글 제목", example = "게시글 제목 ...") String title,
     @Schema(description = "게시글 내용", example = "게시글 글 내용 ...") String content,
     @Schema(description = "게시글 등록자", example = "사용자명") String publisher,
     @Schema(description = "게시글 사진 url", example = "s3 url") String imgUrl,
     @Schema(description = "익명 여부", example = "true:익명 / false:사용자명") String isAnonymous) {
 
-  public static CreatePostResponse from(Post post) { // Post -> Resonse 변환
-    return CreatePostResponse.builder()
+  public static PostResponse from(Post post) { // Post -> Response 변환
+    return PostResponse.builder()
         .postId(post.getId())
+        .title(post.getTitle())
         .content(post.getContent())
         .publisher(post.isAnonymous() ? "익명" : "사용자명") // TODO: User 매핑 필요
         .imgUrl(post.getImgUrl())

src/main/java/com/be/sportizebe/domain/post/entity/Post.java

@@ -36,4 +36,10 @@ public class Post extends BaseTimeEntity {
     @ManyToOne(fetch = FetchType.LAZY)
     @JoinColumn(name = "user_id")
     private User user; // 작성자
+
+    public void update(String title, String content, String imgUrl) {
+        this.title = title;
+        this.content = content;
+        this.imgUrl = imgUrl;
+    }
 }