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

참깨햄스터님의 프로필 이미지
참깨햄스터

작성한 질문수

스프링 입문 - 코드로 배우는 스프링 부트, 웹 MVC, DB 접근 기술

회원 서비스 테스트

java.lang.NullPointerException: Cannot invoke "java.util.Optional.ifPresent(java.util.function.Consumer)" because the return value of "..."is null

작성

·

88

0

[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예)
[질문 내용]
스프링 입문 - 코드로 배우는 스프링 부트, 웹 MVC, DB 접근 기술 강의 회원테스트 기술 부분을 학습하고있는데 테스트케이스 오류가 났습니다 어떻게 해결해야할지 몰라 질문글을 적었습니다

아래는 테스트 코드입니다

import org.junit.jupiter.api.Test;
import static org.assertj.core.api.Assertions.*;
import static org.junit.jupiter.api.Assertions.*;

class MemberServiceTest {
    MemberRepository memberRepository = new MemoryMemberRepository();
    MemberService memberService = new MemberService();

    //테스트는 과감하게 한글로 작성 가능
    //테스트할땐 given,when,then 틀 사용

    @Test
    void 회원가입() {
        //given
        Member member = new Member();
        member.setName("hello");
        //when
        Long saveId = memberService.join(member);

        //then
        Member findMember = memberService.findOne(saveId).get();
        org.assertj.core.api.Assertions.assertThat(member.getName()).isEqualTo(findMember.getName());
    }

    @Test
    void findMembers() {
    }

    @Test
    void findOne() {
    }
}

그리고 MemberService코드입니다

package hello.hello_spring.service;

import hello.hello_spring.domain.Member;
import hello.hello_spring.repository.MemberRepository;
import hello.hello_spring.repository.MemoryMemberRepository;

import java.util.List;
import java.util.Optional;

public class MemberService {
    private final MemberRepository memberRepository = new MemoryMemberRepository();

    /*회원가입*/
    public Long join(Member member){
        validateDuplicateMember(member); //중복회원검증
        memberRepository.save(member);
        return member.getId();
    }

    private void validateDuplicateMember(Member member){
        memberRepository.findByName(member.getName())
                .ifPresent(m -> {
                    throw new IllegalArgumentException("이미 존재하는 회원입니다");
                });
    }
    /*전체회원 조회*/
    public List<Member> findMembers() {
        return memberRepository.findAll();
    }

    public Optional<Member> findOne(Long memberId){
        return  memberRepository.findById(memberId);
    }
}

아래는 오류 메시지 입니다

java.lang.NullPointerException: Cannot invoke "java.util.Optional.ifPresent(java.util.function.Consumer)" because the return value of "hello.hello_spring.repository.MemberRepository.findByName(String)" is null

at hello.hello_spring.service.MemberService.validateDuplicateMember(MemberService.java:22)

at hello.hello_spring.service.MemberService.join(MemberService.java:15)

at hello.hello_spring.service.MemberServiceTest.회원가입(MemberServiceTest.java:24)

at java.base/java.lang.reflect.Method.invoke(Method.java:568)

at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

Process finished with exit code -1

 

어떻게 해결하면 좋을까요?

답변 2

0

안녕하세요. 참깨햄스터님, 공식 서포터즈 OMG입니다.

MemoryMemberRepository 클래스의 findByName()의 반환이 return null; 인지 확인해주세요.

 

감사합니다.

return store.values().stream()
        .filter(member -> member.getName().equals(name))
        .findAny();

로 수정하여 해결하였습니다 감사합니다

0

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

참깨햄스터님의 질문은 java.lang.NullPointerException에 관한 것으로, 테스트 케이스 실시 중 MemberRepository.findByName(String) 메서드 호출 결과가 null이 반환되어 ifPresent(java.util.function.Consumer) 메서드를 호출할 수 없는 문제에 대한 것으로 보입니다.

이러한 문제는 주로 MemberRepository의 구현체 내에서 findByName 메서드의 구현이 Optional 객체를 정확히 반환하지 않아서 발생합니다. 즉, findByName 메서드가 null 대신에 Optional.empty()를 반환하도록 보장해야 합니다.

문제의 핵심은 MemberServicejoin 메서드 내에서 validateDuplicateMember 메서드를 호출할 때, 해당 findByName 메서드가 null 값을 반환하지 않고, 항상 올바른 Optional 인스턴스(찾는 이름의 회원이 없을 경우 Optional.empty())를 반환해야 한다는 것입니다.

해당 문제를 해결하기 위한 유사한 케이스를 보여주는 질문 링크를 추천해드립니다:
- MemberServiceTest.java 에서 오류가 발생합니다. - 이 질문은 NullPointerException에 대한 문제를 다루고 있습니다. 해당 답변에서는 객체가 제대로 초기화되지 않아 발생한 문제의 해결 방안을 제시하고 있습니다.

참깨햄스터님의 경우, 테스트 코드에서 MemberService 객체를 직접 생성하는 대신 스프링 컨텍스트로부터 주입받거나 적절히 초기화하여 MemoryMemberRepository 인스턴스가 올바르게 MemberService에 설정되도록 해야 합니다. 또는, MemoryMemberRepository 구현체에서 findByName 메서드가 올바른 Optional 객체를 반환하도록 구현이 되었는지 확인하는 것이 좋습니다.

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

사용할 객체 선언을 아래와 같이 선언하였습니다

 

@BeforeEach // 각 테스트 실행 전에 실행됨 public void setUp() {     memberRepository = new MemoryMemberRepository();     memberService = new MemberService(memberRepository); // MemberService에 Repository 주입 

그런데 MemberService쪽에서 오류가나 MemberService에서 아래와 같이 코드를 고쳤습니다

 

public MemberService(MemberRepository memberRepository) {
    this.memberRepository = memberRepository;
}

그런데도 오류가 나 아래코드에서 final을 지웠는데도 오류가 납니다

private MemberRepository memberRepository = new MemoryMemberRepository();
참깨햄스터님의 프로필 이미지
참깨햄스터

작성한 질문수

질문하기