🏄 :: (Meogo-36) detault image #37

meltapplee · 2024-09-23T00:22:11Z

close #36

Summary by CodeRabbit

새로운 기능
- 사용자 UUID를 기반으로 게시물을 검색할 수 있는 기능 추가.
- 사용자가 자신의 게시물을 조회할 수 있는 새로운 GET 엔드포인트 /query/my 추가.
- 사용자 프로필 이미지에 대한 기본 이미지 처리 기능 개선.
버그 수정
- 학교 ID로 게시물을 검색하는 메서드의 반환 타입을 nullable로 변경하여 안정성 향상.
문서화
- AWS 설정을 위한 새로운 구성 섹션 추가.

coderabbitai · 2024-09-23T00:22:25Z

Walkthrough

이번 변경사항은 PostRepository, PostController, QueryMyPostService 등 여러 클래스에서 메서드 추가 및 수정, 새로운 데이터 클래스 생성, 서비스 클래스의 의존성 주입 변경 등을 포함합니다. 사용자의 UUID를 기반으로 게시물을 조회하는 기능이 추가되었으며, 사용자 프로필 이미지 처리 및 응답 구조가 간소화되었습니다. AWS 설정을 위한 새로운 구성 섹션도 추가되었습니다.

Changes

파일 경로	변경 요약
`src/main/kotlin/org/meogo/domain/post/domain/PostRepository.kt`	`findByUserId(userId: UUID): List<Post>` 메서드 추가, `findBySchoolId(schoolId: Int): List<Post>?`로 수정.
`src/main/kotlin/org/meogo/domain/post/presentation/PostController.kt`	`QueryMyPostService`를 추가하고 `/query/my` GET 엔드포인트 생성.
`src/main/kotlin/org/meogo/domain/post/presentation/dto/response/QueryMyPostResponse.kt`	`QueryMyPostResponse` 데이터 클래스 추가, 게시물 응답 구조 정의.
`src/main/kotlin/org/meogo/domain/post/service/QueryMyPostService.kt`	사용자 게시물 조회 기능 추가, 트랜잭션 처리 및 댓글 수 계산 로직 포함.
`src/main/kotlin/org/meogo/domain/post/service/QuerySchoolPostService.kt`	`execute` 메서드의 반환 타입을 nullable로 변경하고, null 처리 로직 추가.
`src/main/kotlin/org/meogo/domain/user/domain/User.kt`	`profile` 속성을 non-nullable로 변경, `updateProfile` 메서드의 매개변수 수정.
`src/main/kotlin/org/meogo/domain/user/presentation/dto/response/MyPageResponse.kt`	`enrolledSchool` 속성을 nullable로 추가.
`src/main/kotlin/org/meogo/domain/user/service/MyPageService.kt`	`fileUtil` 의존성 제거 및 응답 구조 간소화.
`src/main/kotlin/org/meogo/domain/user/service/UploadProfileService.kt`	프로필 이미지 삭제 로직 수정 및 `defaultImage` 속성 추가.
`src/main/kotlin/org/meogo/domain/user/service/UserSignUpService.kt`	프로필 이미지 URL 생성 로직 추가 및 생성자 매개변수 수정.
`src/main/resources/application.yml`	AWS 관련 새로운 구성 섹션 추가.

Possibly related PRs

🏄 :: (Meogo-20) user my page #21: PostRepository 인터페이스에 findByUserId(userId: UUID): List<Review>? 메서드 추가, 사용자 식별자 기반 데이터 검색 관련.
🏄 :: (Meogo-22) post #24: PostRepository 인터페이스가 도입되어 Post 엔티티 관리 메서드 포함, 본 PR의 변경 사항과 밀접한 관련.

🐇 변화의 바람이 불어와
새로운 기능이 쑥쑥 자라나
사용자 게시물 찾는 길이 열려
프로필도 더 밝고 예쁘게
코드로 가득한 이 세상에
기쁨의 춤을 춥니다, 후루룩! 🌟

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: 2

