포인트 도메인 구현 #44

13wjdgk · 2023-10-01T08:51:31Z

Changes 📝

포인트 도메인 관련 기능 로직을 구현하였습니다.

1일 1 접속 시 100 포인트 적립
회원가입 시 500 포인트 적립
헬스커넥트 연동 시 500 포인트 적립
포인트 상품 구매
마이페이지 조회
포인트 상품 목록 조회

Details 🌼

ERD : 총 4개의 테이블 추가

UserPoint : 사용자의 최종 사용가능 포인트
PointLog : 포인트 내역 ( 사용, 적립)
PointProduct : 포인트 상품
ProductPurchase : 포인트 상품 구매 내역

1일 1 접속 포인트 적립

JWT Filter에서 사용자 인증을 거친 후 , PointService.addAccessPoint()를 통해 적립 가능 유저 여부 판별
적립 가능한 유저인지 확인이 완료되면 , PointService.addPointEvent() 를 통해 적립
PointLog,UserPoint 테이블에 기록
PointService 클래스 단에 Transaction 적용

회원가입 시 500 포인트 적립

UserSignupService.signUp() 를 통해 회원가입 -> 완료 후 PointService.addSignupPoint()를 통해 적립 가능 유저 여부 판별
적립 가능한 유저인지 확인이 완료되면 , PointService.addPointEvent() 를 통해 적립
PointLog,UserPoint 테이블에 기록
UserSignupService.signUp, PointService 클래스 단에 Transaction 적용

헬스커넥트 연동 시 500 포인트 적립

HealthConnectRegisterService.interlockHealthConnect() 를 통해 헬스커넥트 연동 여부 반영
첫 연동한 유저는 PointService.addHealthConnectPoint() 를 통해 적립
PointLog,UserPoint 테이블에 기록
HealthConnectRegisterService 클래스 , PointService 클래스 단에 Transaction 적용

낙관적 락 이용

상품 구매 요청이 동시에 100번 들어오는 동시성 테스트를 진행
상황 : 유저가 6000 포인트를 가진 상황에서, 5000 상품 구매를 100번 동시에 요청한 상황
결과 : 100번 중 9번 상품 구매 성공
원인 : 첫 구매의 트랜잭션이 커밋 완료되기 전에 두번째 구매가 진행되었기 때문 (두번갱신 문제)
해결 : UserPoint에 @Version 필드를 추가. 첫번째 트랜잭션 커밋 -> version 업데이트 -> 두번째 트랜잭션이 커밋 -> version이 달라 오류를 반환
이 외에도, 동시 접속, 동시 헬스커넥트 연동, 동시 회원가입 동시성 테스트 진행 완료

Check List ☑️

테스트 코드를 통과했다.
merge할 브랜치의 위치를 확인했다. (main ❌)
Assignee를 지정했다.
Label을 지정했다.

[Optional] Etc

*이번 PR 후 해야할 일

TaskExecutor 설정
접속 적립 AOP로 구현 여부 확인
refreshToken 기간 늘리기

*Reference
https://tecoble.techcourse.co.kr/post/2023-08-16-concurrency-managing/?fbclid=IwAR3ily9yhZ0wurw1OVGTE9Xvv_fbmRCDl3LTPBf4A73P6U9JDiv7S6GxnUc_aem_Ab7UjAoeWUcSVfSVHL7zzpWs1R5qUoyWmbqGzKB_t04QYHuSjH6kIwsE6hETr40yD3A

자바 ORM 표준 JPA 프로그래밍 (낙관적 락 - 책)

스프링 DB 2편 - 데이터 접근 활용 기술(트랜잭션 - 인프런 강의)

GIVEN53 · 2023-10-04T05:40:41Z

src/main/java/com/coniverse/dangjang/DangjangApplication.java


 @EnableJpaAuditing
+@EnableAsync


P1) TaskExecutor를 설정하지 않았는데, default로 사용했을 때 발생하는 문제에 대해 찾아보면 좋을 것 같아요!

지금 당장 해결해야할 문제는 아니라고 생각되어서, 클라이언트 api 연동 후 해결하려고 합니다 ! 이번 pr 후 해야할 일들을 위에 정리해뒀습니다 ~

크리티컬한 문제라고 생각해요 다음 pr에 빠르게 반영됐으면 좋겠습니다~

GIVEN53 · 2023-10-04T05:48:48Z

