🏄 :: (Meogo-48) modify my page

[meltapplee]

close #48

Summary by CodeRabbit

• 새로운 기능

  • 사용자 프로필 업데이트 기능 개선: 이름과 등록 학교를 수정할 수 있는 기능 추가.
  • 사용자 정보를 수정하기 위한 새로운 요청 DTO인 UserModifyRequest 추가.
  • 사용자 정보를 조회하기 위한 새로운 서비스 QueryMyPageService 추가.
  • 사용자 ID로 조회 시 예외 처리 개선으로 사용자 조회 시 더 안전한 흐름 제공.
  • 사용자 조회 메서드에서 null 체크 추가로 안전성 향상.

• 버그 수정

  • 사용자 조회 시 예외 처리 제거로 인해 더 간단한 흐름으로 변경.

• 리팩토링

  • UploadProfileService가 ModifyUserInfoService로 이름 변경 및 기능 개선.
  • MyPageService가 QueryMyPageService로 이름 변경.

[coderabbitai]

Walkthrough

이 변경 사항은 User 클래스, UserRepository 인터페이스, UserFacade 클래스, UserController 클래스 및 관련 서비스 클래스에 대한 여러 수정 사항을 포함합니다. 주요 변경 사항으로는 User 클래스의 속성 가시성을 수정하고, updateProfile 메서드의 매개변수를 확장하며, UserRepository 인터페이스가 JPA에서 일반 리포지토리로 변경되었습니다. 또한, 사용자 검색 메서드에서 예외 처리를 제거하고, 새로운 DTO 및 서비스 클래스가 도입되었습니다.

Changes

파일 경로	변경 요약
src/main/kotlin/org/meogo/domain/user/domain/User.kt	name 및 enrolledSchool 속성을 var로 변경하고, updateProfile 메서드를 수정하여 세 개의 매개변수를 받도록 변경.
src/main/kotlin/org/meogo/domain/user/domain/UserRepository.kt	JpaRepository에서 Repository로 변경하고, findById 메서드를 추가하며, findByAccountId 메서드를 수정하여 반환 타입을 변경.
src/main/kotlin/org/meogo/domain/user/facade/UserFacade.kt	getUserByAccountId 및 getUserById 메서드에서 예외 처리를 제거하고, 반환 타입을 nullable로 변경.
src/main/kotlin/org/meogo/domain/user/presentation/UserController.kt	두 개의 새로운 DTO를 추가하고, myPage 메서드를 수정하며, updateProfile 메서드의 매개변수를 변경.
src/main/kotlin/org/meogo/domain/user/presentation/dto/request/UserModifyRequest.kt	UserModifyRequest라는 새로운 데이터 클래스를 추가.
src/main/kotlin/org/meogo/domain/user/service/ModifyUserInfoService.kt	UploadProfileService를 ModifyUserInfoService로 이름 변경하고, 메서드 및 생성자 매개변수를 수정.
src/main/kotlin/org/meogo/domain/user/service/QueryMyPageService.kt	MyPageService를 QueryMyPageService로 이름 변경.
src/main/kotlin/org/meogo/domain/user/service/UserSignInService.kt	UserNotFoundException 임포트를 제거하고, 예외 처리를 수정.
src/main/kotlin/org/meogo/global/auth/AuthDetailsService.kt	loadUserByUsername 메서드에서 사용자 검색 시 null 체크를 추가하고, 사용자 미발견 시 UserNotFoundException을 발생시키도록 변경.
src/main/kotlin/org/meogo/domain/review/service/QueryAllBySchoolIdService.kt	사용자 검색 시 null 체크를 추가하고, 사용자 미발견 시 UserNotFoundException을 발생시키도록 변경.

Possibly related PRs

• 🏄 :: (Meogo-3) user #4: User 클래스의 변경 사항은 PR #4의 User 클래스 수정과 관련이 있으며, 사용자 정보 관리 방법에 대한 변경을 포함합니다.
• 🏄 :: (Meogo-5) signup signin check #6: PR #6의 UserController 변경 사항은 사용자 관리 기능 업데이트와 관련이 있으며, User 클래스 및 해당 메서드와 직접 연결됩니다.
• 🏄 :: (Meogo-20) user my page #21: PR #21의 MyPageService 및 MyPageResponse 업데이트는 사용자 정보 검색 및 표현과 관련이 있으며, User 클래스의 변경 사항과 연관됩니다.
• 🏄 :: (Meogo-36) detault image #37: 프로필 처리와 관련된 User 클래스의 변경 사항은 사용자 프로필 관리와 상호 작용할 수 있는 PR #37의 수정 사항과 관련이 있습니다.
• 🏄 :: (Meogo-40) question tag detail #41: User 클래스의 수정 사항은 사용자 상호 작용과 관련된 기능 향상을 목표로 하는 PR #41과도 관련이 있습니다.

