🚀 2단계 - 수강신청(도메인 모델) 미션완료 #536

kbzz17 · 2024-05-13T12:42:09Z

payment가 구현 시 조금 헷갈렸는데,

제가 구현한 Payment, payments의 의도가 출제 의도와 맞는지 궁금합니다 !!

javajigi

2단계 미션을 도메인 객체 설계부터 구현 잘 했네요. 💯
전체적인 객체 설계 잘 했네요.
추가로 피드백 몇 개 남겼으니 반영해 보면 좋겠어요.

javajigi · 2024-05-14T00:21:34Z

src/main/java/nextstep/courses/CanNotApplyException.java

@@ -0,0 +1,9 @@
+package nextstep.courses;
+
+public class CanNotApplyException extends Exception{


Suggested change

public class CanNotApplyException extends Exception{

public class CanNotApplyException extends Exception {

javajigi · 2024-05-14T00:22:10Z

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

@@ -13,6 +15,8 @@ public class Course {

    private LocalDateTime updatedAt;

+    private List<Session> sessions;


세션을 추가하는 addSession()과 같은 메서드도 있어야 하지 않을까?

javajigi · 2024-05-14T00:23:19Z

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

+    private final ImageType type;
+    private final ImageSize size;
+
+    public Image(ImageCapacity capacity, ImageType type, ImageSize size) {


Suggested change

public Image(ImageCapacity capacity, ImageType type, ImageSize size) {

public Image(int capacity, ImageType type, int width, int height ) {

위아 같은 생성자도 추가로 지원하는 것은 어떨까?

javajigi · 2024-05-14T00:23:32Z

src/main/java/nextstep/courses/domain/Image/ImageCapacity.java

+
+import nextstep.courses.OutOfImageCapacityException;
+
+public class ImageCapacity {


javajigi · 2024-05-14T00:23:38Z

src/main/java/nextstep/courses/domain/Image/ImageSize.java

+import nextstep.courses.ImageSizeRangeException;
+import nextstep.courses.ImageSizeRatioException;
+
+public class ImageSize {


javajigi · 2024-05-14T00:23:49Z

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

+import nextstep.courses.domain.Course;
+import nextstep.courses.domain.Image.Image;
+
+public class FreeSession extends Session{


Suggested change

public class FreeSession extends Session{

public class FreeSession extends Session {

javajigi · 2024-05-14T00:25:20Z

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

+    private final LocalDateTime startDate;
+    private final LocalDateTime endDate;
+
+    public Period(LocalDateTime startDate, LocalDateTime endDate) {


시작일은 종료일보다 앞서야 한다는 유효성 체크를 추가해야 하지 않을까?

javajigi · 2024-05-14T00:26:30Z

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

+
+    public abstract boolean isAppliable(int pay, int order) throws CanNotApplyException;
+
+    public void apply(int pay, NsUser user) throws Exception {


Session의 인스턴스 변수가 많아지고 있다.
Session이 가지는 값 중 수강신청과 관련있는 값을 가지는 Enrollment와 같은 객체를 분리하고 수강 신청 로직을 이 객체로 분리하면 어떨까?
분리했을 때 기능 구현 및 테스트 관점에서 얻는 이점이 있을까?

javajigi · 2024-05-14T00:27:12Z

src/main/java/nextstep/payments/domain/Payments.java

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


결제가 완료됐다고 가정하고 기능 구현하고 있기 때문에 Payment 관련해 특별히 많은 시간을 투자하지 않아도 된다.

javajigi · 2024-05-14T00:27:36Z

src/test/java/nextstep/courses/domain/session/PaidSessionTest.java

+
+        int fee = 20000;
+        int limit = 10;
+        session = new PaidSession(image, period, status, course, fee, limit);


Session 객체와 같이 인스턴스 변수가 많은 객체를 테스트하려면 객체를 생성하는데 어려움이 있다.
중복 코드 또한 많이 발생해 Session을 생성할 때 생성자의 인자가 변경되는 경우 변경할 부분이 많아진다.
https://www.arhohuttunen.com/test-data-builders/ 문서 참고해 Session과 같이 복잡한 객체의 테스트 데이터를 생성할 때 어떤 방법을 사용할 것인지 선택해 보면 좋겠다.
이번 기회에 내가 선호하는 방법을 적용해 보고 앞으로도 쭈욱 활용하는 방식이면 좋겠다.

kbzz17 added 12 commits May 12, 2024 19:16

[feat] 이미지 용량은 1MB 이하여야 한다.

7576ecd

[feat] 허용되는 이미지 타입은 제한되어 있다.

90d9f98

[feat] 이미지 크기와 비율이 맞지 않으면 에러 발생

9092248

[feat] 이미지 클래스 생성

803f52e

[feat] 강의는 시작일과 종료일을 가진다.

fb85cd9

[feat] 유료 강의는 강의료를 지불해야 하고 수강인원 제한이 있다.

1e8a01e

[feat] 무료 강의는 돈과 수강생 제한이 없다.

d34226c

[feat] 강의 상태는 준비, 모집, 마감으로 이루어진다.

9c2642f

[feat] 수강신청한 유저들을 담는 Users 클래스 개발

500f8d6

[feat] 강의 수강신청 기능 개발

373c0ce

[feat] Course, Session 맵핑

4dd4621

[feat] payment 구현

80b91d5

javajigi requested changes May 14, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 2단계 - 수강신청(도메인 모델) 미션완료 #536

🚀 2단계 - 수강신청(도메인 모델) 미션완료 #536

kbzz17 commented May 13, 2024

javajigi left a comment

javajigi May 14, 2024

javajigi May 14, 2024

javajigi May 14, 2024

javajigi May 14, 2024

javajigi May 14, 2024

javajigi May 14, 2024

javajigi May 14, 2024

javajigi May 14, 2024

javajigi May 14, 2024

javajigi May 14, 2024

		@@ -0,0 +1,9 @@
		package nextstep.courses;

		public class CanNotApplyException extends Exception{

		@@ -13,6 +15,8 @@ public class Course {

		private LocalDateTime updatedAt;

		private List<Session> sessions;

	public Image(ImageCapacity capacity, ImageType type, ImageSize size) {
	public Image(int capacity, ImageType type, int width, int height ) {


		import nextstep.courses.OutOfImageCapacityException;

		public class ImageCapacity {

	public class FreeSession extends Session{
	public class FreeSession extends Session {


		public abstract boolean isAppliable(int pay, int order) throws CanNotApplyException;

		public void apply(int pay, NsUser user) throws Exception {

🚀 2단계 - 수강신청(도메인 모델) 미션완료 #536

Are you sure you want to change the base?

🚀 2단계 - 수강신청(도메인 모델) 미션완료 #536

Conversation

kbzz17 commented May 13, 2024

javajigi left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment