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

* feat : 피드백 내용 반영 후 수정 진행

* docs : 나의 학습 목표, 요구사항 정리 후 파일 생성

* docs : 객체에 맞춰 sql 파일 작성

* refactor : session Repository 제작 후 연결 중...(또 열심히 코드 작성하다 보니 작업 단위별로 커밋하는 것을 잊어버렸습니다..)

* feat : CoverImage Repository Sql 수정 및 테스트 코드 작성

* feat : sessionService findById 메서드 생성(Enrollments 작업 전)

* feat : Enrollment Repository 생성 및 내부 로직 생성, 테스트 코드 작성, EnrollmentRecord 생성
refactor : sessionService 테스트 코드 수정,
Enrollment에서 필요한 인스턴스 변수 추가(id)
docs : sql Base Data 추가

* feat : SessionRepository에서 entrollment 추가 메서드 생성 및 테스트 코드 추가
docs : Base data 추가

* docs : Base data autoincrement 추가
refactor : repository insert 시 id 제거

* refactor : 생산성이 떨어지는 테스트 제거, 수강신청 시 Service에서 유효성 검사 진행, 강의 등록 시 Session 객체가 아닌 SessionRecord 객체 전달

* refactor : enrollment 객체에서 payment 인스턴스 변수 제거, 강의 신청 시 payment를 파라미터로 전달 받아 다시 session에게 전달 후 금액 일치 확인, 점진적인으로 리팩터링 진행

* feat : payment 테이블 생성, payment 저장 로직 생성

Author: chanani

README.md

@@ -3,6 +3,7 @@
 ## 단계별 문서
 - [🚀 1단계 - 레거시 코드 리팩터링](./docs/01-refactoring.md)
 - [🚀 2단계 - 수강신청(도메인 모델)](./docs/02-lms-domain-model.md)
+- [🚀 3단계 - 수강신청(DB 적용)](./docs/03-lms-db.md)
 
 ## 진행 방법
 * 학습 관리 시스템의 수강신청 요구사항을 파악한다.

docs/02-lms-domain-model.md

@@ -2,15 +2,15 @@
 ***
 
 ## 코드 리뷰