🐇 변화의 바람이 불어와
사용자 정보가 새로워져요.
프로필 업데이트, 더 간편하게,
모두의 이야기가 담길 거예요!
함께 뛰어놀며, 더 나은 세상으로,
우리 모두의 꿈을 이뤄가요! 🌟

---

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

• X
• Mastodon
• Reddit
• LinkedIn

🪧 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: 9

🧹 Outside diff range and nitpick comments (3)

src/main/kotlin/org/meogo/domain/user/service/QueryMyPageService.kt (1)

10-10: 클래스 이름 변경이 적절합니다.

클래스 이름을 MyPageService에서 QueryMyPageService로 변경한 것은 CQRS 패턴의 Query 부분과 일치하여 좋은 변경입니다. 이는 서비스의 목적을 더 명확히 나타냅니다.

하나의 작은 개선 사항을 제안드립니다:

execute() 메서드에서 userFacade.currentUser()의 결과를 처리할 때, 엘비스 연산자(?:) 대신 requireNotNull()을 사용하는 것이 더 명시적일 수 있습니다. 이렇게 하면 코드의 의도가 더 명확해집니다.

fun execute(): MyPageResponse {
    val user = requireNotNull(userFacade.currentUser()) { "User not found" }
    
    return MyPageResponse(
        user.name,
        user.accountId,
        user.enrolledSchool,
        user.profile
    )
}

이 변경은 선택사항이며, 현재 구현도 충분히 명확하고 효과적입니다.

src/main/kotlin/org/meogo/domain/user/facade/UserFacade.kt (1)

currentUser 메서드 사용 시 안전한 null 처리 필요

currentUser 메서드가 nullable 타입을 반환하므로, 일부 서비스 클래스에서 !! 연산자를 사용하여 강제로 언래핑하고 있습니다. 이는 NullPointerException을 유발할 수 있으므로, 다음 파일들에서 null 처리를 안전하게 구현하도록 수정이 필요합니다:

• src/main/kotlin/org/meogo/domain/comment/service/DeleteCommentService.kt
• src/main/kotlin/org/meogo/domain/post/service/DeletePostService.kt

🔗 Analysis chain

Line range hint 15-18: currentUser 메서드 검토 필요

getUserByAccountId 메서드가 이제 nullable 타입을 반환하므로, currentUser 메서드의 구현을 검토해야 합니다.

1. currentUser 메서드는 여전히 User?를 반환하고 있어 타입 측면에서는 문제가 없습니다.
2. 하지만 getUserByAccountId의 변경으로 인해 currentUser 메서드의 동작이 의도한 대로인지 확인이 필요합니다.
3. null 처리 로직이 적절한지 검토하고, 필요하다면 추가적인 null 체크를 고려해보세요.

currentUser 메서드의 호출 지점들을 확인하여 null 처리가 적절히 이루어지고 있는지 검증해 보세요. 다음 스크립트를 실행하여 currentUser의 사용 패턴을 확인할 수 있습니다:

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Find usages of currentUser() method
rg --type kotlin "\.currentUser\(\)" -A 5

Length of output: 14407

src/main/kotlin/org/meogo/domain/user/presentation/UserController.kt (1)

32-33: 의존성 주입이 적절히 업데이트되었습니다.

새로운 서비스 클래스들이 올바르게 주입되었습니다. 이는 컨트롤러 메서드의 변경 사항과 일치합니다.

가독성 향상을 위해 의존성 주입 순서를 알파벳 순으로 정렬하는 것을 고려해 보세요. 예를 들어:

class UserController(
    private val checkAccountIdService: CheckAccountIdService,
    private val modifyUserInfoService: ModifyUserInfoService,
    private val queryMyPageService: QueryMyPageService,
    private val userSignInService: UserSignInService,
    private val userSignUpService: UserSignUpService
) {
    // ...
}

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between e8146f0 and 04ad08f.

📒 Files selected for processing (8)

