31강 유저 정보 가져오기에서 질문 있습니다
169
작성한 질문수 2
유저 정보를 가져올 때 orElseThrow를 사용하면 userRepository에 입력되어 있는 findByName이 Optional로 선언되어야 한다고 문구가 뜹니다!
근데 Optional<User>로 바꾸면 UserServiceV2에
@Transactional
public void deleteUser(String name){
User user = userRepository.findByName(name);
if(user == null){
throw new IllegalArgumentException();
}
userRepository.delete(user);
}user의 타입을 User로 할 수 없다고 떠서 Optional<User>로 바꾸면 userRepository.delete(user)가 파라미터가 적절하지 않다고 뜹니다ㅜㅜ
일단, 구현은 하고 싶어서 userRepository에 findByName은 User 타입으로 정의하고
// 4. 유저 정보를 가져온다
if(userRepository.findByName(request.getUserName()) == null){
throw new IllegalArgumentException();
}
else{
User user = userRepository.findByName(request.getUserName());
// 5. 유저 정보와 책 정보를 기반으로 UserLoanHistory를 저장
userLoanHistoryRepository.save(new UserLoanHistory(user.getId(),book.getName()));
}bookService를 이렇게 바꾸었는데 상관없을까요?
어떤식으로 해결하는게 맞을까요...
답변 2
0
안녕하세요! bs goose님! 🙂 좋은 질문 주셔서 감사합니다! 🙏
제가 질문 주신 부분에 대해 이해한 내용을 말씀드리면
Optional<User>와User중 하나를 골라 반환해야 하는 상황에서User타입을 반환하도록 하고BookService로직 중 일부를 변경했다
고 해주셨는데요!
우선 둘 중 어떤 타입을 선택하시건 큰 문제는 없습니다! 🙂 단지, User 가 null인 경우를 잘 핸들링 해주시면 되죠! Optional 타입 자체의 등장 배경이, "혹시나 개발자가 null 체크를 잊어버려 NullPointerException 같은 예외가 발생하는 것을 방지 해주기" 위해서니까요~
그리고 작성해주신 코드도 물론 동작하는 코드이지만 (null 예외 처리도 해주셨고요!) 조금 다듬으면 좋을 것 같습니다.
예를 들어 현재는 userRepository.findByName() 을 두 번 호출하기 때문에 불필요한 쿼리가 한 번 나가게 되는데요! 다음과 같이 코드를 바꿀 수 있을 것 같아요.
User user = userRepository.findByName(request.getName());
if (user == null) {
throw new IllegalArgumentException();
}
userLoanHistoryRepository.save(new UserLoanHistory(user.getId(),book.getName()));
답변이 도움이 되었으면 좋겠습니다. 감사합니다! 🙏
패키지 구분에 대해 궁금한게 있습니다
0
34
2
리액트 관련 질문이 있습니다.
0
69
2
스프링부트 버전
0
87
2
7강 강의를 들으려고 했는데 오류가 나서 서버가 안 켜지는거 같아요.
0
66
2
33강. UserLoanHistory의 관계성에 대한 질문
1
58
2
Java JDK 버전 문의의 건
0
144
2
ec2 에서 Linux버전이 달라져서 설치가 안되는것 같은데 자료 최신화좀 해주세요.
0
95
3
h2 console 접속했을 테이블 질문
1
70
1
ec2 서버에서 스프링 실행도 되고 인바운드 설정까지 했는데 index.html 안됨
0
83
2
15강. updateUser() 질문
0
57
2
깃허브 질
0
91
2
여기까지 다 끝냈다고 하셨는데
0
83
2
왜안될까요
0
73
2
MySQL 창이안ㄴ뜹니다
0
59
2
포스트맨
0
55
1
spring 개념적인 질문
0
76
2
인텔리제이 샘플코드 실행 안됨 오류
0
150
2
aws 배포할때 .env 파일에 저장한 환경변수에 관하여 여쭤볼게 있습니다
0
90
1
마이그레이션 오류입니다.
0
179
3
Whitelabel Error Page 오류가 났습니다.
0
182
2
안녕하십니까! 오류가 났습니다.. 도와주세요 ㅜㅜ
1
101
3
궁금한게 있습니다.
0
64
2
DTO 관련
0
74
2
궁금한게 있습니다!
0
73
2





