💎 네이밍 변경 : findAllRelated()
🤔 내 게시글도 함께 조회하는 부분은 아직 ,, 해결해야합니다
- 6/23 해결 !
- member 쪽에 findAllFollowingIncludeMe() 구현 요청 드려서 해결했다 !
- post 쪽에서 해결할 수 있을까 얘기를 나눠봤는데 쿼리가 무조건 2번 나갈 수 밖에 없어서
- me 를 찾는 memberRepo.findById() , memberRepo.findAllFollowing() 이렇게 두번 !
- member 쪽에 요청을 했다
🐣 코드개선
- 기존의 코드는 totalPostLike1 이고, 변경한 코드는 totalPostLike2 입니다.
- count메서드가 Long을 반환해서 count가 0 일 경우 null을 반환하는 문제가 있었습니다.
- 삼항 연산자를 사용하긴 했는데 더 좋은 개선 방법이 있다면 의견 부탁드립니다
6/22 해결 !
- 멘토님께서 getOrDefault 메서드를 알려주셔서 null일 때도 잘 대응할 수 있었습니다.
- getOrDefault는 HashMap get을 했을 때 값이 없을 경우 defaultValue 넣어줍니다.
- computIfPresent는 값을 넣을 때 키가 충돌이 날 경우 새로운 값을 넣어줍니다.
🐢 개선 이전의 코드
- 개선 이전의 쿼리
🍗 개선한 코드
- 기존에 스트림을 탈 때마다 쿼리가 날아갔었는데 Stream을 활용해서 쿼리를 한 번만 날릴 수 있게 되었습니다.
- 여러분들도 스트림의 Collectiors.groupingBy() 를 잘 활용해서 개선해봅시다 !
- 멘토님 say : 여러분들 스트림 공부를 잘 하고 있는 줄 알았는데 아니였군여? 앞으로 스트림을 잘 활용한다고 하셨으니 개선된 코드를 기대하겠습니다 ^0^
- 멘토님 짱 !