-> PR 링크 : 
+> PR 링크 : [#793](https://github.com/next-step/java-lms/pull/793)
 
 ## 나의 학습 목표
 
 ### 1. 객체 중심의 설계를 해보자.
 - 요구사항을 정확히 분석하고, TDD 사이클을 통해 객체의 핵심 비즈니스 로직을 우선적으로 만들어보자.
 - 그 이후 객체를 조립하며, 리팩터링 해보자.
 
-### 1. TDD 사이클을 의식하며 기능을 구현한다. 
+### 2. TDD 사이클을 의식하며 기능을 구현한다. 
 - `실패 → 성공 → 리팩터링` 과정으로 작업한다.
 - 습관처럼 이 과정을 지나치면, 다시 돌아와 사이클을 반복한다.
 - TDD 사이클 습관을 만들자.

docs/03-lms-db.md

@@ -0,0 +1,52 @@
+# 🚀 3단계 - 수강신청(DB 적용)
+***
+
+## 코드 리뷰
+> PR 링크 : [#798](https://github.com/next-step/java-lms/pull/798)
+
+## 나의 학습 목표
+
+### 1. 객체 주도 설계를 진행해보자
+- 평소 DB 설계 시 테이블 주도 설계로 진행해 왔지만 이번 기회에 교육 취지에 맞게 `객체 주도 설계`를 진행해보자.
+
+### 2. 도메인 객체 수정을 최소화하자
+- 이전 단계에서 도메인 객체 설계를 진행했고, 이번 단계에서는 도메인 객체 수정을 최소화하며 DB 테이블과 매핑해보자. 
+
+
+## 핵심 학습 목표
+
+- 2단계에서 구현한 객체 구조(도메인 구조)를 가능한 유지하면서 DB 테이블과 매핑한다.
+- 성능보다 도메인 객체에 로직 구현하는 것을 목표로 연습한다
+  - 객체 구조를 유지하기 위해 여러 번의 DB 쿼리를 실행해도 괜찮다.
+
+## 프로그래밍 요구사항
+
+- 앞 단계에서 구현한 도메인 모델을 DB 테이블과 매핑하고, 데이터를 저장한다.
+  - CRUD 쿼리와 코드를 구현하는데 집중하기 보다 테이블을 설계하고 객체 매핑하는 부분에 집중한다.
+- Payment는 테이블 매핑을 고려하지 않아도 된다.
+
+## 참고할 코드
+
+- DB 테이블 추가 : src/main/resources/schema.sql
+
+```sql
+create table course (
+    id bigint generated by default as identity,
+    title varchar(255) not null,
+    creator_id bigint not null,
+    created_at timestamp not null,
+    updated_at timestamp,
+    primary key (id)
+);
+```
+
+- 테이블에 샘플 데이터를 추가하고 싶다면 src/main/resources/data.sql 파일에 추가 가능함
+
+### CRUD 코드
+
+- src/main/java 폴더의 nextstep.courses.infrastructure.JdbcCourseRepository
+- JdbcCourseRepository 샘플 코드는 Spring JDBC 라이브러리를 활용해 구현함
+
+### CRUD 코드에 대한 테스트 코드
+
+- src/test/java 폴더의 nextstep.courses.infrastructure.CourseRepositoryTest

docs/check-list.md

@@ -5,7 +5,7 @@
 - TDD 사이클로 구현하였는가 ?
 - TDD 사이클 단위로 커밋하였는가 ?
 - 객체에게 상태를 노출시키지 않고 책임을 맡겼는가 ?
-- 테슽으는 객체의 책임과 결과만 검증했는가 ?
+- 테스트는 객체의 책임과 결과만 검증했는가 ?
 - 불변성을 최대한 유지했는가 ?
 - 도메인 용어로 네이밍 했는가 ?
 - 협력 구조가 자연스럽게 읽히는가 ?

src/main/java/nextstep/courses/domain/Course.java

@@ -38,6 +38,10 @@ public void addSession(Session session) {
         this.sessions.add(session);
     }
 
+    public Long getId() {
+        return id;
+    }
+
     public String getTitle() {
         return title;
     }

src/main/java/nextstep/courses/domain/image/CoverImage.java

@@ -6,6 +6,7 @@ public class CoverImage {
 
     public static final double MAX_FILE_SIZE = 1024 * 1024; // 1MB
 
+    private Long id;
     private final long size;
     private final ImageType type;
     private final ImageDimentsion dimentsion;
@@ -14,12 +15,21 @@ public CoverImage(long size, String type, int width, int height) {
         this(size, ImageType.fromName(type.toUpperCase()), width, height);
     }
 
+    public CoverImage(Long id, long size, String type, int width, int height) {
+        this(id, size, ImageType.fromName(type.toUpperCase()), width, height);
+    }
+
     public CoverImage(long size, ImageType type, int width, int height) {
-        this(size, type, new ImageDimentsion(width, height));
+        this(0L, size, type, new ImageDimentsion(width, height));
     }
 
-    public CoverImage(long size, ImageType type, ImageDimentsion dimentsion) {
+    public CoverImage(Long id, long size, ImageType type, int width, int height) {
+        this(id, size, type, new ImageDimentsion(width, height));
+    }
+
+    public CoverImage(Long id, long size, ImageType type, ImageDimentsion dimentsion) {
         validateFileSize(size);
+        this.id = id;
         this.size = size;
         this.type = type;
         this.dimentsion = dimentsion;
@@ -31,6 +41,10 @@ private void validateFileSize(long size) {
         }
     }
 
+    public Long getId() {
+        return id;
+    }
+
     public long getSize() {
         return size;
     }
@@ -39,5 +53,12 @@ public ImageType getType() {
         return type;
     }
 
+    public int getWidth(){
+        return this.dimentsion.getWidth();
+    }
+
+    public int getHeight(){
+        return this.dimentsion.getHeight();
+    }
 
 }

src/main/java/nextstep/courses/domain/image/CoverImageRepository.java

@@ -0,0 +1,8 @@
+package nextstep.courses.domain.image;
+
+public interface CoverImageRepository {
+
+    int save(CoverImage coverImage);
+
+    CoverImage findById(Long id);
+}

src/main/java/nextstep/courses/domain/image/ImageDimentsion.java

@@ -32,4 +32,12 @@ private void validateSize(int width, int height) {
     private static boolean isSize(int width, int height) {
         return width < MIN_WIDTH || height < MIN_HEIGHT;
     }
+
+    public int getWidth() {
+        return width;
+    }
+
+    public int getHeight() {
+        return height;
+    }
 }

src/main/java/nextstep/courses/domain/service/SessionService.java

@@ -0,0 +1,88 @@
+package nextstep.courses.domain.service;
+
+import nextstep.courses.domain.Course;
+import nextstep.courses.domain.CourseRepository;
+import nextstep.courses.domain.image.CoverImage;
+import nextstep.courses.domain.image.CoverImageRepository;
+import nextstep.courses.domain.session.Enrollment;
+import nextstep.courses.domain.session.Enrollments;
+import nextstep.courses.domain.session.Session;
+import nextstep.courses.domain.session.repository.EnrollmentRepository;
+import nextstep.courses.domain.session.repository.SessionRepository;
+import nextstep.courses.record.EnrollmentRecord;
+import nextstep.courses.record.SessionRecord;
+import nextstep.payments.domain.Payment;
+import nextstep.payments.repository.PaymentRepository;
+import nextstep.users.domain.NsUser;
+import nextstep.users.domain.UserRepository;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+import java.util.List;
+import java.util.Optional;
+
+@Service
+public class SessionService {
+
+    private final SessionRepository sessionRepository;
+    private final CoverImageRepository coverImageRepository;
+    private final CourseRepository courseRepository;
+    private final EnrollmentRepository enrollmentRepository;
+    private final UserRepository userRepository;
+    private final PaymentRepository paymentRepository;
+
+    public SessionService(SessionRepository sessionRepository,
+                          CoverImageRepository coverImageRepository,
+                          CourseRepository courseRepository,
+                          EnrollmentRepository enrollmentRepository,
+                          UserRepository userRepository,
+                          PaymentRepository paymentRepository) {
+        this.sessionRepository = sessionRepository;
+        this.coverImageRepository = coverImageRepository;
+        this.courseRepository = courseRepository;
+        this.enrollmentRepository = enrollmentRepository;
+        this.userRepository = userRepository;
+        this.paymentRepository = paymentRepository;
+    }
+
+    public Session findById(Long id) {
+        SessionRecord sessionRecord = sessionRepository.findById(id);
+        CoverImage saveCoverImage = coverImageRepository.findById(sessionRecord.getCoverImageId());
+        Course saveCourse = courseRepository.findById(sessionRecord.getCourseId());
+        List<EnrollmentRecord> enrollmentRecords = enrollmentRepository.findBySessionId(id);
+        Enrollments enrollments = toEnrollments(enrollmentRecords);
+
+        return sessionRecord.toSession(saveCourse, saveCoverImage, enrollments);
+    }
+
+    @Transactional
+    public int save(Session session) {
+        coverImageRepository.save(session.getCoverImage());
+        return sessionRepository.save(session.toSessionRecord());
+    }
+
+    @Transactional
+    public int saveEnrollment(Enrollment enrollment, Payment payment) {
+        SessionRecord sessionRecord = sessionRepository.findById(enrollment.getSessionId());
+        List<EnrollmentRecord> enrollmentRecords = enrollmentRepository.findBySessionId(enrollment.getSessionId());
+        Session session = sessionRecord.toSession(null, null, toEnrollments(enrollmentRecords));
+        session.addEnrollment(enrollment, payment);
+
+        paymentRepository.save(payment);
+
+        return enrollmentRepository.save(enrollment);
+    }
+
+    private Enrollments toEnrollments(List<EnrollmentRecord> enrollmentRecords) {
+        Enrollments enrollments = new Enrollments();
+
+        for (EnrollmentRecord enrollmentRecord : enrollmentRecords) {
+            Optional<NsUser> user = userRepository.findById(enrollmentRecord.getUserId());
+            enrollments.add(enrollmentRecord.toEnrollment(user.orElse(null)));
+        }
+
+        return enrollments;
+    }
+
+
+}

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

@@ -1,26 +1,35 @@
 package nextstep.courses.domain.session;
 
-import nextstep.payments.domain.Payment;
 import nextstep.users.domain.NsUser;
 
 import java.util.Objects;
 
 public class Enrollment {
 
+    private final Long id;
     private final NsUser user;
     private final Long sessionId;
-    private final Payment payment;
 
-    public Enrollment(NsUser user, Long sessionId, Payment payment) {
+    public Enrollment(NsUser user, Long sessionId) {
+        this(0L, user, sessionId);
+    }
+
+    public Enrollment(Long id, NsUser user, Long sessionId) {
+        this.id = id;
         this.user = user;
         this.sessionId = sessionId;
-        this.payment = payment;
     }
 
-    public void isPaymentAmount(SessionPolicy sessionPolicy) {
-        if(sessionPolicy.matchAmount(payment)){
-            throw new IllegalArgumentException("강의 금액과 결제 금액이 일치하지 않습니다.");
-        }
+    public Long getId() {
+        return id;
+    }
+
+    public NsUser getUser() {
+        return user;
+    }
+
+    public Long getSessionId() {
+        return sessionId;
     }
 
     @Override
@@ -32,7 +41,15 @@ public boolean equals(Object o) {
 
     @Override
     public int hashCode() {
-        return Objects.hash(user, sessionId, payment);
+        return Objects.hash(user, sessionId);
     }
 
+    @Override
+    public String toString() {
+        return "Enrollment{" +
+                "id=" + id +
+                ", user=" + user +
+                ", sessionId=" + sessionId +
+                '}';
+    }
 }