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

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;
+    }
+}