• src/main/kotlin/org/meogo/domain/user/domain/User.kt (1 hunks)
• src/main/kotlin/org/meogo/domain/user/domain/UserRepository.kt (1 hunks)
• src/main/kotlin/org/meogo/domain/user/facade/UserFacade.kt (1 hunks)
• src/main/kotlin/org/meogo/domain/user/presentation/UserController.kt (3 hunks)
• src/main/kotlin/org/meogo/domain/user/presentation/dto/request/UserModifyRequest.kt (1 hunks)
• src/main/kotlin/org/meogo/domain/user/service/ModifyUserInfoService.kt (2 hunks)
• src/main/kotlin/org/meogo/domain/user/service/QueryMyPageService.kt (1 hunks)
• src/main/kotlin/org/meogo/domain/user/service/UserSignInService.kt (0 hunks)

💤 Files with no reviewable changes (1)

• src/main/kotlin/org/meogo/domain/user/service/UserSignInService.kt

🧰 Additional context used

🔇 Additional comments (7)

src/main/kotlin/org/meogo/domain/user/presentation/dto/request/UserModifyRequest.kt (1)

1-6: 전반적인 구조와 명명이 적절합니다.

데이터 클래스의 사용과 속성 이름이 명확하고 이해하기 쉽습니다. DTO로서의 역할을 잘 수행할 것으로 보입니다.

src/main/kotlin/org/meogo/domain/user/domain/User.kt (2)

15-15: name 속성의 가변성 변경에 대한 고려 필요

name 속성을 val에서 var로 변경하면 객체의 상태가 변경 가능해집니다. 이는 의도된 변경이지만, 다음 사항을 고려해야 합니다:

1. 스레드 안전성: 여러 스레드에서 동시에 이 속성에 접근할 경우 동기화 문제가 발생할 수 있습니다.
2. 불변성 손실: 객체의 일관성을 유지하기 어려워질 수 있습니다.
3. 예측 가능성 감소: 객체의 상태가 언제든 변경될 수 있어 코드의 예측 가능성이 낮아질 수 있습니다.

이러한 변경이 필요한 이유와 잠재적인 영향을 팀과 논의하는 것이 좋습니다.

#!/bin/bash
# 다른 파일에서 User 객체의 name 속성을 직접 변경하는 코드가 있는지 확인
rg --type kotlin 'user\.name\s*=' -g '!**/User.kt'

---

23-23: enrolledSchool 속성의 가변성 변경에 대한 검토 필요

enrolledSchool 속성을 val에서 var로 변경한 것은 name 속성과 유사한 영향을 미칩니다:

1. 스레드 안전성: 멀티스레드 환경에서 동기화 이슈가 발생할 수 있습니다.
2. 불변성 손실: 객체의 일관성 유지가 더 어려워질 수 있습니다.
3. 예측 가능성 감소: 객체의 상태 변경이 더 자유로워져 코드의 예측이 어려워질 수 있습니다.

또한, enrolledSchool이 Int? 타입이며 기본값이 0인 것에 대해 고려해 보아야 합니다. 0이 유효한 값인지, 아니면 null을 사용하는 것이 더 적절한지 검토가 필요합니다.

이 변경의 필요성과 잠재적 영향에 대해 팀 내에서 논의하는 것이 좋습니다.

✅ Verification successful

enrolledSchool 속성 변경 검증 완료

변경 사항을 확인한 결과, 다른 파일에서 enrolledSchool 속성을 직접 변경하는 코드는 발견되지 않았습니다. 따라서, enrolledSchool을 var로 변경하는 데 따른 외부 영향은 없음을 확인했습니다.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 다른 파일에서 User 객체의 enrolledSchool 속성을 직접 변경하는 코드가 있는지 확인
rg --type kotlin 'user\.enrolledSchool\s*=' -g '!**/User.kt'

Length of output: 518

src/main/kotlin/org/meogo/domain/user/presentation/UserController.kt (3)

4-4: 새로운 import 문이 적절히 추가되었습니다.

새로운 DTO와 서비스 클래스에 대한 import 문이 올바르게 추가되었습니다. 이는 컨트롤러 클래스의 변경 사항과 일치합니다.

Also applies to: 9-10

---

53-53: myPage 메서드가 적절히 업데이트되었습니다.

myPage 메서드가 새로운 queryMyPageService를 사용하도록 올바르게 수정되었습니다. 메서드 시그니처와 반환 타입이 유지되어 API 계약을 잘 지키고 있습니다.

---

55-61: updateProfile 메서드가 적절히 개선되었습니다.

