작성
·
16
0
public class OrderServiceImpl implements OrderService {
MemberRepository memberRepository;
private final DiscountPolicy discountPolicy;
public OrderServiceImpl(MemberRepository memberRepository) {
this.memberRepository = memberRepository;
discountPolicy = new FixDiscountPolicy();
}
private final MemberService memberService=new MemberServiceImpl(memberRepository);
@Override
public Order createOrder(Long memberId, String itemName, int itemPrice) {
Member falsemember = memberService.findMember(memberId);
Member member = memberRepository.findById(memberId);
int discount = discountPolicy.discount(member,itemPrice);
return new Order(memberId,itemName,itemPrice,discount);
}
}
2회독이라서, 15강을 들으면서 자동주입관련해서 생각이 나서, 강의를 들으면서 자체적으로 작성해봤습니다.
MemberRepository를 주입해서 의존성 문제를 해결하려고 여러방면으로 고민하던중
왜 falseMember를 구하는 로직 중 MemberService 부분에서 계속 널 포인터익셉션이 터지는지 이해가 가지않아 질문드립니다.
MeberServie의 구현체에도 레포지토리를 주입받아서 널포인터 익셉션이 안터지는 것이 정상이라고 생각되는데 원인 말씀해주시면 감사드립니다.
답변 1
0
안녕하세요, 인프런 AI 인턴입니다. cowldnd123님.
질문하신 내용을 살펴보면 MemberService
객체가 생성될 때 memberRepository
가 아직 제대로 초기화되지 않은 상태라서 NullPointerException
이 발생하고 있는 것으로 보입니다. 스프링 컨텍스트에서 빈으로 관리되는 객체가 아닌 경우, 생성자나 필드에서 초기화 순서가 문제가 될 수 있습니다.
OrderServiceImpl
클래스는 memberRepository
를 생성자 파라미터로 받고 있으며, memberService
는 바로 클래스 필드의 초기화 블록에서 memberRepository
를 이용하여 초기화됩니다.memberRepository
가 아직 전혀 초기화되지 않은 상태에서 OrderServiceImpl
의 인스턴스가 초기화되기 때문에, NullPointerException
이 발생합니다.memberService
초기화 시점 조정: memberService
초기화를 생성자 내부나 메서드에서 처리하여 memberRepository
가 초기화된 이후에 실행되도록 바꿔보세요.예를 들면:
public OrderServiceImpl(MemberRepository memberRepository) {
this.memberRepository = memberRepository;
this.discountPolicy = new FixDiscountPolicy();
this.memberService = new MemberServiceImpl(memberRepository); // 생성자에서 초기화
}
이러면 생성자 주입 방식으로 인해 memberRepository
가 먼저 할당된 후 memberService
가 정상적으로 초기화됩니다.
@Autowired
를 사용하여 의존성 관리에 대한 책임을 스프링 컨텍스트로 넘깁니다.추가적으로 질문자님의 이해를 돕고자 같은 주제에 대한 질문과 해결책의 예를 제공하는 유사한 문제를 다룬 이 게시글을 추천드립니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.