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

안녕하세요, 작성자 님. 공식 서포터즈 y2gcoder 입니다.

일단 원인은 알고 계신 것처럼 Optional 관련 문제가 맞습니다. 그래서 보이는 부분만 보면 SessionAttribute에 넣을 때 optional 객체에서 값을 꺼내준 다음 넣으셔야 할 것 같습니다.

그리고 제 개인적으로는 2가지 중 마음에 드시는 쪽으로 수정하시는 것을 추천드립니다.

1. MemberRepository에서 JpaRepository를 상속받고 있습니다. JpaRepository를 상속받으면 Spring Data JPA에서 자동적으로 해주는 일이 많은 대신 그만큼 제약이 있습니다. 특히 findBy~ 문들은 거의 규칙대로 작성해주셔야 합니다. findById로 해주셔야 하고, 무엇보다 findById()가 이미 있습니다.
   이를 임의로 수정하기는 것은 손해가 많습니다. JpaRepository를 상속받아서 한다면 저는 MemberRepository를 MemberJpaRepository로 이름을 변경한 후, 이에 의존하는 MemberRepository를 둘 것 같습니다.
   MemberRepository - MemberJpaRepository(기존의 MemberRepository)
   그리고나서 MemberRepository에 원하시는대로 Member findById(Long id)를 만든 뒤, 해당 메서드에서는 MemberJpaRepository 가 자동으로 만들어주는 Optional<Member> findById(Long id)로 Optional 객체를 받아온 다음, Member가 있는지 확인하고 꺼내서 리턴해주는 식으로 해줄 수 있을 것 같습니다.

   @Repository
   public class MemberRepository {
       private final MemberJpaRepository memberJpaRepository;
   
       public MemberRepository(MemberJpaRepository memberJpaRepository) {
           this.memberJpaRepository = memberJpaRepository;
       }
       
       //...생략
       
       public Member findById(Long id) {
           return memberJpaRepository.findById(id).orElseThrow(Member엔티티찾지못했을때예외);
       }
   }

   //기존 MemberRepository
   public interface MemberJpaRepository extends JpaRepository<Member, Long> {
       Optional<Member> findByMemberId(String memberId);
   
       //기존 findById 삭제
   }

    

2. 다른 방식으로는 해당 링크처럼 MemberRepository에서 JpaRepository가 아닌 Repository로 상속받는 것입니다. 이렇게 하면 쿼리 메서드 형식만 지켜주면 자기가 원하는대로 메서드를 만드실 수 있습니다. Member findById(Long id) 로 만드실 수 있습니다. 대신 이 때는 save나 기존에 자동으로 만들어주던 것들도 임의로 만들어서 사용하셔야 합니다.

   public interface MemberRepository extends Repository<Member, Long> {
       Member findById(Long id);
       Member save(Member member);
       //...
   }

감사합니다.

It seems return type of the method findByid(Long id) should be Optional(Member).