🏄 :: (Meogo-42) post good #43

meltapplee · 2024-09-30T14:39:19Z

close #42

Summary by CodeRabbit

새로운 기능
- 북마크 관리 기능 추가: 사용자가 특정 학교와 관련된 북마크를 생성, 조회 및 삭제할 수 있습니다.
- "좋아요" 기능 추가: 사용자가 게시물에 "좋아요"를 추가하고 삭제할 수 있는 기능이 구현되었습니다.
버그 수정
- 게시물의 "좋아요" 수를 관리하는 메서드가 개선되었습니다.
문서화
- 새로운 REST 컨트롤러와 서비스 클래스에 대한 설명 추가.

coderabbitai · 2024-09-30T14:39:25Z

Walkthrough

이번 변경 사항은 Bookmark, BookmarkRepository, BookmarkService, Good, GoodRepository, GoodController, GoodService, Post 클래스 및 관련 파일을 포함하여 북마크 및 좋은 기능을 관리하는 새로운 서비스와 리포지토리 인터페이스를 도입합니다. 이 변경으로 사용자가 북마크와 좋은 기능을 추가하고 삭제할 수 있는 REST API가 제공되며, 데이터베이스와의 상호작용을 위한 새로운 메서드가 추가되었습니다.

Changes

파일 경로	변경 요약
`src/main/kotlin/org/meogo/domain/bookmark/Bookmark.kt`	`Bookmark` 클래스 추가: `UUID` 타입의 `id`, `Int` 타입의 `schoolId`, `User` 타입의 `user` 속성 포함.
`src/main/kotlin/org/meogo/domain/bookmark/BookmarkRepository.kt`	`BookmarkRepository` 인터페이스 추가: CRUD 메서드 및 특정 학교와 사용자에 대한 메서드 포함.
`src/main/kotlin/org/meogo/domain/bookmark/BookmarkService.kt`	`BookmarkService` 클래스 추가: 북마크를 관리하는 메서드 (`execute`, `queryBookmarkedPost`, `queryIsBookmarked`, `deleteBookmark`) 포함.
`src/main/kotlin/org/meogo/domain/good/domain/Good.kt`	`Good` 클래스 추가: `UUID` 타입의 `id`와 사용자 및 게시물과의 관계 유지.
`src/main/kotlin/org/meogo/domain/good/domain/GoodRepository.kt`	`GoodRepository` 인터페이스 추가: 사용자와 게시물에 따른 `Good` 객체 검색 메서드 포함.
`src/main/kotlin/org/meogo/domain/good/presentation/GoodController.kt`	`GoodController` 클래스 추가: HTTP 요청을 처리하는 메서드 (`addGood`, `deleteGood`) 포함.
`src/main/kotlin/org/meogo/domain/good/service/GoodService.kt`	`GoodService` 클래스 추가: `addGood` 및 `deleteGood` 메서드를 통해 게시물에 대한 좋은 기능을 관리.
`src/main/kotlin/org/meogo/domain/post/domain/Post.kt`	`Post` 클래스 수정: `deleteGood` 메서드 추가 및 `update` 메서드에 선택적 매개변수 추가.

Possibly related PRs

