🚀 2단계 - 수강신청(도메인 모델) #691

nno0obb · 2025-04-08T15:54:18Z

제가 잘 이해하지 못한 것 같습니다...
요구사항에 맞춰서 테스트가 모두 없는 것이 맞을까요?
필요하면 Repository 코드도 수정하는 것이 맞는 걸까요?
비즈니스 로직을 제 나름대로 클래스에 넣어보았는데... 괜찮은 건지 모르겠습니다.

CourseCoverImage 는 자체적으로 이미지를 넣어서 테스트 하는 것이 맞을까요?
"Image" 관련 자바 클래스까지 공부하는 것을 의도하신 것인지 궁금합니다.

NsUser 에 "func::registerCourse" 를 넣었는데 이 부분은 어떻게 테스트를 해야하는 것인가요?
직접 강의, 유저, 모두 만들면 되는 것일까요?
이를 위해 AttendeeList 가 꼭 필요해진것 같은데 요구사항에는 명시되어 있지 않지만 나름대로 구현하는 것까지가 이번 스텝의 의도일까요?

jinyoungchoi95

안녕하세요 홍균님 😄

몇가지 코멘트 남겨두었으니 확인부탁드려요 :)

궁금하거나 고민이 되는 부분이 있으시다면 언제든 pr 코멘트 또는 dm으로 요청 부탁드립니다.
감사합니다 🙇‍♂️

jinyoungchoi95 · 2025-04-10T11:15:48Z

src/main/java/nextstep/README.md

테스트 추가나 수정은 자유롭게 진행해주셔도 됩니다 😄

jinyoungchoi95 · 2025-04-10T11:16:54Z

gradle.properties

+org.gradle.java.home=/opt/homebrew/Cellar/openjdk@11/11.0.26/libexec/openjdk.jdk/Contents/Home
+# 넵, 위 설정은 IDE 에서 Java 실행 환경을 위해 필요합니다.


이걸 설정하게되면 gradle을 실행하는 과정에서 해당 path가 비어있는 다른 사람의 경우에는 실행이 불가능할 것 같아요. 실제 서버에서도 누락되면 실행이 되지 않을 것이구요.

어플리케이션 실행과 관련된 gradle을 설정할때는 가능하면 특정 path에 의존하지 않는 것이 좋습니다 :)

jinyoungchoi95 · 2025-04-10T11:53:59Z

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

+
+import javax.imageio.ImageIO;
+
+public class CourseCoverImage {


CourseCoverImage 는 자체적으로 이미지를 넣어서 테스트 하는 것이 맞을까요?
"Image" 관련 자바 클래스까지 공부하는 것을 의도하신 것인지 궁금합니다.

자바클래스를 학습할 정도까지는 필요없고, 요구된 내용의 이미지 관련된 도메인로직을 처리하는 정도면 충분할 것 같습니다 😄

jinyoungchoi95 · 2025-04-10T11:57:05Z

src/main/java/nextstep/users/domain/NsUser.java

@@ -124,6 +128,22 @@ public boolean isGuestUser() {
        }
    }

+    public void registerCourse(Course course) {


NsUser 에 "func::registerCourse" 를 넣었는데 이 부분은 어떻게 테스트를 해야하는 것인가요?

말씀해주신 내용을 이해못했어요. NsUser.registerCourse를 테스트하는 것은 NsUserTest에서 단위 테스트를 만들어보면 되는 것 아닐까요?

조금 더 자세하게 말씀해주시면 확인해보겠습니다 😄

jinyoungchoi95 · 2025-04-10T11:58:00Z

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

+
+import nextstep.users.domain.NsUser;
+
+public class AttendeeList {


이를 위해 AttendeeList 가 꼭 필요해진것 같은데 요구사항에는 명시되어 있지 않지만 나름대로 구현하는 것까지가 이번 스텝의 의도일까요?

요구사항에는 어떤 객체를 만들어라라는 구현내용은 명시하지 않기 때문에 홍균님이 생각해주시는 방향대로 구현을 해주시면 되어요 :)

jinyoungchoi95 · 2025-04-10T12:08:15Z

src/main/java/nextstep/users/domain/NsUser.java

+        if (course.getAttendees().size() >= course.getMaxAttendees()) {
+            throw new IllegalArgumentException("강의 수강 인원이 초과되었습니다.");
+        }


유료 강의는 강의 최대 수강 인원을 초과할 수 없다.

누가 수강등록에 대한 것을 가져가느냐에 대한 것을 고민해보면 좋을 것 같아요 😄

"수강"이라는 것은 강의에 회원이 들어가는 것이므로 Course에서 수강하고 있는 인원을 들고 있는 것이 더 자연스럽지 않을까요? 그럼 Course.등록()에서는 유료강의인 경우 이미 수강최대인원이 가득찬 케이스에서 예외를 터트리도록 해줄 수 있을 것 같구요 :)

