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

jangso825님의 프로필 이미지
jangso825

작성한 질문수

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

회원 리포지토리 테스트 케이스 작성

findByName()부분에서 java.lang.NullPointException 에러가 해결되지 않습니다.

작성

·

329

0

안녕하십니까 강사님. 이렇게 유익한 강의를 만들어주셔서 감사합니다.

다름이 아니라 이 부분에 관한 다른 질문들을 보고 따라해봤지만 제 경우에는 해결되지 않아 도움을 부탁드립니다.

 

[MemoryMemberRepositoryTest] ->findByName에서 오류가 발생합니다.

package hello.hellospring1.repository;

import java.util.List;
import java.util.Optional;
import hello.hellospring1.domain.Member;
import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.Test;

import static org.assertj.core.api.Assertions.*;



import java.net.PortUnreachableException;

import static org.hamcrest.MatcherAssert.assertThat;

public class MemoryMemberRepositoryTest {

    MemoryMemberRepository repository = new MemoryMemberRepository();


    @AfterEach
    public void afterEach() {
        repository.clearStore();
    }

   
    @Test
    public void  findByName() {
        Member member1 = new Member();
        member1.setName("spring1");
        repository.save(member1);

        Member  member2= new Member(); //위에거 복사 & shift+f6 누르면 member 이름 한번에 바꿀 수 있음
        member2.setName("spring2"); //여기서 spring2는 자동으로 안바뀜 직접 바꾸기
        repository.save(member2);

        Member result = repository.findByName("spring1").get();

        Assertions.assertThat(result).isEqualTo(member1);
    }



}

 

 

[MemberRepository]

package hello.hellospring1.repository;

import hello.hellospring1.domain.Member;

import java.util.ArrayList;
import java.util.Map;
import java.util.HashMap;
import java.util.List;
import java.util.Optional;

public class MemoryMemberRepository implements MemberRepository {

    private static final Map<Long, Member> store = new HashMap<>();
    private static long sequence = 0L; 



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

    @Override
    public Optional<Member> findById(Long id) {
        return Optional.ofNullable(store.get(id));
    }

    @Override
    public Optional<Member> findByName(String name) {
        return store.values().stream()
                .filter(member -> member.getName().equals(name))
                .findAny();
    }

    @Override
    public List<Member> findAll() {
        return new ArrayList<>(store.values());
    }



    public void clearStore(){
        store.clear();
    }

    @Override
    public Optional<Member> findId(Long memberId) {
        return Optional.empty();
    }
}

[에러]

답변 3

2

setName 메서드 내 로직이 없습니다.

따라서 name을 가지지 않는 멤버가 store에 저장됩니다.

이 상태에서 spring1 이름을 가진 회원을 찾으려할 때 member의 name을 가져와 equals()를 실행할 때 NPE가 발생하게 됩니다.

setter 로직을 완성해주세요.

jangso825님의 프로필 이미지
jangso825
질문자

덕분에 문제를 해결하고 문제의 원인이 무엇인지 파악하는데 도움이 많이 되었습니다. 감사합니다

0

jangso825님의 프로필 이미지
jangso825
질문자

제가 올린 부분 중에 잘못 이해했거나 잘못된 부분이 있다면 알려주시길 바랍니다.

답변해주셔서 감사합니다.

[Member]

package hello.hellospring1.domain;

public class Member  {

    private Long id;
    private String name;

    public Long getId() {
        return id;
    }

    public void setId(Long id) {
        this.id = id;
    }

    public String getName(){
        return name;
    }


    public void setName(String spring) {
    }
}

[에러 로그]

 

0

안녕하세요. jangso825님, 공식 서포터즈 David입니다.

에러 로그 전체를 캡처해주세요.

Member 쪽 코드도 함께 올려주세요.

감사합니다.

jangso825님의 프로필 이미지
jangso825

작성한 질문수

질문하기