src/main/java/com/coniverse/dangjang/domain/auth/filter/JwtValidationFilter.java

@@ -39,6 +41,8 @@
 @Component
 @RequiredArgsConstructor
 public class JwtValidationFilter extends OncePerRequestFilter {
+	private final PointService pointService;


P2) Jwt 검증 필터에 서비스 레이어를 주입해서 사용하는 것이 좋은 방법일지 고민이 되네요
결합도가 높아지고 여러 개의 책임을 지는 것은 아닐까? 하는 생각입니다
presentation layer에 AOP를 적용하는 방법은 어떨까요?

좋은 방법인 것 같아요 ~ 이번 pr 후 고려해서 수정해보겠습니다 ~

GIVEN53 · 2023-10-04T05:49:22Z

src/main/java/com/coniverse/dangjang/domain/auth/filter/JwtValidationFilter.java

@@ -39,6 +41,8 @@
 @Component
 @RequiredArgsConstructor
 public class JwtValidationFilter extends OncePerRequestFilter {
+	private final PointService pointService;
+	private final UserRepository userRepository;


P3) 사용하지 않는 DI인 것 같습니다

GIVEN53 · 2023-10-04T05:51:36Z

...ava/com/coniverse/dangjang/domain/healthmetric/dto/request/HealthConnectRegisterRequest.java

@@ -0,0 +1,4 @@
+package com.coniverse.dangjang.domain.healthmetric.dto.request;
+
+public record HealthConnectRegisterRequest(boolean healthConnectInterlock) {


P3) JavaDoc 필요!

GIVEN53 · 2023-10-04T05:52:51Z

...in/java/com/coniverse/dangjang/domain/healthmetric/service/HealthConnectRegisterService.java