jinyoungchoi95 · 2025-04-10T12:08:57Z

src/main/java/nextstep/users/domain/NsUser.java

+        if (course.getCourseStatus() == CourseStatus.PREPARING) {
+            throw new IllegalArgumentException("강의가 준비중입니다.");
+        }


이런 로직도 Course에 들어가보면 좋을 것 같아요!

강의에 대한 검증 책임 자체를 Course에 넣어주는 걸 고려해보시면 좋겠습니다 😄

jinyoungchoi95 · 2025-04-10T12:09:25Z

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

+    private String imageFilePath;
+    private File imageFile;
+
+    private BufferedImage courseCoverImageData;


Suggested change

private String imageFilePath;

private File imageFile;

private BufferedImage courseCoverImageData;

private final String imageFilePath;

private final File imageFile;

private final BufferedImage courseCoverImageData;

불변할 수 있겠네요!

jinyoungchoi95 · 2025-04-10T12:09:58Z

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

+        }
+
+        if (courseCoverImageData.getWidth() < MIN_WIDTH || courseCoverImageData.getHeight() < MIN_HEIGHT) {
+            throw new IllegalArgumentException("강의 커버 이미지는 최소 300x200 픽셀 이상이어야 합니다.");


이미지 크기는 1MB 이하여야 한다.

도 추가되면 좋겠네요 😄

jinyoungchoi95 · 2025-04-10T12:10:11Z

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

+            imageFilePath = DEFAULT_IMAGE_FILE_PATH;
+        }
+
+        if (Arrays.stream(SUPPORTED_IMAGE_EXTENSIONS).noneMatch(imageFilePath::endsWith)) {


kakao-bt-7274 added 12 commits April 8, 2025 22:25

[LMS] step1: Feedback

9c08e02

[LMS] step1: Feedback next-step#1

b5f0d4d

[LMS] step1: Feedback next-step#2

4d5593c

[LMS] step1: Feedback next-step#3

8ee8f65

[LMS] step2: README.md

214234a

[LMS] step2: Design Domain

e26e30c

[LMS] step2: README.md

f0e6e28

[LMS] step2: CourseCoverImage

51577a7

[LMS] step2: README.md

4341b6c

[LMS] step2: CourseCoverImageTest

de1e19a

[LMS] step2: Course

15d7dd2

[LMS] step2: Course

d03f4aa

jinyoungchoi95 requested changes Apr 10, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 2단계 - 수강신청(도메인 모델) #691

🚀 2단계 - 수강신청(도메인 모델) #691

nno0obb commented Apr 8, 2025

jinyoungchoi95 left a comment

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

jinyoungchoi95 Apr 10, 2025

		org.gradle.java.home=/opt/homebrew/Cellar/openjdk@11/11.0.26/libexec/openjdk.jdk/Contents/Home
		# 넵, 위 설정은 IDE 에서 Java 실행 환경을 위해 필요합니다.


		import javax.imageio.ImageIO;

		public class CourseCoverImage {


		import nextstep.users.domain.NsUser;

		public class AttendeeList {

🚀 2단계 - 수강신청(도메인 모델) #691

Are you sure you want to change the base?

🚀 2단계 - 수강신청(도메인 모델) #691

Conversation

nno0obb commented Apr 8, 2025

jinyoungchoi95 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