Outside diff range and nitpick comments (2)

src/main/resources/application.yml (1)

30-41: 잘 구현되었습니다!

AWS 설정을 위한 cloud 구성 섹션이 올바르게 추가되었습니다. 중요한 정보에 환경 변수를 사용하는 것은 좋은 방법입니다.

region이 ap-northeast-2로 하드코딩되어 있는데, 향후 다른 리전에 배포해야 할 경우를 대비해 환경 변수를 사용하는 것이 좋겠습니다.

src/main/kotlin/org/meogo/domain/post/service/QueryMyPostService.kt (1)

11-35: 기능의 정확성을 보장하기 위해 단위 테스트를 추가하는 것이 좋습니다.

QueryMyPostService 클래스의 execute() 메서드에 대한 단위 테스트를 작성하여 다양한 시나리오에서 기능이 예상대로 작동하는지 확인하세요. 이를 통해 향후 변경 사항이 기존 동작을 손상시키지 않도록 할 수 있습니다.

Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits

Files that changed from the base of the PR and between 43117cc and 2c935d2.

Files selected for processing (11)

src/main/kotlin/org/meogo/domain/post/domain/PostRepository.kt (2 hunks)
src/main/kotlin/org/meogo/domain/post/presentation/PostController.kt (3 hunks)
src/main/kotlin/org/meogo/domain/post/presentation/dto/response/QueryMyPostResponse.kt (1 hunks)
src/main/kotlin/org/meogo/domain/post/service/QueryMyPostService.kt (1 hunks)
src/main/kotlin/org/meogo/domain/post/service/QuerySchoolPostService.kt (1 hunks)
src/main/kotlin/org/meogo/domain/user/domain/User.kt (1 hunks)
src/main/kotlin/org/meogo/domain/user/presentation/dto/response/MyPageResponse.kt (1 hunks)
src/main/kotlin/org/meogo/domain/user/service/MyPageService.kt (2 hunks)
src/main/kotlin/org/meogo/domain/user/service/UploadProfileService.kt (2 hunks)
src/main/kotlin/org/meogo/domain/user/service/UserSignUpService.kt (1 hunks)
src/main/resources/application.yml (1 hunks)

Additional comments not posted (20)

src/main/kotlin/org/meogo/domain/user/presentation/dto/response/MyPageResponse.kt (1)

6-6: 잘 되었습니다!

enrolledSchool 속성을 추가하여 MyPageResponse 데이터 모델을 확장한 것은 적절해 보입니다. 널 허용 타입을 사용하여 선택적인 값을 잘 표현했습니다. 이름 지정과 데이터 타입도 기존 코드 스타일과 일관성이 있습니다.

src/main/kotlin/org/meogo/domain/post/presentation/dto/response/QueryMyPostResponse.kt (1)

1-10: 잘 정의된 DTO 클래스입니다!

QueryMyPostResponse 데이터 클래스는 사용자의 게시물을 조회하기 위한 응답 DTO로 잘 정의되어 있습니다. 클래스의 속성들은 게시물의 주요 속성을 포함하고 있으며, 속성 이름과 타입이 명확하고 적절합니다. 이 DTO는 사용자 게시물 조회 응답 데이터를 일관되고 구조화된 방식으로 표현하는 데 도움이 될 것입니다.
src/main/kotlin/org/meogo/domain/post/domain/PostRepository.kt (3)
4-4: 좋아 보입니다!

UUID 타입을 사용하기 위해 java.util.UUID 패키지를 임포트하는 것은 적절해 보입니다.

16-17: 좋아 보입니다!

findByUserId 메서드는 userId를 기반으로 Post 목록을 조회하는 기능을 제공합니다. 메서드 시그니처와 반환 타입이 적절해 보입니다.

18-18: 메서드 시그니처 변경을 확인하세요.