+				pointService.addHealthConnectPoint(user);
+			}
+		} else if (!request.healthConnectInterlock() && user.getHealthConnect().equals(HealthConnect.CONNECTING)) {
+			user.setHealthConnect(HealthConnect.DISCONNECTED);


위와 같은 맥락

한번도 연동한 적 없는 유저와, 연동이 이미 해제되어 있는 유저는 아무런 처리를 하지 않습니다. 연동되어 있는 유저일때만 연동을 해지합니다

GIVEN53 · 2023-10-04T06:32:16Z

src/main/java/com/coniverse/dangjang/domain/user/controller/UserController.java

+	 * @since 1.0.0
+	 */
+	@GetMapping("/mypage")
+	public ResponseEntity<SuccessSingleResponse<?>> getMyPage(@AuthenticationPrincipal User user) {


P4) response data 필드가 null이 아니면 제네릭에 와일드카드 대신 클래스명을 명시해주셨으면 좋겠습니당

GIVEN53 · 2023-10-04T06:33:23Z

src/main/java/com/coniverse/dangjang/domain/user/entity/User.java

@@ -59,10 +65,19 @@ public class User extends BaseEntity implements Persistable<String> {
 	private boolean medicine;
 	private boolean injection;
 	private String profileImagePath;
+	@Enumerated(EnumType.STRING)
+	private HealthConnect healthConnect = HealthConnect.NEVER_CONNECTED;
+	@Column(name = "ACCESSED_AT", nullable = false)


유저는 BaseEntity를 상속받아서 updatedAt 필드가 존재하는데 추가로 중복되는 필드를 작성하신 이유가 있을까요?

updatedAt은 객체가 수정되었을 때만 기록되지만, accessedAt은 접속하였을 때만 업데이트합니다.

GIVEN53 · 2023-10-04T06:36:45Z

src/main/java/com/coniverse/dangjang/domain/user/repository/UserRepository.java

+	 *
+	 * @since 1.0.0
+	 */
+	@Modifying(flushAutomatically = true, clearAutomatically = true)


포인트에서 서술한 내용과 동일

GIVEN53 · 2023-10-04T06:37:26Z

src/main/java/com/coniverse/dangjang/domain/user/service/MypageService.java

+ * @since 1.0.0
+ */
+@Service
+@AllArgsConstructor


P3) @requiredargsconstructor로 수정 부탁드려여

GIVEN53 · 2023-10-04T06:38:23Z

src/main/java/com/coniverse/dangjang/domain/user/service/UserSearchService.java

+	 * @return User 사용자
+	 * @since 1.0.0
+	 */
+	public Optional<User> findNickname(String nickname) {


P3) JavaDoc은 "존재하는" 닉네임인지 조회하여 "확인" 인데 조회만 하고 있어요

GIVEN53 · 2023-10-06T01:46:14Z

src/main/java/com/coniverse/dangjang/domain/point/service/PointService.java

+		if (balancePoint >= 0) {
+			return balancePoint;
+		} else {
+			throw new BusinessException(400, "포인트가 부족합니다.");


P3) Exception 클래스를 구현해 주세요

GIVEN53 · 2023-10-06T01:46:59Z

src/main/java/com/coniverse/dangjang/domain/point/service/PointService.java

+	}
+
+	/**
+	 * 헬스커넥트 포인트 적립


P3) JavaDoc 수정 필요!

GIVEN53 · 2023-10-06T01:56:15Z

src/main/java/com/coniverse/dangjang/domain/auth/filter/JwtValidationFilter.java

 			Authentication auth = getAuthentication(token);
 			SecurityContextHolder.getContext().setAuthentication(auth);
+		} else if (jwtStatus.equals(JWTStatus.EXPIRED)) {
+			request.setAttribute("exception", jwtStatus.getMessage());
+		} else if (jwtStatus.equals(JWTStatus.INVALID)) {


P3) elif 하나로 합쳐도 될 것같아요

조건문 수정했습니다 ~

GIVEN53

전체적으로 수정이 이루어지면서 JavaDoc과 불일치하는 메서드가 많아요 아마 IDE가 잡아줄텐데 같이 수정 부탁드려요

GIVEN53 · 2023-10-06T02:03:15Z

src/main/java/com/coniverse/dangjang/domain/point/controller/PointController.java

+	 * @since 1.0.0
+	 */
+	@PostMapping
+	public ResponseEntity<SuccessSingleResponse> usePoint(@Valid @RequestBody UsePointRequest request,


SuccessSingleResponse 제네릭 명시안하면 IDE가 잡아주지 않나요?

GIVEN53 · 2023-10-06T02:04:59Z

src/main/java/com/coniverse/dangjang/domain/point/entity/PurchaseHistory.java

+	private PointId pointId;
+	private String phone;
+	@ColumnDefault("false")
+	private boolean completed;


구매가 완료되었다는 의미의 필드인가요? 해당 필드 사용 이유가 궁금해여

GIVEN53 · 2023-10-06T02:08:40Z

src/main/java/com/coniverse/dangjang/domain/point/service/PointSearchService.java

+	 */
+	public PointProduct findPointProductById(String pointProduct) {
+		return pointProductRepository.findById(pointProduct)
+			.orElseThrow(() -> new BusinessException(400, "포인트 상품이 없습니다."));


P3) Exception 클래스를 구현해 주세요

GIVEN53 · 2023-10-06T02:08:45Z

src/main/java/com/coniverse/dangjang/domain/point/service/PointSearchService.java

+	 */
+	public UserPoint findUserPointByOauthId(String oauthId) {
+		return userPointRepository.findById(oauthId)
+			.orElseThrow(() -> new BusinessException(400, "유저 포인트가 없습니다."));


P3) Exception 클래스를 구현해 주세요

새로운 예외를 구현하기 보다, 표준 예외를 재사용하는 방법으로 수정하겠습니다 ~

GIVEN53 · 2023-10-06T02:17:33Z

src/main/java/com/coniverse/dangjang/global/support/enums/JWTStatus.java

+
+import lombok.RequiredArgsConstructor;
+
+@RequiredArgsConstructor


위에서 서술한 내용과 동일

GIVEN53 · 2023-10-06T02:20:22Z

src/test/java/com/coniverse/dangjang/domain/user/service/MypageServiceTest.java

+	private User 유저;
+
+	@Test
+	void getMypage() {


P3) 테스트 설명으로 수정해주세요

coniverse-github-app · 2023-10-08T21:16:48Z

Analysis Details

0 Issues

0 Bugs
0 Vulnerabilities
0 Code Smells

Coverage and Duplications

No coverage information (97.30% Estimated after merge)
No duplication information (0.90% Estimated after merge)

Project ID: co-niverse_dangjang-backend_AYj2jZJELehUZAlqDvRk

View in SonarQube

13wjdgk added 17 commits September 20, 2023 18:26

feat : User 스키마 point col 추가

34517db

feat : 1일 1접속 포인트 적립 구현

87e882c

feat : 회원가입 포인트 구현

392d038

feat : healthConnect 연동 포인트 구현

599a1fa

feat : 포인트 상품 내역 조회

3c1030f

refactor : 존재하지 않는 유저 Exception 404 -> 401로 변경

604e946

feat : 마이페이지 조회

3a29e6a

feat : 헬스커넥트 연동 중복 방지

0c5cd82

test : 포인트,마이페이지 테스트

8493bf5

refactor : 스키마 수정에 따른 sql 업데이트

d1df78e

refactor : 유저,포인트 불필요한 코드 수정

2fc9d48

refactor : user_point 생성

fe9fcc1

refactor : point_product 상품명 변경

0be296b

refactor : user_point @Version 낙관적 락 추가

c68c4ec

docs : 주석 추가

f7c4381

feat : 포인트 상품 구매 내역 테이블 생성

fbce7af

merge : dev 브랜치와 merge

ff6e134

13wjdgk added the feature 기능 label Oct 1, 2023

13wjdgk self-assigned this Oct 1, 2023

This comment has been minimized.

Sign in to view

13wjdgk added 2 commits October 3, 2023 15:39

refactor : code smell public 제거

bedce01

refactor : 비동기 메서드 테스트 수정

f1dddf4

This comment has been minimized.

Sign in to view

feat : AuthToken 재발급 기능 구현

be62036

GIVEN53 reviewed Oct 4, 2023

View reviewed changes

13wjdgk added 5 commits October 5, 2023 19:50

refactor : JWT ExpiredJwtException 예외 메세지 수정

2451509

refactor : 포인트 상품 리스트 조회 Response 변경

7df8bb8

refactor : PointHistory 테이블,PointProduct pointName 필드 네이밍 변경

91840c1

docs : 주석 수정

216199c

refactor : interlockHealthConnect 메서드 DisConnected 유저 기기연동 요청 로직 수정

561b2c9

13wjdgk force-pushed the feat/point branch from e02af48 to 4d8b826 Compare October 5, 2023 17:52

This comment has been minimized.

Sign in to view

refactor : code smell 수정, 안쓰는 변수 제거

0b02082

13wjdgk force-pushed the feat/point branch from 7cfb8b9 to 0b02082 Compare October 5, 2023 18:06

Merge branch 'dev' into feat/point

1908f65

This comment has been minimized.

Sign in to view

GIVEN53 reviewed Oct 6, 2023

View reviewed changes

13wjdgk added 10 commits October 7, 2023 02:00

docs : purchaseProduct 메서드 javaDoc 수정

b41b3fe

refactor : jwt validation 확인 로직 조건문 수정

9228314

refactor : 포인트 관련 예외처리 수정

c86ccf6

docs : PointService의 javaDocs 수정

322b5c2

test : javaDoc 추가 및 예외 수정

9c67ce7

docs : javaDoc 수정

123ca26

refactor : interlockHealthConnect patch 수정

1eaf606

test : IllegalArgumentException 수정

381c718

refactor : 회원가입시 weight HealthMetric 등록

fa3adbb

refactor : health-connect 엔드포인트 patch 인증 추가

323d0a0

This comment has been minimized.

Sign in to view

refactor : getMypage 테스트명 변경

30f8690

This comment has been minimized.

Sign in to view

GIVEN53 approved these changes Oct 7, 2023

View reviewed changes

refactor : EarnPoint RequiredArgsConstructor로 변경

e7a8051

This comment has been minimized.

Sign in to view

13wjdgk merged commit 60cefbe into dev Oct 7, 2023
1 check passed

GIVEN53 deleted the feat/point branch October 8, 2023 13:07

		@@ -0,0 +1,4 @@
		package com.coniverse.dangjang.domain.healthmetric.dto.request;

		public record HealthConnectRegisterRequest(boolean healthConnectInterlock) {


		import lombok.RequiredArgsConstructor;

		@RequiredArgsConstructor


		@EnableJpaAuditing
		@EnableAsync

포인트 도메인 구현 #44

포인트 도메인 구현 #44

Conversation

13wjdgk commented Oct 1, 2023 • edited Loading

Changes 📝

Details 🌼

Check List ☑️

[Optional] Etc

This comment has been minimized.

This comment has been minimized.

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

This comment has been minimized.

This comment has been minimized.

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

GIVEN53 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

coniverse-github-app bot commented Oct 8, 2023

Analysis Details

0 Issues

Coverage and Duplications

13wjdgk commented Oct 1, 2023 •

edited

Loading