fix: 보유금액이 선결제 금액보다 작을 경우 상태값 변경(200->400) (#167)

* feat: 장바구니 추가 기능 구현 * feat: 장바구니 조회 기능 구현 * feat: 상품 주문과 식권 사용 기능 구현 * style: 포인트트랜잭션 레포지토리명 변경 * deploy: 재배포 * deploy: 재배포2 * deploy: 로그인 로직 수정에 따른 재배포 * deploy: 로그인 로직 수정에 따른 재배포2 * feat: 식권 사용 기능 구현 * feat: yml 수정 사항 반영 * feat: 식권 사용 로직 수정 * feat: 주문 로직 수정 완료 * fix: 장바구니에 아무것도 없을 때 주문 시 active 되는 현상 수정 * feat: 주문 완료 시 리턴 값 수정 * feat: 내 그룹 조회 기능 구현 * feat: 기본 프로필 이미지 url 추가 * feat: 그룹 멤버 전체 조회 기능 구현 * feat: 1인당 사용 가능 금액 설정 컬럼 추가 * feat: 내 지갑 조회 기능 구현 * feat: 선결제 기능 구현 * feat: 유저 홈 화면 조회 기능 구현 * feat: 그룹 상세 조회 기능 구현 * feat: 가게 상세 조회 쿼리 수정 * deploy: 변경사항 반영을 위한 커밋 * deploy: 수정 사항 반영 재배포 * deploy: rds 교체로 인한 재배포 * deploy: rds 교체로 인한 재배포2 * feat: 홈 화면 소속팀 조회 쿼리 제대로 안되던 현상 해결, 대표이미지 반환 추가 * feat: 그룹 상세 조회 응답 컬럼명 변경 * feat: memberLimit 컬럼 삭제 * feat: 선결제 시 기존 결제 내역과 멱등성 문제 해결 * fix: 팀 상세 조회 시 이미지 쿼리가 의미 없이 join 되던 쿼리 수정 * feat: 가게 검색 기능 대폭 수정 * feat: 프론트 요구사 반영 * feat: searchcondition 삭제 * feat: 팀타입 desription 리턴 하도록 수정 * feat: 카테고리별 팀 조회 시 팀 생성일 컬럼 추가 * feat: 매장찾기_상세 페이지 조회 기능 구현 * feat: 비밀 코드 조회 기능 엔드 포인트 수정 * fix: conflict fix * hotfix: 서버 복구 * hotfix: 서버 복구2 * feat: Barobill api 연동 * fix: 홈 화면에 같은 데이터가 중복되는 현상 수정 * infra: ElastiCache 연결을 위한 docker-compose 설정 * infra: cicd script 수정 * infra: cicd script ec2 주소 설정 * infra: cicd script 수정2 * infra: cicd script 수정3 * infra: cicd script 수정4 * infra: cicd script 수정5 * infra: cicd script 수정6 * infra: cicd script 수정7 * infra: cicd script 수정7 * infra: cicd script 수정8 * infra: cicd script 수정9 * infra: cicd script 수정10 * infra: cicd script 수정11 * infra: cicd script 수정12 * infra: cicd script 수정13 * infra: cicd script 수정14 * infra: cicd script 수정15 * infra: cicd script 수정16 * infra: cicd script 수정17 * infra: cicd script 수정18 * infra: cicd script 수정19 * infra: cicd script 수정20 * infra: cicd script 수정21 * infra: cicd script 수정22 * fix: 보유금액이 선결제 금액보다 작을 경우 상태값 변경(200->400)
9oormthon-univ · Nov 27, 2024 · b6c9a32 · b6c9a32
1 parent 6de00c6
commit b6c9a32
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 10 deletions.
diff --git a/src/main/java/com/jangburich/domain/store/service/PrepayService.java b/src/main/java/com/jangburich/domain/store/service/PrepayService.java
@@ -42,17 +42,9 @@ public Message prepay(String userId, PrepayRequest prepayRequest) {
         Store store = storeRepository.findById(prepayRequest.storeId())
                 .orElseThrow(() -> new IllegalArgumentException("유효하지 않은 가게 id 입니다."));
 
-        if (!team.getTeamLeader().getUser_id().equals(user.getUserId())) {
-            return Message.builder()
-                    .message("팀의 리더가 아닌 사람은 선결제를 할 수 없습니다.")
-                    .build();
-        }
+        team.validateIsTeamLeader(team.getTeamLeader().getUser_id(), user.getUserId());
 
-        if (prepayRequest.prepayAmount() > user.getPoint()) {
-            return Message.builder()
-                    .message("보유하고 있는 금액이 선결제 하려는 금액보다 적습니다.")
-                    .build();
-        }
+        user.validateHasPointWithPrepayAmount(prepayRequest.prepayAmount(), user.getPoint());
 
         user.usePoint(prepayRequest.prepayAmount());
         PointTransaction pointTransaction = PointTransaction

diff --git a/src/main/java/com/jangburich/domain/team/domain/Team.java b/src/main/java/com/jangburich/domain/team/domain/Team.java
@@ -66,4 +66,10 @@ public Team(String name, String description, String secretCode, TeamLeader teamL
         this.point = point;
         this.teamType = teamType;
     }
+
+    public void validateIsTeamLeader(Long userId, Long userId1) {
+        if (!userId.equals(userId1)) {
+            throw new IllegalArgumentException("팀의 리더가 아닌 사람은 선결제를 할 수 없습니다.");
+        }
+    }
 }
diff --git a/src/main/java/com/jangburich/domain/user/domain/User.java b/src/main/java/com/jangburich/domain/user/domain/User.java
@@ -80,4 +80,10 @@ public static User create(String userId, String nickname, String email, String i
 		newUser.setPoint(0);
 		return newUser;
 	}
+
+	public void validateHasPointWithPrepayAmount(int prepayAmount, Integer point) {
+		if (prepayAmount > point) {
+			throw new IllegalArgumentException("보유하고 있는 금액이 선결제 하려는 금액보다 적습니다.");
+		}
+	}
 }