🚀 3단계 - 수강신청(DB 적용) #531

kimsunghyun1995 · 2024-05-03T07:14:23Z

안녕하세요

3단계 DB적용 PR 입니다.

기존에는 추상클래스 Session을 통해 유료강의와 무료강의 클래스를 나눴었는데,
추상클래스를 DB와 매핑시키려니 어려움이 있어, 기존 클래스 Session으로 매핑하였습니다.
다른 방법이 있을까요??

감사합니다

* 테스트도 이동

javajigi

DB 매핑 잘 했네요. 👍
단, 매핑할 때 잘못 이해한 부분이 있는 것 같아 피드백 남겼어요.

pr에 남긴 질문에 대한 해결 방법은 DB 매핑을 하는 객체와 비지니스 로직을 구현을 담당하는 도메인 객체를 분리하면 가능할 것 같아요.
이렇게 구현할 경우 비지니스 로직 구현을 담당하는 도메인 객체 설계가 DB 매핑에 영향을 받지 않을 것 같아요.

javajigi · 2024-05-05T06:05:15Z

src/main/java/nextstep/session/domain/Enrollment.java

+		this.id = id;
+		this.maximumNumberOfParticipants = maximumNumberOfParticipants;
+		this.sessionPrice = sessionPrice;
+		this.sessionStatus = sessionStatus;
+		this.nsUsers = nsUsers;


부 생성자가 주 생성자를 호출하도록 구현

javajigi · 2024-05-05T06:07:20Z

src/main/java/nextstep/session/domain/Enrollment.java

+		if (sessionStatus != SessionStatus.PREPARING) {
+			throw new IllegalArgumentException("강의가 준비중이 아닙니다.");
+		}


메서드 이름은 수강 신청 가능 기간임을 물어보고 있는데 예외처리를 하는 것은 뭔가 어색함
아래에 있는 isParticipantsSession() 메서드도 같다.
메서드 이름을 바꾸거나 예외 처리를 없애던가 선택하면 어떨까?

javajigi · 2024-05-05T06:07:41Z

src/main/java/nextstep/session/domain/Enrollment.java

+				'}';
+	}
+
+}


Enrollment 객체 도출한 점 👍

javajigi · 2024-05-05T06:11:17Z

src/main/java/nextstep/users/domain/NsUser.java

@@ -18,6 +18,8 @@ public class NsUser {

    private String email;

+    private long sessionId;


이와 같이 구현할 경우 NsUser : Session의 관계는 1 : n 관계로 NsUser는 한 개의 Session만 수강 가능해 진다.
NsUser : Session의 관계는 m : n 관계여야 하지 않을까?

javajigi · 2024-05-05T06:12:46Z

src/test/java/nextstep/session/infrastructure/EnrollmentRepositoryTest.java

+import static org.assertj.core.api.Assertions.assertThat;
+
+@JdbcTest
+class EnrollmentRepositoryTest {


RepositoryTest 테스트의 경우 구현과 실행 측면에서 많은 비용이 발생하고 유지보수 비용 또한 발생한다.
현장에서도 DB에 대한 테스트를 구현하는 것이 좋을까?

아니요 DB에 대한 테스트는 굳이 할 필요가 없어보입니다!
뿐만 아니라, JPA을 사용한다면, 쿼리 메소드를 사용하기 때문에 테스트 코드를 작성할 이유가 없을 것 같습니다.

kimsunghyun1995 added 27 commits April 30, 2024 15:13

doc: step3 요구사항 정리

fd5bc40

feat: 수강신청만을 위한 클래스 Enrollment 추가

683b74c

feat: imageInfo 테이블 매핑 및 CRUD 기능 구현

bb4e0ee

feat: 쿼리 사용을 위한 Getter 추가

c79a28c

feat: image_info DDL 및 예시데이터 추가

b2e8b79

test: imageInfo Repository 테스트 코드 추가

0ca406b

refactor: 사용하지 않는 패키지 정리

b0584e8

doc: 클래스 DB 매핑 정보 수정

bec5bda

refactor: 수강신청 금액도 Session -> Enrollment로 이동

939744c

* 테스트도 이동

feat: enrollment repository 추가

8de065d

feat: nsUser 테이블 필드 추가 및 데이터 추가

611a834

feat: nsUser에 session_id 필드 추가 및 findallBySessionId 쿼리 추가

5fef765

test: findAllBySessionId 메소드 테스트 추가

bbd2583

fix: 데이터 수정

969a55e

refactor: 사용하지 않는 import 정리 및 final 추가

9e4dbf2

feat: EnrollRepository 기능 구현

de2692c

test: EnrollmentRepository 테스트

f1baa1e

feat: 쿼리 사용을 위한 getter 추가

b49e4f4

feat: session과 연관관계를 위한 sessionId 필드 추가

a7d6469

fix: mapper 사용을 위한 추상 클래스 삭제

6b339bf

refactor: sessionId 필드 추가로 인한 테스트 코드 수정

5a9ec82

doc: 진행상황 정리

885a682

feat: session 테이블 및 데이터 생성

f475bad

feat: session 데이터를 위한 enroll과 iamgeinfo 데이터 sessionId로 가져오는 기능 구현

a446509

feat: notFound 익셉션 추가

9e66e65

feat: sessionRepository 구현체 구현

fad7a82

test: sessionRepository 테스트 추가

876b073

javajigi requested changes May 5, 2024

View reviewed changes

kimsunghyun1995 added 2 commits May 16, 2024 20:50

refactor: 부 생성자가 주 생성자 호출하도록 코드 정리

d17d14c

refactor: 수강신청 인원에 대한 확인 메소드 네이밍 변경

232fd52

kimsunghyun1995 added 8 commits May 22, 2024 10:57

feature: NsUser : Session의 관계(M:N)에 대해 새로운 도메인 생성

9822a8c

feature: student_enrollment_information db 데이터 생성

04eef92

fix: 기존 M:N 관계 도메인 수정

9262189

fix: studentEnrollmentInformation 도메인 생성으로 인한 repository 수정

5e6f227

feat: User 도메인에 복수개 id로 조회하는 in 쿼리 추가

5811399

test: 복수개의 ID로 조회하는 쿼리 테스트

5c63ef1

test: 의존성 추가

627a90b

fix: enrollment에 User 리스트 생성하는 부분 추가

f0e800a

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 3단계 - 수강신청(DB 적용) #531

🚀 3단계 - 수강신청(DB 적용) #531

kimsunghyun1995 commented May 3, 2024

javajigi left a comment

javajigi May 5, 2024

javajigi May 5, 2024

javajigi May 5, 2024

javajigi May 5, 2024

javajigi May 5, 2024

kimsunghyun1995 May 16, 2024 •

edited

Loading

+              				'}';
+              	}
+              }

		@@ -18,6 +18,8 @@ public class NsUser {

		private String email;

		private long sessionId;

🚀 3단계 - 수강신청(DB 적용) #531

Are you sure you want to change the base?

🚀 3단계 - 수강신청(DB 적용) #531

Conversation

kimsunghyun1995 commented May 3, 2024

javajigi left a comment

Choose a reason for hiding this comment

javajigi May 5, 2024

Choose a reason for hiding this comment

javajigi May 5, 2024

Choose a reason for hiding this comment

javajigi May 5, 2024

Choose a reason for hiding this comment

javajigi May 5, 2024

Choose a reason for hiding this comment

javajigi May 5, 2024

Choose a reason for hiding this comment

kimsunghyun1995 May 16, 2024 • edited Loading

Choose a reason for hiding this comment

kimsunghyun1995 May 16, 2024 •

edited

Loading