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

chanani · 2025-12-10T06:19:12Z

안녕하세요 ! 수강신청 4단계 리뷰 요청드립니다.🙇‍♂️

최초 테이블 설계 시 session 테이블에서 cover_image_id를 가지고 있었습니다.
1:1 관계에서 1:N 상태로 변경되면서 기존과 다르게 cover_image 테이블에서 session_id를 가지고 있어야 된다고 판단되어 불가피하게 테이블을 수정했습니다.
DB 테이블에 데이터가 존재한다는 가정하에 리팩터링을 진행하라는 요구사항이 있었는데, 이를 지키지 못했습니다.

제가 마지막 요구사항을 잘 이해하고 진행한건지 확신이 들지 않습니다.
우아한테크코스(무료), 우아한테크캠프 Pro(유료)와 같이 선발된 인원만 수강 가능해야 한다. 라는 조건에서 선발 여부도 수정이 가능한 쪽으로 작업을 진행했는데, 혹시라도 요구사항을 제대로 파악하지 못한 부분이 있을 경우 수정하겠습니다.

… 통해서 Entity, Domain 클래스 반환)

… 후 필요한 클래스에 상속(SessionCore 요구사항 추가로 인해 상속이 아닌 인스턴스 변수로 사용하는 것이 좋겠다고 판단.)

SessionCoreV2에서 추가된 요구사항 적용 예정

…드 작성

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

refactor : 수강신청 객체에 상태값 추가 및 테스트 코드 작성

javajigi

드디어 미션의 마지막 단계넨요. 👍
요구사항은 정확히 잘 파악해 구현 잘 했네요.
단, Session의 복잡도를 낮추고 인터페이스 사용과 관련한 피드백 남겼으니 한번 고민해 보는 시간을 가지면 좋겠습니다.

javajigi · 2025-12-10T22:43:02Z

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

+import java.util.ArrayList;
+import java.util.List;
+
+public class CoverImages {


일급 콜렉션 적용 👍

javajigi · 2025-12-10T22:44:01Z

src/main/java/nextstep/courses/domain/session/constant/EnrollmentStatus.java

+
+import java.util.Arrays;
+
+public enum EnrollmentStatus {


javajigi · 2025-12-10T22:44:08Z

src/main/java/nextstep/courses/domain/session/constant/SelectionStatus.java

+
+import java.util.Arrays;
+
+public enum SelectionStatus {


javajigi · 2025-12-10T22:44:21Z

src/main/java/nextstep/courses/domain/session/constant/SessionRecruitmentStatus.java

@@ -0,0 +1,15 @@
+package nextstep.courses.domain.session.constant;
+
+public enum SessionRecruitmentStatus {


상태 값을 분리 👍

javajigi · 2025-12-10T22:46:21Z

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

+    }
+
+    private boolean isSelectedAndCancelled(EnrollmentStatus enrollmentStatus) {
+        return this.selectionStatus.equals(SelectionStatus.SELECTED) && enrollmentStatus.equals(EnrollmentStatus.CANCELLED);


Suggested change

return this.selectionStatus.equals(SelectionStatus.SELECTED) && enrollmentStatus.equals(EnrollmentStatus.CANCELLED);

return this.selectionStatus.isSelected() && enrollmentStatus.isCancled();

enum도 class와 같이 메시지를 보내는 방식으로 구현 가능함

javajigi · 2025-12-10T22:48:25Z

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

+
+import java.time.LocalDateTime;
+
+public interface SessionCoreFacade {


인터페이스의 메서드가 너무 많다.
인터페이스의 메서드가 너무 많을 경우 인터페이스 설계가 잘못되었을 가능성이 높음
이럴 경우 인터페이스를 책임에 따라 분리하거나, 불필요한 메서드를 제거할 필요가 있음
또한 이 인터페이스가 필요한지에 대해서도 고민해 보면 좋겠다.

javajigi · 2025-12-10T22:52:38Z

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

        SessionRecord sessionRecord = sessionRepository.findById(enrollment.getSessionId());
        List<EnrollmentRecord> enrollmentRecords = enrollmentRepository.findBySessionId(enrollment.getSessionId());
-        Session session = sessionRecord.toSession(null, null, toEnrollments(enrollmentRecords));
+        Session session = SessionMapper.toDomain(sessionRecord, null, null, toEnrollments(enrollmentRecords));
        session.addEnrollment(enrollment, payment);

        paymentRepository.save(payment);

        return enrollmentRepository.save(enrollment);


이 미션의 객체 설계에서 수강 신청 가능 여부를 확인하는 것이 복잡도가 가장 높은 부분이이다.
이 부분은 Session 객체 내에 두는 것이 아니라 아래와 같이 Session을 통해 수강신청에 필요한 객체를 아래의 Enrollment와 같은 객체로 생성한 후 위임함으로써 복잡도를 낮추는 방식으로 접근해 보는 것은 어떨까?

아래 코드에서 Enrollment는 수강 신청 가능 여부를 판단하는 로직을 담당하는 객체로 찬한님 코드의 Enrollment와는 다름
찬한님의 Enrollment는 아래 코드에서 Student임

SessionRecord sessionRecord = ... List<EnrollmentRecord> enrollmentRecords = ... Enrollment enrollment = sessionRecord.createEnrollment(enrollmentRecords); Student student = enrollment.enroll(user); enrollmentRepository.save(student.getUserId(), student.getSessionId());

위 코드에서 리팩터링 힌트를 얻을 수 있는 부분이 있다면 리팩터링해 본다.

chanani added 9 commits December 7, 2025 20:12

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

504ad4d

refactor : 피드백 받은 내용 수정 진행(Session, SessionRecord에서 직접 변환하지 않고 Mapper…

487809c

… 통해서 Entity, Domain 클래스 반환)

refactor : Session 클래스에서 상속 받던 SessionCore 인스턴스 변수로 이동, BaseEntity 생성…

4260bb7

… 후 필요한 클래스에 상속(SessionCore 요구사항 추가로 인해 상속이 아닌 인스턴스 변수로 사용하는 것이 좋겠다고 판단.)

refactor : SessionCoreFacade interface 생성, SessionCoreV2 생성

48db9b3

SessionCoreV2에서 추가된 요구사항 적용 예정

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

cb43233

…드 작성

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

b43e419

feat : 선발 여부, 승인 여부 상태 Enum 생성

ec74984

refactor : 수강신청 객체에 상태값 추가 및 테스트 코드 작성

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

adf6a1b

javajigi requested changes Dec 10, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

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

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

Uh oh!

chanani commented Dec 10, 2025

Uh oh!

javajigi left a comment

Uh oh!

javajigi Dec 10, 2025

Uh oh!

javajigi Dec 10, 2025

Uh oh!

javajigi Dec 10, 2025

Uh oh!

javajigi Dec 10, 2025

Uh oh!

javajigi Dec 10, 2025

Uh oh!

javajigi Dec 10, 2025

Uh oh!

javajigi Dec 10, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		@@ -0,0 +1,15 @@
		package nextstep.courses.domain.session.constant;

		public enum SessionRecruitmentStatus {

	return this.selectionStatus.equals(SelectionStatus.SELECTED) && enrollmentStatus.equals(EnrollmentStatus.CANCELLED);
	return this.selectionStatus.isSelected() && enrollmentStatus.isCancled();


		import java.time.LocalDateTime;

		public interface SessionCoreFacade {

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

Are you sure you want to change the base?

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

Uh oh!

Conversation

chanani commented Dec 10, 2025

Uh oh!

javajigi left a comment

Choose a reason for hiding this comment

Uh oh!

javajigi Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

javajigi Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants