작성
·
385
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 로직을 완성해주세요.
0

제가 올린 부분 중에 잘못 이해했거나 잘못된 부분이 있다면 알려주시길 바랍니다.
답변해주셔서 감사합니다.
[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) {
    }
}[에러 로그]


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