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

경북대 BE_김영인 6주차 과제(3단계) #324

Open
wants to merge 44 commits into
base: youngini
Choose a base branch
from

Conversation

Youngini
Copy link

@Youngini Youngini commented Aug 3, 2024

Point 기능을 추가하는 3단계 과제 입니다.

Youngini added 30 commits July 29, 2024 16:37
Copy link

@gongdongho12 gongdongho12 left a comment

Choose a reason for hiding this comment

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

3단계 작업하시느라 고생하셨습니다!

  • 1, 2단계의 리뷰내용 반영
  • Rebase로 Conflict 해소 부탁드립니다
  • 실제 배포된 엔드포인트 DM으로 공유 부탁드려요!

현재 작업해주신 내용은 포인트를 사용하는 관점 없이 충전만 하는 것 같네요 🤔
시간이 부족하긴 했지만 한번 주문에 포인트로 현금을 대체하거나 1퍼센트씩 적립해주면 더 좋을 것 같습니다!

Comment on lines +50 to +53
- [x] member entity에 point 추가
- [x] 특정 회원의 point 조회
- [x] 특정 회원에 point 추가

Choose a reason for hiding this comment

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

포인트를 만들었는데 사용할 방안은 FE 분들과 싱크 완료되었을까요?

Comment on lines +19 to +20
@Column(columnDefinition = "integer default 0")
private Integer point;

Choose a reason for hiding this comment

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

포인트는 멤버에 넣으셧군요

Comment on lines +31 to +33
public void addPoint(int point){
this.point += point;
}

Choose a reason for hiding this comment

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

포인트를 더하는 기능만 만들었군요?

Comment on lines +65 to +70
public PointResponse addPoint(Long user_id, PointRequest pointRequest) {
Member member = getMemberById(user_id);
member.addPoint(pointRequest.getPoint());
memberRepository.save(member);
return new PointResponse(member);
}

Choose a reason for hiding this comment

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

한 계정으로 웹 사이트에 접속을 두명이 했을 때, 포인트 충전을 각각 10씩 하면 어떤 결과가 일어날까요?
가져오는 시점과 업데이트 시점이 달라질 수 있지 않을까요?

이부분 관련해서 스레드 나눠서 동시성 테스트랑 이 로직에 트랜잭션 같이 적용 부탁드려요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants