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

31강에서 BookService 클래스에 강의와 똑같이 코드를 작성하였는데 실행하면 UserServiceV2 오류가 발생합니다.

UserRepository의 Optional<User> 형식을 받지 못해서 생기는 오류인 것 같은데, 어떻게 수정해야 하나요?

error: incompatible types: Optional<User> cannot be converted to User

User user = userRepository.findByName(name);

^

• UserServiceV2.java

   

@Transactional
public void deleteUser(String name) {
    //SELECT * FROM user WHERE name = ?
    User user = userRepository.findByName(name);
    if (user == null) {
        throw new IllegalArgumentException();
    }
    userRepository.delete(user);
}

• BookService.java

@Transactional
public void loanBook(BookLoanRequest request){
    //1. 책 정보 가져오기
    Book book = bookRepository.findByName(request.getBookName()).orElseThrow(IllegalArgumentException::new);

    //2. 대출 기록 정보 확인하여 대출 중인지 확인
    //3. 대출 중이면 예외 발생
    if(userLoanHistoryRepository.existsByBookNameAndIsReturn(book.getName(), false)){
        throw new IllegalArgumentException("진작 대출되어 있는 책입니다.");
    }

    //4. 유저 정보 가져오기
    User user = userRepository.findByName(request.getUserName())
            .orElseThrow(IllegalArgumentException::new);

    //5. 유저 정보와 책 정보 기반 UserLoanHistory 저장
    userLoanHistoryRepository.save(new UserLoanHistory(user.getId(), book.getName()));
}

• UserRepository.java

  package com.group.libraryapp.domain.user;
  
  import org.springframework.data.jpa.repository.JpaRepository;
  
  import java.util.Optional;
  
  public interface UserRepository extends JpaRepository<User, Long> {
      Optional<User> findByName(String name);
  }

  ---

  우선 오류를 없애기 위해 UserRepository에서 Optional을 빼고 null처리를 하도록 수정하였는데, 실행 시 오류는 없지만 웹UI로 테스트하면 서버 내부 오류가 발생했다고 뜹니다. 어떻게 수정해야 제대로 처리되는지 모르겠습니다.

   

추가로 이렇게 수정하였을 때, 이후 코드를 작성할 때 Optional 형식이 아니어서 발생하는 다른 오류가 없는지도 궁금합니다.

• UserRepository.java

package com.group.libraryapp.domain.user;

import org.springframework.data.jpa.repository.JpaRepository;

import java.util.Optional;

public interface UserRepository extends JpaRepository<User, Long> {
    User findByName(String name);
}

• BookService.java

@Transactional
public void loanBook(BookLoanRequest request){
    //1. 책 정보 가져오기
    Book book = bookRepository.findByName(request.getBookName()).orElseThrow(IllegalArgumentException::new);

    //2. 대출 기록 정보 확인하여 대출 중인지 확인
    //3. 대출 중이면 예외 발생
    if(userLoanHistoryRepository.existsByBookNameAndIsReturn(book.getName(), false)){
        throw new IllegalArgumentException("진작 대출되어 있는 책입니다.");
    }

    //4. 유저 정보 가져오기
    User user = userRepository.findByName(request.getUserName());
    if(user == null){
        throw new IllegalArgumentException();
    }

    //5. 유저 정보와 책 정보 기반 UserLoanHistory 저장
    userLoanHistoryRepository.save(new UserLoanHistory(user.getId(), book.getName()));
}

• UserServiceV2.java

   

  @Transactional
  public void deleteUser(String name) {
      //SELECT * FROM user WHERE name = ?
      User user = userRepository.findByName(name);
      if (user == null) {
          throw new IllegalArgumentException();
      }
      userRepository.delete(user);
  }