🚀 4단계 - 수강신청(요구사항 변경) (#808)

* docs : 목표 및 요구사항 md 파일 작성

* refactor : 피드백 받은 내용 수정 진행(Session, SessionRecord에서 직접 변환하지 않고 Mapper 통해서 Entity, Domain 클래스 반환)

* refactor : Session 클래스에서 상속 받던 SessionCore 인스턴스 변수로 이동, BaseEntity 생성 후 필요한 클래스에 상속(SessionCore 요구사항 추가로 인해 상속이 아닌 인스턴스 변수로 사용하는 것이 좋겠다고 판단.)

* refactor : SessionCoreFacade interface 생성, SessionCoreV2 생성
SessionCoreV2에서 추가된 요구사항 적용 예정

* refactor : 강의 진행 상태 enum 정보 변경, SessionCoreV2 클래스에 모집 상태 값 추가 및 테스트 코드 작성

* feat : 커버 이미지 일급컬랙션 생성 및 테스트 코드 생성

* refactor : 강의에 여러 이미지 등록할 수 있도록 수정
- session 테이블 에서 cover_image_id를 관리했는데 cover_image 테이블에서 session_id 관리할 수 있도록 수정
- DB 테이블에 데이터가 존재한다는 가정하에 리팩터링 진행하라는 요구사항이 있었는데, 최초 DB 설계 시 문제로 이를 지키지 못했습니다. 하지만 실무에서 이런 상황이 온다면, 기존 데이터에 문제되지 않도록 스크립트를 작성할 것같습니다.

* feat : 선발 여부, 승인 여부 상태 Enum 생성
refactor : 수강신청 객체에 상태값 추가 및 테스트 코드 작성

* refactor : 스키마, sql 수정, Jdbc 수강신청 등록, 조회 수정

* refactor : enum에 메시지 보낼 수 있도록 메서드 추가

* refactor : SessionCoreFacade 메서드 제거, SessionCoreV2를 SessionCore로 리팩터링

* refactor : 수강신청 검증로직 Session에서 EnrollmentApply 클래스로 이동 및 테스트 코드 일괄 수정(컴파일 오류 발생하지 않도록 작업 진행)

* docs : pr 링크 기입 및 README 수정

Author: chanani

README.md

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

docs/04-lms-refactor.md

@@ -0,0 +1,43 @@
+# 🚀 4단계 - 수강신청(요구사항 변경)
+
+***
+
+## 코드 리뷰
+
+> PR 링크 : [#808](https://github.com/next-step/java-lms/pull/808)
+
+## 나의 학습 목표
+
+### 1. 컴파일 오류가 발생하지 않도록 리팩터링하자.
+
+- 프로그래밍 요구사항에서와 같이 리팩터링할 때 컴파일 에러가 최대한 발생하지 않도록 구현한다.
+- 이 목표를 지키려면 기존 코드를 바로 수정하는 것이 아닌 기존 코드를 복사한 뒤 점진적으로 코드를 수정해 나아간다.
+
+## 핵심 학습 목표
+
+- DB 테이블이 변경될 때도 스트랭글러 패턴을 적용해 점진적인 리팩터링을 연습한다.
+    - [스트랭글러(교살자) 패턴 - 마틴 파울러](https://martinfowler.com/bliki/StranglerFigApplication.html)
+    - [스트랭글러 무화과 패턴](https://learn.microsoft.com/ko-kr/azure/architecture/patterns/strangler-fig)
+
+## 변경된 기능 요구사항
+
+### 강의 수강신청은 강의 상태가 모집중일 때만 가능하다.
+
+- 강의가 진행 중인 상태에서도 수강신청이 가능해야 한다.
+    - 강의 진행 상태(준비중, 진행중, 종료)와 모집 상태(비모집중, 모집중)로 상태 값을 분리해야 한다.
+
+### 강의는 강의 커버 이미지 정보를 가진다.
+
+- 강의는 하나 이상의 커버 이미지를 가질 수 있다.
+
+### 강사가 승인하지 않아도 수강 신청하는 모든 사람이 수강 가능하다.
+
+- 우아한테크코스(무료), 우아한테크캠프 Pro(유료)와 같이 선발된 인원만 수강 가능해야 한다.
+    - 강사는 수강신청한 사람 중 선발된 인원에 대해서만 수강 승인이 가능해야 한다.
+    - 강사는 수강신청한 사람 중 선발되지 않은 사람은 수강을 취소할 수 있어야 한다.
+
+## 프로그래밍 요구사항
+
+- 리팩터링할 때 컴파일 에러와 기존의 단위 테스트의 실패를 최소화하면서 점진적인 리팩터링이 가능하도록 한다.
+- DB 테이블에 데이터가 존재한다는 가정하에 리팩터링해야 한다.
+    - 즉, 기존에 쌓인 데이터를 제거하지 않은 상태로 리팩터링 해야 한다.

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

@@ -1,47 +1,35 @@
 package nextstep.courses.domain;
 
+import nextstep.courses.domain.session.BaseEntity;
 import nextstep.courses.domain.session.Session;
 
 import java.time.LocalDateTime;
 import java.util.ArrayList;
 import java.util.List;
 
-public class Course {
-    private Long id;
+public class Course extends BaseEntity {
 
     private String title;
 
     private Long creatorId;
 
-    private LocalDateTime createdAt;
-
-    private LocalDateTime updatedAt;
-
     private List<Session> sessions = new ArrayList<>();
 
-    public Course() {
-    }
 
     public Course(String title, Long creatorId) {
         this(0L, title, creatorId, LocalDateTime.now(), null);
     }
 
     public Course(Long id, String title, Long creatorId, LocalDateTime createdAt, LocalDateTime updatedAt) {
-        this.id = id;
+        super(id, createdAt, updatedAt);
         this.title = title;
         this.creatorId = creatorId;
-        this.createdAt = createdAt;
-        this.updatedAt = updatedAt;
     }
 
     public void addSession(Session session) {
         this.sessions.add(session);
     }
 
-    public Long getId() {
-        return id;
-    }
-
     public String getTitle() {
         return title;
     }
@@ -50,18 +38,12 @@ public Long getCreatorId() {
         return creatorId;
     }
 
-    public LocalDateTime getCreatedAt() {
-        return createdAt;
-    }
 
     @Override
     public String toString() {
         return "Course{" +
-                "id=" + id +
-                ", title='" + title + '\'' +
+                "  title='" + title + '\'' +
                 ", creatorId=" + creatorId +
-                ", createdAt=" + createdAt +
-                ", updatedAt=" + updatedAt +
                 '}';
     }
 }

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

@@ -7,6 +7,7 @@ public class CoverImage {
     public static final double MAX_FILE_SIZE = 1024 * 1024; // 1MB
 
     private Long id;
+    private final long sessionId;
     private final long size;
     private final ImageType type;
     private final ImageDimentsion dimentsion;
@@ -15,21 +16,22 @@ 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 id, Long sessionId, long size, String type, int width, int height) {
+        this(id, sessionId, size, ImageType.fromName(type.toUpperCase()), width, height);
     }
 
-    public CoverImage(long size, ImageType type, int width, int height) {
-        this(0L, size, type, new ImageDimentsion(width, height));
+    public CoverImage(long size,  ImageType type, int width, int height) {
+        this(0L, 0L, size, type, new ImageDimentsion(width, height));
     }
 
-    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 sessionId,long size, ImageType type, int width, int height) {
+        this(id, sessionId, size, type, new ImageDimentsion(width, height));
     }
 
-    public CoverImage(Long id, long size, ImageType type, ImageDimentsion dimentsion) {
+    public CoverImage(Long id, Long sessionId, long size, ImageType type, ImageDimentsion dimentsion) {
         validateFileSize(size);
         this.id = id;
+        this.sessionId = sessionId;
         this.size = size;
         this.type = type;
         this.dimentsion = dimentsion;

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

@@ -1,8 +1,10 @@
 package nextstep.courses.domain.image;
 
+import java.util.List;
+
 public interface CoverImageRepository {
 
     int save(CoverImage coverImage);
 
-    CoverImage findById(Long id);
+    List<CoverImage> findBySessionId(Long id);
 }

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

@@ -0,0 +1,34 @@
+package nextstep.courses.domain.image;
+
+import java.util.ArrayList;
+import java.util.List;
+
+public class CoverImages {
+
+    private List<CoverImage> coverImages;
+
+    public CoverImages() {
+        this.coverImages = new ArrayList<>();
+    }
+
+    public CoverImages(CoverImage... coverImage) {
+        this.coverImages = List.of(coverImage);
+    }
+
+    public CoverImages(List<CoverImage> coverImages) {
+        this.coverImages = coverImages;
+    }
+
+    public void add(CoverImage coverImage) {
+        coverImages.add(coverImage);
+    }
+
+    public int size() {
+        return coverImages.size();
+    }
+
+    public List<CoverImage> getCoverImages() {
+        return coverImages;
+    }
+
+}

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

@@ -4,9 +4,12 @@
 import nextstep.courses.domain.CourseRepository;
 import nextstep.courses.domain.image.CoverImage;
 import nextstep.courses.domain.image.CoverImageRepository;
+import nextstep.courses.domain.image.CoverImages;
 import nextstep.courses.domain.session.Enrollment;
+import nextstep.courses.domain.session.EnrollmentApply;
 import nextstep.courses.domain.session.Enrollments;
 import nextstep.courses.domain.session.Session;
+import nextstep.courses.domain.session.mapper.SessionMapper;
 import nextstep.courses.domain.session.repository.EnrollmentRepository;
 import nextstep.courses.domain.session.repository.SessionRepository;
 import nextstep.courses.record.EnrollmentRecord;
@@ -22,7 +25,7 @@
 import java.util.Optional;
 
 @Service
-public class SessionService {
+public class SessionService  {
 
     private final SessionRepository sessionRepository;
     private final CoverImageRepository coverImageRepository;
@@ -47,26 +50,29 @@ public SessionService(SessionRepository sessionRepository,
 
     public Session findById(Long id) {
         SessionRecord sessionRecord = sessionRepository.findById(id);
-        CoverImage saveCoverImage = coverImageRepository.findById(sessionRecord.getCoverImageId());
+        List<CoverImage> saveCoverImages = coverImageRepository.findBySessionId(id);
         Course saveCourse = courseRepository.findById(sessionRecord.getCourseId());
         List<EnrollmentRecord> enrollmentRecords = enrollmentRepository.findBySessionId(id);
         Enrollments enrollments = toEnrollments(enrollmentRecords);
 
-        return sessionRecord.toSession(saveCourse, saveCoverImage, enrollments);
+        return SessionMapper.toDomain(sessionRecord, saveCourse, saveCoverImages, enrollments);
     }
 
     @Transactional
     public int save(Session session) {
-        coverImageRepository.save(session.getCoverImage());
-        return sessionRepository.save(session.toSessionRecord());
+        for (CoverImage coverImage : session.getCoverImages()) {
+            coverImageRepository.save(coverImage);
+        }
+
+        return sessionRepository.save(SessionMapper.toEntity(session));
     }
 
     @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);
+    public int saveEnrollment(NsUser user, Long sessionId, Payment payment) {
+        SessionRecord sessionRecord = sessionRepository.findById(sessionId);
+        List<EnrollmentRecord> enrollmentRecords = enrollmentRepository.findBySessionId(sessionId);
+        EnrollmentApply enrollmentApply = new EnrollmentApply(toEnrollments(enrollmentRecords), payment, sessionRecord.createdSessionCore());
+        Enrollment enrollment = enrollmentApply.enroll(user, sessionId);
 
         paymentRepository.save(payment);
 
@@ -84,5 +90,4 @@ private Enrollments toEnrollments(List<EnrollmentRecord> enrollmentRecords) {
         return enrollments;
     }
 
-
 }

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

@@ -0,0 +1,28 @@
+package nextstep.courses.domain.session;
+
+import java.time.LocalDateTime;
+
+public abstract class BaseEntity {
+
+    private final Long id;
+    private final LocalDateTime createdAt;
+    private final LocalDateTime updatedAt;
+
+    public BaseEntity(Long id, LocalDateTime createdAt, LocalDateTime updatedAt) {
+        this.id = id;
+        this.createdAt = createdAt;
+        this.updatedAt = updatedAt;
+    }
+
+    public Long getId() {
+        return id;
+    }
+
+    public LocalDateTime getCreatedAt() {
+        return createdAt;
+    }
+
+    public LocalDateTime getUpdatedAt() {
+        return updatedAt;
+    }
+}