Skip to content

3단계 - 수강신청(DB 적용) #769

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: heewonham
Choose a base branch
from
Open

3단계 - 수강신청(DB 적용) #769

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
845e2a8
feat : (review) 값 객체 equal, hashcode 추가
Apr 21, 2025
fbca015
feat : (review) students 일급 컬렉션 추가 및 테스트 코드 추가
Apr 21, 2025
37e1180
feat : (review) 상수 위치 변경, 의미있는 상수 이름 변경, 함수 추가
Apr 21, 2025
623f840
feat : session Repository 및 테스트 코드 추가
Apr 21, 2025
c17e5ec
feat : sessionStudent repository 및 테스트 코드 추가
Apr 21, 2025
f8e00f3
feat : (review) namedjdbctemplate 사용 및 sessionstudent 레포삭제
Apr 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions src/main/java/nextstep/courses/domain/Image.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,16 +1,16 @@
package nextstep.courses.domain;

public class Image {
private static final int MAX_IMAGE_SIZE = 1024 * 1024;
private static final int WIDTH_LIMIT = 300;
private static final int HEIGHT_LIMIT = 200;
private static final double REQUIRED_ASPECT_RATIO = 1.5;
private static final double ASPECT_RATIO_TOLERANCE = 0.01;
private String fileName;
private String contentType;
private long sizeInBytes;
private int width;
private int height;
private static final int IMAGE_SIZE_1MB = 1024 * 1024;
private static final int WIDTH_LIMIT = 300;
private static final int HEIGHT_LIMIT = 200;
private static final double REQUIRED_ASPECT_RATIO = 1.5;
private static final double ASPECT_RATIO_TOLERANCE = 0.01;

public Image(String fileName, String contentType, long sizeInBytes, int width, int height) {
validate(contentType, sizeInBytes, width, height);
Expand All @@ -22,7 +22,7 @@ public Image(String fileName, String contentType, long sizeInBytes, int width, i
}

private void validate(String contentType, long sizeInBytes, int width, int height) {
if (sizeInBytes > IMAGE_SIZE_1MB) {
if (sizeInBytes > MAX_IMAGE_SIZE) {
throw new IllegalArgumentException("이미지 크기는 1MB 이하여야 합니다.");
}

Expand Down
2 changes: 1 addition & 1 deletion src/main/java/nextstep/courses/domain/PaidSession.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ protected void validateRegistration(Long studentId, Payment payment) {
if (!studentId.equals(payment.getNsUserId())) {
throw new IllegalArgumentException("결제한 사용자와 일치하지 않습니다.");
}
if (this.tuitionFee.getAmount() != payment.getAmount()) {
if (!this.tuitionFee.isSameAmount(payment.getAmount())) {
throw new IllegalArgumentException("결제 금액과 일치하지 않습니다.");
}
}
Expand Down
5 changes: 3 additions & 2 deletions src/main/java/nextstep/courses/domain/TuitionFee.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,9 @@ public TuitionFee(int amount) {
this.amount = amount;
}

public int getAmount() {
return amount;
public boolean isSameAmount(int amount) {
Copy link
Author

@heewonham heewonham Apr 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#728 (comment)
#728 (comment)
이 두 부분이 약간 상충되는 것 같아 질문드립니다.
어떤 부분은 get을 쓰고 어떤 부분은 이렇게 함수를 만드는데 구분할 수 있는 방법은 무엇일까요?

Copy link
Member

@nooose nooose Apr 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋은 질문입니다! 👍

로직 흐름에서 도메인 객체의 책임에 해당하는 동작이라면,
객체에 메시지를 보내는 방식을 사용합니다.

예를 들어 사용자 나이 관련 로직이 있다고 가정해보겠습니다.

// 나이 증가
var age = member.getAge(); // 게터 사용
member.setAge(age + 1);

member.increaseAge(); // 메시지 전달 (게터 사용X)

// 성인 여부로 특정 기능 사용을 판단하는 로직
boolean isAdult = member.getAge() >= 19; //  게터 사용
boolean isAdult = member.isAdult(); // 메시지 전달 권장 (게터 사용X)

// 나이가 증가했는지 검증 (게터 사용)
var member = new Member(age = 10);
member.increaseAge();
member.getAge() == 11 // true 기대, 게터로 충분한 검증 가능

// 외부 응답 규격 DTO 생성 (게터 사용)
new MemberResponse(member.id, member.age);

단순히 값을 꺼내서 검증하거나, 외부로 값을 전달하기 위한 목적이라면
게터를 사용하는 것이 자연스럽다고 생각해요

Copy link
Member

@nooose nooose Apr 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그럼에도 불구하고 게터만으로 객체의 상태를 검증하기 어려운 경우,
테스트를 목적으로 하는 메서드를 public 대신 protected로 선언하는 방법도 있습니다.

보통 테스트 클래스는 테스트 대상 클래스와 같은 패키지에 위치하므로, protected 메서드에도 접근할 수 있어서요.
이러면 테스트의 편의성을 확보하면서도, 외부에서의 불필요한 접근은 제한할 수 있게됩니다 😄

Copy link
Author

@heewonham heewonham Apr 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nooose 아하 이해했습니다. 도메인 객체 책임에 해당하는지를 생각해보면 되겠군요!!
상세한 설명 감사합니다 🙏

return this.amount == amount;
}

@Override
public boolean equals(Object o) {
Expand Down