작성
·
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 로직을 완성해주세요.
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) {
}
}
[에러 로그]
덕분에 문제를 해결하고 문제의 원인이 무엇인지 파악하는데 도움이 많이 되었습니다. 감사합니다