Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Refactor] user 페이지에 남은 금액 남은 횟수 추가 #231

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

sss4920
Copy link
Contributor

@sss4920 sss4920 commented Feb 21, 2024

🚩 관련 이슈

📋 구현 기능 명세

  • user 페이지 남은 금액, 남은 횟수 추가
  • response에 값 추가

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지
    보니까 user.getSavedAmount와 count가 null인 경우를 테스트하면 500이 떠서 이유를 보니까 npe가 떠서 null처리를 안에 변수 하나를 더 써서 처리해주긴 했습니다..! 전체적인 테스트 코드 설계를 해보고 테스트 케이스 짜보는 것도 좋은 경험이 될 것 같아요!

그리고 안에 response보니까 userResponse 부분만 내부 클래스를 사용해서 구현한 것 같은데 모두 이런식으로 바꿀 것인지? 아니면 userDto를 하나 더 파서 거기다가 넣어주고 쓰는 방식으로 한다든지 통일을 하면 좋을 것 같기도 합니다..!(이건 물론 장단점 비교 후에 해보는 것이 좋지 않을까! 최근에 17버전에서 record 쓰고나니 데이터 불변성 등을 보장하는 것에 대해서 생각해보니 저런 방법도 좋은 시도라고 생각합니다.☺️

  • 어떤 부분에 리뷰어가 집중해야 하는지
    제가 밑에 있던 코드들을 조금 수정했는데 예외 케이스는 있을지, 테스트 케이스로는 무엇들이 있을지 고민해보면 좋을 것 같습니다.

  • 개발하면서 어떤 점이 궁금했는지
    흠 간단한 테스트 코드 설정.. 다 하면 브랜치파서 적용해보도록 하겠습니다!

📸 결과물 스크린샷

{
    "code": 200,
    "message": "유저 조회 성공",
    "data": {
        "userData": {
            "userId": 151,
            "nickname": "다니",
            "userLevel": "기사",
            "fcmIsAllowed": true,
            "accumulatedAmount": 100,
            "amountSavedHundredDays": 0,
            "amountSavedTwoWeeks": 0,
            "amountSpentTwoWeeks": 0,
            "remainingAmount": 149900,
            "remainingCount": 1
        }
    }
}

🛠️ 테스트

  • 테스트

🚀 API Endpoint

  • /user

@sss4920 sss4920 added soohyun develop soohyun things refactor labels Feb 21, 2024
@sss4920 sss4920 self-assigned this Feb 21, 2024
Copy link
Contributor

@funnysunny08 funnysunny08 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨어요!!👍👍

Copy link
Contributor

@alpha-kwhn alpha-kwhn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!!

@sss4920 sss4920 merged commit 45ec08e into dev Feb 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor soohyun develop soohyun things
Projects
None yet
3 participants