findBySchoolId 메서드의 반환 타입이 List<Post>에서 List<Post>?로 변경되었습니다. 이는 메서드가 null을 반환할 수 있음을 나타냅니다.

이 메서드를 호출하는 코드에서는 nullable 반환 타입을 적절히 처리해야 합니다. null 체크를 추가하거나 안전 호출(?.)을 사용하는 등의 방법으로 null 케이스를 처리해주세요.

이 메서드의 사용 예시를 검색하려면 다음 스크립트를 실행하세요:

Verification successful

변경 사항을 승인합니다.

findBySchoolId 메서드의 반환 타입이 List<Post>에서 List<Post>?로 변경된 것을 확인했습니다. 모든 호출부에서 nullable 반환 타입을 적절히 처리하고 있습니다.
Scripts executed

The following scripts were executed for the analysis:

Script:
#!/bin/bash
# Description: 이 스크립트는 `findBySchoolId` 메서드의 사용 예시를 검색합니다.

# Test: `findBySchoolId` 메서드 호출 검색. Expect: nullable 반환 타입 처리 여부 확인.
rg --type kotlin -A 5 $'findBySchoolId'
Length of output: 836
src/main/kotlin/org/meogo/domain/user/service/MyPageService.kt (2)

11-12: 생성자 단순화 승인

MyPageService 생성자에서 fileUtil 종속성을 제거하여 생성자를 단순화하고 MyPageService와 FileUtil 간의 결합도를 낮추는 것을 승인합니다.

fileUtil을 더 이상 사용하지 않는다는 것은 이 서비스 내에서 파일 유틸리티 기능이 더 이상 활용되지 않음을 나타냅니다. fileUtil을 사용했던 코드가 이에 맞게 업데이트되었는지 확인하세요.

21-22: 응답 구조 단순화 승인

execute 메서드의 응답 구조를 단순화하여 프로필 이미지의 URL을 생성하지 않고 user.enrolledSchool과 user.profile 속성을 직접 MyPageResponse에 반환하는 것을 승인합니다.

이 변경은 생성자에서 fileUtil 종속성을 제거하는 것과 일치합니다. MyPageResponse를 사용하는 클라이언트가 새로운 응답 구조를 처리하도록 업데이트되었는지 확인하세요.

src/main/kotlin/org/meogo/domain/post/service/QuerySchoolPostService.kt (1)

16-17: 좋아 보입니다!

함수 서명을 변경하여 schoolId에 해당하는 게시물이 없는 경우를 명시적으로 처리할 수 있게 되었습니다. null 대신 빈 리스트를 반환하는 것은 잠재적인 null 포인터 예외를 방지하고 코드의 가독성을 높이는 좋은 방법입니다.

함수 로직이 정확하고 문제가 없어 보입니다.
src/main/kotlin/org/meogo/domain/user/domain/User.kt (2)
30-33: updateProfile 메서드의 파라미터 타입 변경을 검토하세요.

updateProfile 메서드의 파라미터 타입을 널 허용하지 않도록 변경하면 업데이트된 프로필 속성 정의와 일치하게 되고 타입 안전성이 향상됩니다. 그러나 이 변경으로 인해 updateProfile 메서드 호출 시 널이 아닌 프로필 문자열을 항상 전달해야 합니다.

코드베이스에서 updateProfile 메서드를 호출하는 모든 부분을 검토하여 널이 아닌 프로필 문자열이 전달되는지 확인하세요.

다음 스크립트를 실행하여 updateProfile 메서드 호출을 검증하세요:
#!/bin/bash
# Description: 모든 `updateProfile` 메서드 호출이 널이 아닌 프로필 인수를 제공하는지 검증합니다.

# Test: `updateProfile` 메서드 호출을 검색합니다. Expect: 프로필 인수가 누락되거나 널인 호출이 없어야 합니다.
rg --type kotlin -A 5 $'updateProfile\('
25-25: 프로필 속성의 널 허용 여부 변경을 검토하세요.