🏄 :: (Meogo-9) create review #10: BookmarkService 클래스는 사용자와 학교에 연관된 북마크를 관리하며, 이는 Review 클래스와 유사한 개념적 구조를 가집니다.
🏄 :: (Meogo-11) all school review #12: Review 클래스 및 리포지토리는 사용자와 학교와의 연관성을 가지며, 이는 Bookmark 클래스와 유사한 구조입니다.
🏄 :: (Meogo-13) modif review #15: Review 클래스의 속성과 메서드 추가는 사용자 상호작용에 중점을 두고 있으며, 이는 Bookmark 클래스의 사용자 중심 설계와 유사합니다.
🏄 :: (Meogo-20) user my page #21: UserRepository의 변경 사항은 Bookmark와 Review 맥락에서 사용자 관리를 관련짓는 요소입니다.
🏄 :: (Meogo-28) query school post #29: Post 클래스의 수정 사항은 사용자 관련 속성을 포함하며, 이는 Bookmark와 Review 클래스의 사용자 중심 설계와 일치합니다.
🏄 :: (Meogo-32) modify community post #33: Post 클래스의 업데이트는 사용자 관련 속성과 메서드를 포함하며, 이는 Bookmark와 Review 클래스의 사용자 연관성과 유사합니다.
🏄 :: (Meogo-38) question crud #39: CommentController의 변경 사항은 사용자 상호작용을 포함하며, 이는 Bookmark와 Review 기능의 핵심 요소입니다.
🏄 :: (Meogo-40) question tag detail #41: Question 클래스 및 리포지토리의 수정 사항은 사용자 상호작용과 관련이 있으며, 이는 Bookmark와 Review 클래스의 사용자 중심 설계와 유사합니다.

🐰 새로운 기능이 생겼어요,
북마크와 좋은 기능이 함께해요!
사용자와 학교를 잇는 다리,
즐거운 클릭으로 세상을 밝혀요.
모두 함께 즐길 수 있는 이 순간,
행복한 토끼가 춤을 춰요! 🎉

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Generate unit testing code for this file.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

coderabbitai

Actionable comments posted: 13

🧹 Outside diff range and nitpick comments (4)

src/main/kotlin/org/meogo/domain/bookmark/BookmarkRepository.kt (2)
7-7: 인터페이스 선언이 적절합니다. 작은 개선 제안이 있습니다.

BookmarkRepository 인터페이스가 JpaRepository<Bookmark, UUID>를 올바르게 확장하고 있어 Bookmark 엔티티에 대한 기본적인 CRUD 작업이 가능합니다.

가독성을 높이기 위해 Bookmark와 UUID에 대한 import 문을 추가하는 것이 좋습니다. 다음과 같이 변경해 보세요:
import org.meogo.domain.bookmark.Bookmark
import java.util.UUID

interface BookmarkRepository : JpaRepository<Bookmark, UUID> {
    // ...
}
8-12: 사용자 정의 메서드들이 잘 구현되어 있습니다. 작은 개선 제안이 있습니다.

세 가지 사용자 정의 메서드(findAllBySchoolId, existsBySchoolIdAndUser, deleteBySchoolIdAndUser)가 Spring Data JPA 명명 규칙을 따르고 있으며, 북마크 관리에 필요한 작업들을 적절히 다루고 있습니다.

findAllBySchoolId 메서드의 반환 타입을 개선할 수 있습니다. 다음과 같이 변경하는 것이 좋습니다:
fun findAllBySchoolId(schoolId: Int): List<Bookmark>
이렇게 하면 빈 리스트를 반환할 수 있어 null 체크를 줄일 수 있습니다. JPA는 결과가 없을 때 빈 컬렉션을 반환하므로, 이 방식이 더 Kotlin스러운 접근법입니다.
src/main/kotlin/org/meogo/domain/good/domain/Good.kt (1)

13-24: 클래스 이름을 재고해 보세요.

Bookmark에서 Good으로의 클래스 이름 변경이 의도를 명확히 전달하지 않을 수 있습니다. 클래스의 목적을 더 잘 설명하는 이름을 고려해 보세요. 예를 들어, Like 또는 Favorite과 같은 이름이 더 적절할 수 있습니다.

그 외의 클래스 구조와 관계 설정은 적절합니다:

Post와 User 엔티티와의 관계가 잘 정의되어 있습니다.

지연 로딩(LAZY)을 사용하여 성능을 최적화하고 있습니다.

BaseUUIDEntity를 상속받아 공통 기능을 재사용하고 있습니다.
src/main/kotlin/org/meogo/domain/good/presentation/GoodController.kt (1)
1-28: 문서화 개선 제안

코드의 가독성과 유지보수성을 높이기 위해 KDoc 주석을 추가하는 것이 좋습니다. 컨트롤러와 각 메서드의 목적을 설명하는 주석을 추가해주세요.