메서드 이름 변경, 새로운 매개변수 추가, 그리고 서비스 로직 변경이 잘 이루어졌습니다. 이 변경으로 사용자 정보 수정과 프로필 이미지 업로드를 함께 처리할 수 있게 되었습니다.

다음 사항들을 확인해 주세요:

1. modifyUserInfoService.execute() 메서드가 file 매개변수가 null일 때 적절히 처리하는지 확인하세요.
2. 클라이언트 측에서 이 API 변경에 대응할 수 있도록 문서화가 업데이트되었는지 확인하세요.

예외 처리를 개선하기 위해 다음과 같은 방식을 고려해 보세요:

@PatchMapping("/modify")
fun updateProfile(
    @RequestPart(name = "image", required = false) file: MultipartFile?,
    @Valid @RequestPart(name = "request") request: UserModifyRequest
) = modifyUserInfoService.execute(request, file)

이렇게 하면 request에 대한 유효성 검사가 자동으로 수행되며, file이 선택적임을 명시적으로 나타낼 수 있습니다.

src/main/kotlin/org/meogo/domain/user/service/ModifyUserInfoService.kt (1)

31-31: 프로필 이미지 URL이 null인 경우 처리 확인 필요

파일이 제공되지 않을 경우 url이 null이 됩니다. updateProfile 메서드가 url이 null인 경우를 올바르게 처리하는지 확인해야 합니다.

다음 스크립트를 실행하여 updateProfile 메서드가 url이 null인 경우를 처리하는지 확인할 수 있습니다:

✅ Verification successful

프로필 이미지 URL이 null인 경우 정상적으로 처리됩니다.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: updateProfile 메서드가 url이 null인 경우를 처리하는지 확인합니다.

# updateProfile 메서드 정의를 검색하여 url 매개변수가 nullable인지 확인합니다.
rg --type kotlin 'fun updateProfile' -A 5

Length of output: 1224

[coderabbitai]

Actionable comments posted: 0

🧹 Outside diff range and nitpick comments (2)

src/main/kotlin/org/meogo/global/auth/AuthDetailsService.kt (1)

14-15: LGTM: loadUserByUsername 메서드의 개선된 오류 처리

loadUserByUsername 메서드의 변경은 사용자를 찾지 못했을 때 명시적으로 UserNotFoundException을 발생시켜 오류 처리를 개선합니다. 이는 코드의 안정성을 높이고 디버깅을 용이하게 합니다.

더 나은 가독성을 위해 다음과 같이 메서드를 분리하는 것을 고려해 보세요:

override fun loadUserByUsername(username: String): UserDetails {
    val user = getUserOrThrow(username)
    return AuthDetails(user.accountId, user.role)
}

private fun getUserOrThrow(username: String): User =
    userFacade.getUserByAccountId(username) ?: throw UserNotFoundException

이렇게 하면 메서드의 주요 로직과 예외 처리를 분리하여 코드의 가독성과 유지보수성을 더욱 향상시킬 수 있습니다.

src/main/kotlin/org/meogo/domain/review/service/QueryAllBySchoolIdService.kt (1)

27-27: 사용자 조회 로직 개선 승인 및 제안

사용자를 찾지 못했을 때 UserNotFoundException을 throw하는 것은 좋은 개선입니다. 이는 null 포인터 예외를 방지하고 더 명확한 오류 처리를 제공합니다.

가독성을 더욱 높이기 위해 다음과 같이 Elvis 연산자를 사용하는 것을 고려해 보세요:

val user = userFacade.getUserById(review.userId) ?: throw UserNotFoundException()

이렇게 하면 코드의 의도가 더 명확해지고 간결해집니다.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between 04ad08f and c342b3c.

📒 Files selected for processing (3)