프로필 속성을 널 허용하지 않도록 변경하면 User 클래스의 타입 안전성이 향상됩니다. 그러나 이 변경으로 인해 프로필 속성이 널일 수 있거나 기본값이 null인 것에 의존하는 기존 코드가 깨질 수 있습니다.

코드베이스에서 User 인스턴스를 생성하는 모든 부분을 검토하여 항상 유효한 프로필 문자열이 제공되는지 확인하세요.

다음 스크립트를 실행하여 User 생성자 호출을 검증하세요:
src/main/kotlin/org/meogo/domain/user/service/UploadProfileService.kt (3)

8-8: 잘 되었습니다!

@Value 어노테이션을 사용하기 위해 필요한 import 문이 추가되었습니다. 이 변경 사항은 적절해 보입니다.

17-20: 잘 되었습니다!

UserRepository와 defaultImage 속성이 생성자를 통해 주입되도록 변경되었습니다. 이는 Spring의 의존성 주입 패턴을 따르고 있으며, @Value 어노테이션을 사용하여 설정 파일에서 기본 이미지 URL을 읽어오는 것도 적절해 보입니다.

27-27: 잘 되었습니다!

사용자의 프로필 이미지가 기본 이미지인 경우 삭제하지 않도록 조건이 변경되었습니다. 이는 불필요한 기본 이미지 삭제를 방지하여 스토리지 사용을 최적화할 수 있습니다. 이 변경 사항은 uploadProfile 메서드의 기능에 영향을 미치지 않으면서도 개선된 점이 있어 보입니다.

src/main/kotlin/org/meogo/domain/post/service/QueryMyPostService.kt (1)

11-35: 잘 구현되었습니다!

이 클래스는 현재 사용자의 게시물을 쿼리하고 관련 세부 정보를 반환하는 새로운 기능을 제공합니다. 전반적인 구현이 깔끔하고 @Transactional을 사용하여 데이터 일관성을 보장합니다.

src/main/kotlin/org/meogo/domain/user/service/UserSignUpService.kt (4)

9-11: LGTM!

새로운 import 문은 FileUtil과 Path 클래스를 사용하기 위해 필요하며, @Value 어노테이션은 애플리케이션 속성에서 값을 주입하는 데 사용됩니다. import 문에는 문제가 없어 보입니다.

19-22: LGTM!

UserSignUpService 클래스의 생성자에 FileUtil 타입의 fileUtil과 String 타입의 defaultImage 파라미터가 추가되었습니다. 이를 통해 회원 가입 시 사용자의 프로필 이미지를 처리할 수 있게 되었습니다.

defaultImage 파라미터는 @Value 어노테이션을 사용하여 애플리케이션 속성에서 주입되므로, 코드를 수정하지 않고도 기본 이미지를 변경할 수 있는 유연성을 제공합니다.

파라미터 선언과 어노테이션 사용이 적절해 보입니다.

26-27: LGTM!

fileUtil.generateObjectUrl 메서드를 사용하여 사용자의 프로필 이미지 URL을 생성하는 코드가 추가되었습니다. defaultImage를 이미지 소스로 사용하고, Path.USER를 이미지 저장 경로로 사용하여 URL을 생성합니다.

이 코드를 통해 회원 가입 시 사용자에게 기본 프로필 이미지가 설정됩니다. Path.USER를 사용하는 것으로 보아, 이미지는 사용자 관련 파일을 위한 특정 디렉토리에 저장되는 것으로 보입니다.

코드 변경 사항이 적절해 보입니다.

33-34: LGTM!

User 객체의 role 속성에 후행 쉼표가 추가되었습니다. 이는 코드 포맷팅의 일관성을 위한 것으로, 기능적인 영향은 없습니다.

User 객체에 profile 속성이 추가되었고, 그 값은 image 변수로 설정됩니다. 이를 통해 사용자의 프로필 이미지 URL을 저장하고 계정과 연결할 수 있게 되었습니다.

코드 변경 사항이 적절해 보입니다.