다음과 같이 KDoc 주석을 추가해보세요:
/**
 * 게시물에 대한 '좋아요' 기능을 관리하는 컨트롤러입니다.
 *
 * @property goodService 좋아요 관련 비즈니스 로직을 처리하는 서비스
 */
@RestController
@RequestMapping("/good")
class GoodController(
    private val goodService: GoodService
) {
    /**
     * 특정 게시물에 좋아요를 추가합니다.
     *
     * @param postId 좋아요를 추가할 게시물의 ID
     * @return 좋아요 추가 결과
     */
    @PostMapping
    fun addGoodToPost(@RequestParam(name = "post_id") postId: Long): ResponseEntity<Any> {
        // ... (이전에 제안한 구현)
    }

    /**
     * 특정 게시물에서 좋아요를 제거합니다.
     *
     * @param postId 좋아요를 제거할 게시물의 ID
     * @return 좋아요 제거 결과
     */
    @DeleteMapping
    fun removeGoodFromPost(@RequestParam(name = "post_id") postId: Long): ResponseEntity<Any> {
        // ... (이전에 제안한 구현)
    }
}

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between 774f1c3 and 8d7635e.

📒 Files selected for processing (8)

src/main/kotlin/org/meogo/domain/bookmark/Bookmark.kt (1 hunks)
src/main/kotlin/org/meogo/domain/bookmark/BookmarkRepository.kt (1 hunks)
src/main/kotlin/org/meogo/domain/bookmark/BookmarkService.kt (1 hunks)
src/main/kotlin/org/meogo/domain/good/domain/Good.kt (1 hunks)
src/main/kotlin/org/meogo/domain/good/domain/GoodRepository.kt (1 hunks)
src/main/kotlin/org/meogo/domain/good/presentation/GoodController.kt (1 hunks)
src/main/kotlin/org/meogo/domain/good/service/GoodService.kt (1 hunks)
src/main/kotlin/org/meogo/domain/post/domain/Post.kt (1 hunks)

🔇 Additional comments (10)

src/main/kotlin/org/meogo/domain/good/domain/GoodRepository.kt (1)

1-10: 전반적인 구조가 잘 구성되어 있습니다.

패키지 선언, 임포트, 그리고 인터페이스 정의가 적절하게 이루어져 있습니다. JpaRepository를 상속받아 Good 엔티티에 대한 기본적인 CRUD 작업을 수행할 수 있도록 잘 설계되었습니다.

src/main/kotlin/org/meogo/domain/bookmark/BookmarkRepository.kt (1)

1-6: 패키지 선언과 임포트가 적절합니다.

패키지 선언과 임포트가 BookmarkRepository 인터페이스의 기능에 맞게 잘 구성되어 있습니다.

src/main/kotlin/org/meogo/domain/bookmark/Bookmark.kt (2)

1-10: 패키지 선언과 임포트가 적절합니다.

패키지 선언과 임포트된 클래스들이 Bookmark 엔티티의 구현에 필요한 요소들을 잘 포함하고 있습니다. 불필요한 임포트도 없어 깔끔해 보입니다.

17-20: User 엔티티와의 관계 설정이 적절합니다.

User 엔티티와의 다대일 관계가 잘 설정되어 있습니다. FetchType.LAZY를 사용하여 성능 최적화를 고려한 점과 user_id를 nullable = false로 설정하여 데이터 무결성을 보장한 점이 좋습니다.
src/main/kotlin/org/meogo/domain/good/domain/Good.kt (2)
3-10: import 문이 적절합니다.

필요한 모든 클래스가 올바르게 import되었으며, 불필요한 import는 없습니다.

1-1: 패키지 변경이 적절합니다.

패키지 구조가 개선되었습니다. 이는 코드 구조를 더 명확하게 만듭니다.

다른 파일에서 이 클래스를 import하는 부분을 확인하기 위해 다음 스크립트를 실행하세요:

✅ Verification successful

Good 클래스의 패키지 변경이 올바르게 완료되었습니다.