• src/main/kotlin/org/meogo/domain/review/service/QueryAllBySchoolIdService.kt (2 hunks)
• src/main/kotlin/org/meogo/domain/user/facade/UserFacade.kt (1 hunks)
• src/main/kotlin/org/meogo/global/auth/AuthDetailsService.kt (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• src/main/kotlin/org/meogo/domain/user/facade/UserFacade.kt

🧰 Additional context used

🔇 Additional comments (3)

src/main/kotlin/org/meogo/global/auth/AuthDetailsService.kt (1)

3-3: LGTM: UserNotFoundException 가져오기 추가

UserNotFoundException을 가져오는 것은 loadUserByUsername 메서드의 새로운 예외 처리에 필요합니다. 이는 코드의 가독성과 유지보수성을 향상시킵니다.

src/main/kotlin/org/meogo/domain/review/service/QueryAllBySchoolIdService.kt (2)

6-6: import 문 추가 승인

UserNotFoundException을 import하는 것은 적절합니다. 이는 queryAllBySchoolId 함수에서 사용자를 찾지 못했을 때 이 예외를 throw하는 새로운 로직과 일치합니다.

---

Line range hint 1-43: 전체 검토 요약

이 PR의 변경 사항들은 전반적으로 코드의 품질을 향상시킵니다. UserNotFoundException의 도입과 사용은 오류 처리를 개선하고 잠재적인 null 포인터 예외를 방지합니다. 이러한 변경으로 QueryAllBySchoolIdService의 안정성과 신뢰성이 향상되었습니다.

제안된 작은 개선 사항을 고려해 보시고, 그 외에는 이 PR을 승인해도 좋을 것 같습니다.

[coderabbitai]

Actionable comments posted: 4

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between c342b3c and 6e53d22.

📒 Files selected for processing (2)

• src/main/kotlin/org/meogo/domain/user/domain/UserRepository.kt (1 hunks)
• src/main/kotlin/org/meogo/domain/user/service/ModifyUserInfoService.kt (2 hunks)

🧰 Additional context used

🔇 Additional comments (3)

src/main/kotlin/org/meogo/domain/user/domain/UserRepository.kt (1)

9-10: findById 메서드 추가 승인

findById 메서드의 추가는 적절해 보입니다:

• UUID로 사용자를 찾는 기능을 명시적으로 제공합니다.
• 반환 타입이 User?로 되어 있어, 사용자가 존재하지 않을 경우를 적절히 처리할 수 있습니다.
• 이는 이전 리뷰 코멘트에서 제안된 null 처리 방식과 일치합니다.

src/main/kotlin/org/meogo/domain/user/service/ModifyUserInfoService.kt (2)

15-20: 클래스 이름 변경 및 생성자 매개변수 수정이 적절합니다.

클래스 이름을 UploadProfileService에서 ModifyUserInfoService로 변경한 것은 기능의 확장을 잘 반영하고 있습니다. 또한, 생성자 매개변수의 재정렬과 userFacade 추가는 논리적으로 보입니다.

---

24-32: execute 메서드의 전반적인 변경사항이 적절합니다.

메서드 시그니처 변경과 새로운 로직 구현이 확장된 기능을 잘 반영하고 있습니다. 사용자 정보 업데이트와 프로필 이미지 관리가 효과적으로 통합되었습니다.

[coderabbitai]

⚠️ Potential issue

findByAccountId 메서드의 반환 타입 변경에 대한 우려

findByAccountId 메서드의 반환 타입을 User?에서 User로 변경한 것은 잠재적인 문제를 야기할 수 있습니다:

• 이 변경은 메서드가 항상 User 객체를 반환한다고 가정합니다.
• 존재하지 않는 accountId로 조회할 경우, 예기치 않은 NullPointerException이 발생할 수 있습니다.
• Kotlin의 null 안전성 원칙에 위배됩니다.

다음과 같이 수정을 제안합니다:

-fun findByAccountId(accountId: String): User
+fun findByAccountId(accountId: String): User?

이렇게 하면 accountId에 해당하는 사용자가 없을 경우를 안전하게 처리할 수 있습니다.

📝 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 findByAccountId(accountId: String): User
	fun findByAccountId(accountId: String): User?

[coderabbitai]

🛠️ Refactor suggestion

프로필 이미지 삭제 조건 개선 제안

현재 구현은 이전 리뷰 의견을 부분적으로 반영했지만, 더 개선할 수 있습니다. 다음과 같이 수정하는 것이 좋겠습니다:

if (file != null && user.profile != defaultImage) {
    fileUtil.delete(user.profile, Path.USER)
}

이렇게 하면 코드의 가독성이 향상되고, 조건의 의도가 더 명확해집니다.

[coderabbitai]

🛠️ Refactor suggestion

UserNotFoundException 처리 개선 권장

현재 UserNotFoundException을 던지는 방식이 개선되었지만, 더 나은 에러 처리를 위해 다음과 같이 수정하는 것이 좋겠습니다:

val user = userFacade.currentUser() ?: throw UserNotFoundException()

이렇게 하면 예외에 추가 정보를 포함시킬 수 있고, 예외 처리의 일관성을 유지할 수 있습니다.