src/main/kotlin/org/meogo/domain/post/presentation/PostController.kt (2)

9-9: 좋아 보입니다!

QueryMyPostService를 사용하기 위해 import 문이 필요합니다. 변경 사항이 올바르게 보입니다.

34-35: 좋아 보입니다!

QueryMyPostService를 주입하기 위해 생성자 매개변수가 추가되었습니다. 변경 사항이 올바르게 보입니다.

coderabbitai · 2024-09-23T00:28:11Z

src/main/kotlin/org/meogo/domain/post/service/QueryMyPostService.kt

+    fun execute(): List<QueryMyPostResponse> {
+        val user = userFacade.currentUser() ?: throw UserNotFoundException
+        val posts = postRepository.findByUserId(user.id!!)
+
+        return posts.map {
+                post ->
+            val comment = commentRepository.findByPost(post).count()
+            QueryMyPostResponse(
+                id = post.id,
+                title = post.title,
+                content = post.content,
+                date = post.format(post.date),
+                good = post.good,
+                comment = comment
+            )
+        }.sortedByDescending { it.date }
+    }


확장성을 위해 페이지네이션 지원을 추가하는 것이 좋습니다.

사용자의 게시물 수가 많아질 경우 모든 게시물을 한 번에 로드하는 것은 성능 문제를 야기할 수 있습니다. 페이지네이션을 구현하여 게시물을 청크 단위로 로드하는 것이 좋습니다.

디자인이 확정 나면 수정하도록 할게

@meltapplee, 알겠습니다. 디자인이 확정된 후에 수정하실 예정이시군요. 해당 내용을 추후에 다룰 수 있도록 새로운 GitHub 이슈를 등록할까요?

@meltapplee, 알겠습니다. 필요하실 때 언제든지 말씀해주세요.

(｡•́︿•̀｡)

src/main/kotlin/org/meogo/domain/post/presentation/PostController.kt

meltapplee added 3 commits September 23, 2024 09:19

add :: default image

53c20d5

add :: myPost api

87fe217

modify :: querySchoolPost response

2c935d2

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

meltapplee self-assigned this Sep 23, 2024

coderabbitai bot reviewed Sep 23, 2024

View reviewed changes

meltapplee merged commit b1313a1 into main Sep 23, 2024
2 checks passed

meltapplee deleted the 36-detault-image branch September 23, 2024 01:33

coderabbitai bot mentioned this pull request Sep 25, 2024

🏄 :: (Meogo-38) question crud #39

Merged

coderabbitai bot mentioned this pull request Oct 8, 2024

🏄 :: (Meogo-48) modify my page #49

Merged

This was referenced Oct 18, 2024

🏄 :: (Meogo-14)refactor #50

Merged

🏄 :: (Meogo-52) feign client #53

Merged

coderabbitai bot mentioned this pull request Dec 4, 2024

🏄 :: (Meogo-57) bookmark test #58

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🏄 :: (Meogo-36) detault image #37

🏄 :: (Meogo-36) detault image #37

meltapplee commented Sep 23, 2024 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Sep 23, 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 23, 2024 •

edited

Loading

meltapplee Sep 23, 2024

coderabbitai bot Sep 23, 2024

meltapplee Sep 23, 2024

coderabbitai bot Sep 23, 2024

🏄 :: (Meogo-36) detault image #37

🏄 :: (Meogo-36) detault image #37

Conversation

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

Summary by CodeRabbit

coderabbitai bot commented Sep 23, 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 23, 2024 • edited Loading

Choose a reason for hiding this comment

meltapplee Sep 23, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 23, 2024

Choose a reason for hiding this comment

meltapplee Sep 23, 2024

Choose a reason for hiding this comment

coderabbitai bot Sep 23, 2024

Choose a reason for hiding this comment

meltapplee commented Sep 23, 2024 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Sep 23, 2024 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)

coderabbitai bot Sep 23, 2024 •

edited

Loading