모든 import 문이 새로운 패키지 경로를 사용하고 있습니다.
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:
#!/bin/bash
# 설명: Good 클래스의 import 문을 검색합니다.

# 테스트: Good 클래스의 import 문을 검색합니다. 예상 결과: 새로운 패키지 경로를 사용하는 import 문만 나타나야 합니다.
rg --type kotlin "import.*org\.meogo\.domain\.good\..*Good"
Length of output: 393
src/main/kotlin/org/meogo/domain/good/presentation/GoodController.kt (1)

14-18: 클래스 선언이 적절합니다

GoodController 클래스의 선언이 Spring 프레임워크의 모범 사례를 잘 따르고 있습니다. @RestController와 @RequestMapping("/good") 어노테이션이 올바르게 사용되었으며, 생성자를 통한 의존성 주입도 잘 구현되어 있습니다.
src/main/kotlin/org/meogo/domain/good/service/GoodService.kt (2)
1-10: 패키지 선언 및 임포트가 적절합니다.

패키지 선언과 필요한 모든 클래스들의 임포트가 올바르게 되어 있습니다. 코드 구조가 잘 정리되어 있습니다.

11-16: 클래스 선언과 의존성 주입이 적절합니다.

@Service 어노테이션과 생성자를 통한 의존성 주입이 올바르게 구현되어 있습니다. 좋은 구조입니다.

다음 스크립트를 실행하여 의존성들이 프로젝트에 올바르게 정의되어 있는지 확인하세요:

✅ Verification successful

의존성 확인 완료

GoodRepository, UserFacade, PostRepository가 프로젝트 내에 올바르게 정의되어 있습니다.
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:
#!/bin/bash
# 의존성 클래스들이 프로젝트에 존재하는지 확인

echo "Checking GoodRepository..."
rg --type kotlin "interface GoodRepository"

echo "Checking UserFacade..."
rg --type kotlin "class UserFacade"

echo "Checking PostRepository..."
rg --type kotlin "interface PostRepository"
Length of output: 620
src/main/kotlin/org/meogo/domain/post/domain/Post.kt (1)

Line range hint 52-58: null 값 처리에 대한 고려 필요

update 메서드에서 schoolId, keyWord, image에 대해 null 값을 허용하도록 변경되었습니다. 이는 유연성을 제공하지만, 잠재적인 문제를 야기할 수 있습니다.

다음 사항을 고려해보세요:

null 값이 의도적으로 허용되는지 확인

필요한 경우 null 값에 대한 유효성 검사 추가

이러한 변경이 애플리케이션의 다른 부분에 미치는 영향 검토

다음 스크립트를 실행하여 Post 클래스의 사용을 확인하세요:

coderabbitai · 2024-09-30T14:41:49Z

src/main/kotlin/org/meogo/domain/good/domain/GoodRepository.kt

