save()메소드의 리턴값에 대해서

학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

1. 강의 내용과 관련된 질문을 남겨주세요.
2. 인프런의 질문 게시판과 자주 하는 질문(링크)을 먼저 확인해주세요.
(자주 하는 질문 링크: https://bit.ly/3fX6ygx)
3. 질문 잘하기 메뉴얼(링크)을 먼저 읽어주세요.
(질문 잘하기 메뉴얼 링크: https://bit.ly/2UfeqCG)

질문 시에는 위 내용은 삭제하고 다음 내용을 남겨주세요.
=========================================
[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예/아니오)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예/아니오)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예/아니오)

[질문 내용]
안녕하세요! 강의 정말 잘 듣고 있습니다.

MemoryMemberRepository클래스의 save() 메소드에 대해서 궁금한게 있습니다.

@Override
public Member save(Member member) {
    member.setId(++sequence);
    store.put(member.getId(), member);
    return member;

}

여기서 리턴값에 멤버를 그대로 반환하는데요, 이러면 외부에서 member값을 변형할 수 있고 그 변형이 DB에 저장된 값에도 영향을 미치는 위험이 있지 않나 해서요.

return new Member(member); 이런식으로 하는게 더 나은 방법이 아닌가 싶어서 여쭤봅니다.

안녕하세요, 인프런 AI 인턴입니다. JH B님.

save 메서드에서 member 객체를 그대로 반환하는 것에 대한 고민은 매우 유용한 질문입니다. 여러분이 질문하신 대로, 반환된 객체를 외부에서 수정할 수 있게 되면 저장소에 예상치 못한 변화가 있을 수 있다는 우려가 있습니다.

그러나, 실제로 대다수의 구현에서는 save 메서드가 직접적으로 데이터베이스에 연결된 것이 아니고, 연습용이나 간단한 교육용 코드에서는 객체를 그대로 반환해도 큰 문제가 생기지 않을 수 있습니다. 이 방법은 구현의 간편함과 메서드 체이닝 등의 이점을 제공합니다.

하지만, 실전적인 코드에서는 외부로의 직접 노출을 방지하기 위해 immutable 객체를 반환하거나, 복사본을 반환하는 방법이 사용될 수 있습니다. 즉, return new Member(member); 같은 방식으로 복사본을 반환해 원본 객체가 외부에서 변경되지 않도록 하는 것입니다.

관련하여 이전에도 비슷한 질문과 논의가 있었습니다. 참고하실 수 있도록 관련 링크를 공유드립니다:

이해안되는부분이 후반부에 나오겠지했는데 답을 알수없어 질문드립니다ㅜ
- 이 링크에서는 서비스 계층에서 save 메서드의 반환값을 사용하는 것에 대해 논의합니다.
안녕하세요 너무 기초적인 질문같지만 save메서드에서
- 여기는 save 메서드가 저장소에 어떤 데이터를 저장하는지를 설명합니다.

이러한 실천들은 종종 복잡성과 장단점을 모두 고려해야 한다는 것을 이해하는 것이 중요합니다. 객체의 설계, 프로젝트 요구사항, 팀의 코드 스타일에 따라 적절한 방법을 선택하는 것이 좋습니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.

인프런 커뮤니티 질문&답변