Step2 - 도메인설계 #486
Open
Step2 - 도메인설계 #486
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ksy90101
requested changes
Apr 15, 2024
| @@ -9,13 +9,12 @@ public class Course { | |||
|
|
|||
| private Long creatorId; | |||
|
|
|||
| private Sessions sessions; | |||
There was a problem hiding this comment.
과정이 기수별로 있다. 라는것은 과정 == Course // 기수 == Session으로
에를들어 현재 듣고 계신 TDD 과정이 있고 1기~18기까지가 있었죠.
즉, 1개의 Course에 여러개의 Session이 잇어야 할거 같습니다.
Comment on lines
+3
to
+8
| public class FreeSession { | ||
| private final Session session; | ||
|
|
||
| public FreeSession(Session session) { | ||
| this.session = session; | ||
| } |
There was a problem hiding this comment.
이걸 따로 만든 이유가 있나요?
유료와 무료를 나눠서 객체를 관리하고 싶다면 조합보단 상속이 더 어울리는 형태가 아닐까 생각이 들어요.
|
|
||
| public class ImageRate { | ||
|
|
||
| private final int MIN_WIDTH_SIZE = 300; |
There was a problem hiding this comment.
Suggested change
| private final int MIN_WIDTH_SIZE = 300; | |
| private static final int MIN_WIDTH_SIZE = 300; |
| } | ||
|
|
||
| private void validRate(int width, int height) { | ||
| if (width / height != 3 / 2) { |
|
|
||
| private final int MAX_SIZE = 1; | ||
|
|
||
| private int size; |
There was a problem hiding this comment.
이 size의 용량을 MB로 설정하셨군요.
기본적으로 어떤 용량으로 설정하는지 한번 고민해보면 좋을거 같습니다.
예를들어 S3는 용량을 default가 무엇일까요?
혹은 서버에 저장할때.
| public class PaySession { | ||
|
|
||
| private final Session session; | ||
| private int joinLimit; |
There was a problem hiding this comment.
- [] 유료 강의 최대 수강 인원 초과 검증.
이게 아직 안된거 같아요!
| private final LocalDateTime endDate; | ||
|
|
||
| public Period() { | ||
| this(LocalDateTime.now(), LocalDateTime.now().plusWeeks(1)); |
| } | ||
|
|
||
| private void validPaid(Payment payment) { | ||
| if(!payment.isPaid()) |
There was a problem hiding this comment.
실제 강의엔 수강료가 있고 그 수강료 만큼 가격을 내야 하지 않을까요?
유료 강의는 수강생이 결제한 금액과 수강료가 일치할 때 수강 신청이 가능하다.
| this.period = new Period(); | ||
| } | ||
|
|
||
| public void join(JoinUser joinUser) { |
There was a problem hiding this comment.
강의 수강신청은 강의 상태가 모집중일 때만 가능하다.
이것도 필요하지 않을까요?
| @@ -0,0 +1,9 @@ | |||
| package nextstep.courses.domain; | |||
|
|
|||
| public class FreeSession { | |||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요 2단계 리뷰 부탁드립니다.
과정이 기수별로 관리된다고했는데 기수를 과정의 상위로 봐야할지 과정의 일부로 봐야할지 잘 모르겠습니다.
어떤 방향으로 설정하는게 맞을까요?