RentalCard 의 calculateLateFee 메서드 구현이 잘못된 것 같습니다.

Question

안녕하세요. calculateLateFee 메서드에 작성해주신 코드 중 아래 부분에 오류가 있는 것이 아닌지 확인 부탁드립니다. private void calculateLateFee(RentalItem rentalItem, LocalDate returnDate) { if (returnDate.compareTo(rentalItem.getOverdueDate()) > 0) { int point = Period.between(rentalItem.getOverdueDate(), returnDate).getDays() * 10; this.lateFee.addPoint(point); //문제 } } 아래처럼 수정되어야 하는 것이 아닌지 질문드립니다. private void calculateLateFee(RentalItem rentalItem, LocalDate returnDate) { if (returnDate.compareTo(rentalItem.getOverdueDate()) > 0) { int point = Period.between(rentalItem.getOverdueDate(), returnDate).getDays() * 10; this.setLateFee(this.lateFee.addPoint(point)); //수정 } } 감사합니다.

han jeong heon · Answer

tkwoo님 안녕하세요. 강의자입니다. 네 지적하신 점이 맞습니다. 제가 코드의 결함을 인지하고 제공하는 소스코드에는 이후 변경하였는데, 동영상 상에는 공지를 제대로 하지 못했네요. 죄송합니다. 다시 확인해서 공지를 하도록 하겠습니다. 지적 감사합니다.

communityai8509 · Answer

안녕하세요, 인프런 AI 인턴입니다. tkwoo님이 질문해 주신 것처럼 calculateLateFee 메서드의 구현에서 지적한 부분은 코드의 불변성 또는 메서드의 반환되는 객체의 상태를 올바르게 갱신하는 부분에서 발생하는 문제로 보입니다. 우선 원래 코드에서는 this.lateFee.addPoint(point); 라는 메서드를 사용하고 있는데, 이는 lateFee 라는 객체에 포인트를 추가하려고 하지만 addPoint 메서드가 새로운 객체를 반환하고 그렇지 않으면 올바르게 사용되지 않을 가능성이 있습니다. 만약 addPoint 메서드가 불변 객체 패턴을 따르고 새로운 LateFee 객체를 반환한다면, this.setLateFee(this.lateFee.addPoint(point)); 처럼 후속적으로 반환된 객체를 현재 객체에 설정하는 방식이 더 적절합니다. 이와 연관된 논의들은 별도의 게시판에서도 활발히 다루어졌습니다. 아래 링크를 참고하시면 더욱 깊이 있는 이해를 도울 수 있을 것입니다: 보상트랜잭션 후 클라이언트 알림 방법 등 : 비동기 통신에서 상태 변경 및 오버듀 처리 관련 논의. VO, Entity 궁금한 부분이 있습니다. : Value Object와 Entity 패턴의 사용 및 상태 변경에 관한 심도 있는 논의. 각 링크에서 제공되는 상세한 설명과 토론을 통해 문제 해결에 도움이 될 수 있을 것입니다. 저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.