+import java.util.UUID
+
+interface GoodRepository : JpaRepository<Good, UUID> {
+    fun findByUserAndPost(user: User, post: Post): Good


⚠️ Potential issue

findByUserAndPost 메서드의 반환 타입과 이름을 재고해 보세요.

현재 메서드 정의에는 두 가지 잠재적인 문제가 있습니다:

반환 타입이 non-nullable Good입니다. 만약 일치하는 엔티티가 없을 경우 이는 예외를 발생시킬 수 있습니다.

메서드 이름이 단일 엔티티를 반환할 것임을 암시하지만, 실제로는 사용자와 게시물 조합에 대해 여러 "좋아요"가 존재할 수 있습니다.

다음과 같이 수정하는 것을 고려해보세요:

-fun findByUserAndPost(user: User, post: Post): Good +fun findByUserAndPost(user: User, post: Post): Good?

또는 여러 엔티티를 반환하도록 변경:

-fun findByUserAndPost(user: User, post: Post): Good +fun findAllByUserAndPost(user: User, post: Post): List<Good>

선택한 접근 방식에 따라 서비스 레이어에서 적절히 처리해야 합니다.

Committable suggestion was skipped due to low confidence.

coderabbitai · 2024-09-30T14:41:49Z

src/main/kotlin/org/meogo/domain/bookmark/Bookmark.kt

+@Entity
+class Bookmark(
+    id: UUID? = null,
+
+    val schoolId: Int? = 0,


⚠️ Potential issue

schoolId 속성의 널 허용 여부와 기본값을 재고해 보세요.

schoolId가 널을 허용하고 기본값으로 0을 사용하는 것이 의도된 설계인지 확인해 주세요. 만약 schoolId가 필수 필드라면, 널을 허용하지 않도록 변경하고 의미 있는 기본값을 설정하거나 기본값 없이 필수 파라미터로 만드는 것이 좋습니다.

다음과 같이 변경을 고려해 보세요:

- val schoolId: Int? = 0, + val schoolId: Int,

또는 의미 있는 기본값이 없다면:

- val schoolId: Int? = 0, + val schoolId: Int,

그리고 생성자에서 schoolId를 필수 파라미터로 받도록 변경하세요.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

@Entity

class Bookmark(

id: UUID? = null,

val schoolId: Int? = 0,

@Entity

class Bookmark(

id: UUID? = null,

val schoolId: Int,

coderabbitai · 2024-09-30T14:41:49Z

src/main/kotlin/org/meogo/domain/good/presentation/GoodController.kt

@@ -0,0 +1,28 @@
+package org.meogo.domain.good.presentation
+
+import lombok.RequiredArgsConstructor


⚠️ Potential issue

Lombok 어노테이션 제거 필요

Kotlin에서는 Lombok이 필요하지 않습니다. @RequiredArgsConstructor 어노테이션과 import를 제거해주세요. Kotlin은 이미 주 생성자에서 프로퍼티를 선언할 때 자동으로 생성자 매개변수를 만들어줍니다.

다음과 같이 수정해주세요:

-import lombok.RequiredArgsConstructor - import org.meogo.domain.good.service.GoodService import org.springframework.http.HttpStatus import org.springframework.web.bind.annotation.DeleteMapping @@ -10,7 +9,6 @@ import org.springframework.web.bind.annotation.RequestParam import org.springframework.web.bind.annotation.ResponseStatus import org.springframework.web.bind.annotation.RestController -@RequiredArgsConstructor @RestController @RequestMapping("/good") class GoodController(

Also applies to: 13-13

coderabbitai · 2024-09-30T14:41:50Z

src/main/kotlin/org/meogo/domain/good/presentation/GoodController.kt

+    @ResponseStatus(HttpStatus.CREATED)
+    @PostMapping
+    fun addGood(@RequestParam(name = "post_id")postId: Long) =
+        goodService.addGood(postId)


🛠️ Refactor suggestion

addGood 메서드 개선 제안

addGood 메서드의 구현은 대체로 올바르지만, 다음과 같은 개선사항을 제안합니다:

@RequestParam 어노테이션의 공백을 일관성 있게 수정해주세요.

메서드 이름을 더 구체적으로 변경하는 것이 좋습니다 (예: addGoodToPost).

오류 처리 로직을 추가하는 것이 좋습니다.

다음과 같이 수정해보세요:

@ResponseStatus(HttpStatus.CREATED) @PostMapping fun addGoodToPost(@RequestParam(name = "post_id") postId: Long): ResponseEntity<Any> { return try { goodService.addGood(postId) ResponseEntity.status(HttpStatus.CREATED).build() } catch (e: Exception) { ResponseEntity.status(HttpStatus.BAD_REQUEST).body("Failed to add good: ${e.message}") } }

coderabbitai · 2024-09-30T14:41:50Z

src/main/kotlin/org/meogo/domain/good/presentation/GoodController.kt

+    @ResponseStatus(HttpStatus.NO_CONTENT)
+    @DeleteMapping
+    fun deleteGood(@RequestParam(name = "post_id")postId: Long) =
+        goodService.deleteGood(postId)


🛠️ Refactor suggestion

deleteGood 메서드 개선 제안

deleteGood 메서드도 addGood 메서드와 유사한 개선이 필요합니다:

@RequestParam 어노테이션의 공백을 일관성 있게 수정해주세요.

메서드 이름을 더 구체적으로 변경하는 것이 좋습니다 (예: removeGoodFromPost).

오류 처리 로직을 추가하는 것이 좋습니다.

다음과 같이 수정해보세요:

@DeleteMapping fun removeGoodFromPost(@RequestParam(name = "post_id") postId: Long): ResponseEntity<Any> { return try { goodService.deleteGood(postId) ResponseEntity.noContent().build() } catch (e: Exception) { ResponseEntity.status(HttpStatus.BAD_REQUEST).body("Failed to remove good: ${e.message}") } }

이렇게 수정하면 @ResponseStatus 어노테이션이 필요 없어지므로 제거할 수 있습니다.

coderabbitai · 2024-09-30T14:41:50Z

src/main/kotlin/org/meogo/domain/bookmark/BookmarkService.kt

+    fun queryBookmarkedPost(schoolId: Int): Int {
+        val posts = bookmarkRepository.findAllBySchoolId(schoolId)
+        return posts?.size ?: 0
+    }


🛠️ Refactor suggestion

메서드 이름 변경 및 쿼리 최적화 필요

현재 메서드 이름 queryBookmarkedPost는 북마크된 게시물을 조회하는 것처럼 보이지만, 실제로는 북마크 수를 반환합니다. 또한, 모든 북마크를 조회한 후 크기를 계산하는 것은 비효율적일 수 있습니다.

다음과 같이 개선할 수 있습니다:

메서드 이름을 countBookmarksBySchoolId로 변경하여 의도를 명확히 합니다.

쿼리를 최적화하여 직접 개수를 반환하도록 합니다.

fun countBookmarksBySchoolId(schoolId: Int): Int { return bookmarkRepository.countBySchoolId(schoolId) }

BookmarkRepository에 countBySchoolId 메서드를 추가하고, 이를 통해 데이터베이스 레벨에서 효율적으로 개수를 계산할 수 있습니다.

coderabbitai · 2024-09-30T14:41:50Z

src/main/kotlin/org/meogo/domain/bookmark/BookmarkService.kt

+    fun queryIsBookmarked(schoolId: Int): Boolean {
+        val user = userFacade.currentUser() ?: throw UserNotFoundException
+        return bookmarkRepository.existsBySchoolIdAndUser(schoolId, user)
+    }


🛠️ Refactor suggestion

예외 처리 일관성 개선 필요

UserNotFoundException을 메서드 내부에서 직접 처리하는 것은 일관성 없는 예외 처리를 야기할 수 있습니다.

예외 처리를 서비스 레벨 상위에서 통일적으로 처리하는 것이 좋습니다. 예를 들어:

fun queryIsBookmarked(schoolId: Int): Boolean { val user = userFacade.currentUser() return bookmarkRepository.existsBySchoolIdAndUser(schoolId, user) }

그리고 UserFacade에서 currentUser()가 null을 반환하지 않고 항상 UserNotFoundException을 던지도록 수정하세요. 이렇게 하면 모든 메서드에서 일관된 방식으로 사용자 존재 여부를 처리할 수 있습니다.

coderabbitai · 2024-09-30T14:41:50Z

src/main/kotlin/org/meogo/domain/bookmark/BookmarkService.kt

+    fun deleteBookmark(schoolId: Int) {
+        val user = userFacade.currentUser() ?: throw UserNotFoundException
+        bookmarkRepository.deleteBySchoolIdAndUser(schoolId, user)
+    }


⚠️ Potential issue

트랜잭션 처리 및 예외 상황 고려 필요

deleteBookmark 메서드에 몇 가지 개선이 필요합니다:

@Transactional 어노테이션이 누락되어 있습니다.

삭제하려는 북마크가 존재하지 않는 경우에 대한 처리가 없습니다.

다음과 같이 수정하는 것이 좋습니다:

@Transactional fun deleteBookmark(schoolId: Int) { val user = userFacade.currentUser() val deleted = bookmarkRepository.deleteBySchoolIdAndUser(schoolId, user) if (deleted == 0) { // 북마크가 존재하지 않는 경우에 대한 처리 // 예: throw BookmarkNotFoundException(schoolId, user.id) } }

이렇게 하면 트랜잭션 처리가 보장되고, 존재하지 않는 북마크를 삭제하려는 시도를 적절히 처리할 수 있습니다.

coderabbitai · 2024-09-30T14:41:50Z

src/main/kotlin/org/meogo/domain/bookmark/BookmarkService.kt

+@Service
+class BookmarkService(
+    private val bookmarkRepository: BookmarkRepository,
+    private val userFacade: UserFacade,
+    private val postRepository: PostRepository
+) {


⚠️ Potential issue

사용되지 않는 의존성 제거 필요

PostRepository가 주입되었지만 클래스 내에서 사용되지 않고 있습니다. 이는 불필요한 의존성을 야기할 수 있습니다.

다음과 같이 수정하는 것이 좋습니다:

@Service class BookmarkService( private val bookmarkRepository: BookmarkRepository, - private val userFacade: UserFacade, - private val postRepository: PostRepository + private val userFacade: UserFacade ) {

만약 PostRepository가 향후 사용될 예정이라면, 주석을 추가하여 그 의도를 명확히 해주세요.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

@Service

class BookmarkService(

private val bookmarkRepository: BookmarkRepository,

private val userFacade: UserFacade,

private val postRepository: PostRepository

) {

@Service

class BookmarkService(

private val bookmarkRepository: BookmarkRepository,

private val userFacade: UserFacade

) {

coderabbitai · 2024-09-30T14:41:51Z

src/main/kotlin/org/meogo/domain/post/domain/Post.kt

+    fun deleteGood() {
+        this.good -= 1
+    }


⚠️ Potential issue

음수 값 체크 추가 필요

deleteGood() 메서드에서 good 속성이 0 이하로 내려가지 않도록 체크하는 것이 좋습니다. 이는 잘못된 상태를 방지하는 데 도움이 됩니다.

다음과 같이 수정하는 것을 고려해보세요:

fun deleteGood() { - this.good -= 1 + if (this.good > 0) { + this.good -= 1 + } }

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

fun deleteGood() {

this.good -= 1

}

fun deleteGood() {

if (this.good > 0) {

this.good -= 1

}

}

meltapplee added 3 commits September 30, 2024 23:38

refactor :: bookmark

e228963

add :: good

6fde974

add :: add delete good api

8d7635e

meltapplee added the 기능개발 make something label Sep 30, 2024

meltapplee self-assigned this Sep 30, 2024

coderabbitai bot reviewed Sep 30, 2024

View reviewed changes

meltapplee merged commit 5f035ca into main Oct 2, 2024
2 checks passed

coderabbitai bot mentioned this pull request Oct 2, 2024

🏄 :: (Meogo-44) bookmark #45

Merged

meltapplee deleted the 42-post-good branch October 18, 2024 03:21

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🏄 :: (Meogo-42) post good #43

🏄 :: (Meogo-42) post good #43

meltapplee commented Sep 30, 2024 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Sep 30, 2024 •

edited

Loading

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

coderabbitai bot left a comment

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

coderabbitai bot Sep 30, 2024

		@@ -0,0 +1,28 @@
		package org.meogo.domain.good.presentation

		import lombok.RequiredArgsConstructor

🏄 :: (Meogo-42) post good #43

🏄 :: (Meogo-42) post good #43

Conversation

meltapplee commented Sep 30, 2024 • edited by coderabbitai bot Loading

Summary by CodeRabbit

coderabbitai bot commented Sep 30, 2024 • edited Loading

Walkthrough

Changes

Possibly related PRs

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 30, 2024

Choose a reason for hiding this comment

meltapplee commented Sep 30, 2024 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Sep 30, 2024 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)