[#145] 좋아요 추가, 삭제 api 구현 #160
Conversation
Jacoco Test Coverage Reports
|
| * </Pre> | ||
| * @param bookshelfId | ||
| * @param userPrincipal | ||
| * @return status : ok | ||
| */ | ||
| @PostMapping() | ||
| @PreAuthorize(value = "hasAnyRole('ROLE_ADMIN', 'ROLE_USER')") | ||
| public ResponseEntity<Void> createLike(@PathVariable Long bookshelfId, | ||
| @AuthenticationPrincipal UserPrincipal userPrincipal) { | ||
| bookshelfLikeService.createBookshelfLike(userPrincipal.getUserId(), bookshelfId); | ||
| return ResponseEntity.ok().build(); | ||
| } | ||
|
|
||
| /*** | ||
| * <Pre> | ||
| * 책장 좋아요 해제 | ||
| * </Pre> | ||
| * @param bookshelfId | ||
| * @param userPrincipal | ||
| * @return status : ok | ||
| */ | ||
| @DeleteMapping() | ||
| @PreAuthorize(value = "hasAnyRole('ROLE_ADMIN', 'ROLE_USER')") | ||
| public ResponseEntity<Void> deleteLike(@PathVariable Long bookshelfId, | ||
| @AuthenticationPrincipal UserPrincipal userPrincipal) { | ||
| bookshelfLikeService.deleteBookshelfLike(userPrincipal.getUserId(), bookshelfId); | ||
| return ResponseEntity.ok().build(); | ||
| } |
There was a problem hiding this comment.
만약에, 집 컴퓨터와 폰으로 좋아요를 안 누른 상태의 같은 페이지를 열고 집 컴퓨터로만 좋아요를 눌렀다고 가정해볼게요.
해당 상태로 폰을 들고 밖으로 나가서 페이지를 본다면 해당 책장에 좋아요가 안눌려있겠죠?
이 때에 다시 좋아요를 누른다면 서버의 좋아요 상태는 어떻게 바뀌어야 할까요?
클라이언트는 무엇을 보고 어떤 메소드를 선택해서 요청을 보내야 할까요?
There was a problem hiding this comment.
집 컴퓨터와 폰으로 좋아요를 안 누른 상태의 같은 페이지를 열고 집 컴퓨터로만 좋아요를 눌렀다고 가정해볼게요.
해당 상태로 폰을 들고 밖으로 나가서 페이지를 본다면 해당 책장에 좋아요가 안눌려있겠죠?
이부분은 무슨 말씀일까요?
There was a problem hiding this comment.
같은 페이지를 다른 브라우저에서 보고 있을 때,
- 집브라우저는 좋아요가 눌린상태
- 모바일 브라우저는 좋아요가 안눌린상태,
서로 다른 상태를 가지고있을 때 모바일 브라우저로 다시 좋아요를 누른다면
프론트엔드와 백엔드는 어떤 액션을 취해야 하는지 생각을 여쭤본겁니다!
| @Override | ||
| public String getMessage() { | ||
| return super.getMessage(); | ||
| } | ||
|
|
||
| @Override | ||
| public ErrorCode getErrorCode() { | ||
| return ALREADY_EXISTS_BOOKSHELF_LIKE; | ||
| } |
|
|
||
| boolean existsByBookshelfIdAndUserId(Long bookshelfId, Long userId); |
There was a problem hiding this comment.
existsLike? 비슷한 메소드 네임으로도 줄일 수 있을것 같아요
| @JsonManagedReference | ||
| @OneToMany(mappedBy = "bookshelf", cascade = CascadeType.PERSIST, orphanRemoval = true) | ||
| private final Set<BookshelfLike> bookshelfLikes = new HashSet<>(); |
There was a problem hiding this comment.
혹시, 좋아요의 개수는 매번 카운트 해서 보여주기로 했나요?
There was a problem hiding this comment.
아직 이야기된것은 없으나 그럴거라 예상합니다. 책장 상세 조회 응답에 추가할 예정입니다.
There was a problem hiding this comment.
혹시 그렇다면
- 매번 count 쿼리를 사용할것인지
- 좋아요 수를 저장할 컬럼을 추가할것인지
고민해보고 말씀해주세요!
|
|
||
| @Override | ||
| public void createBookshelfLike(Long userId, Long bookshelfId) { | ||
| User user = userService.getById(userId); | ||
| Bookshelf bookshelf = bookshelfService.getById(bookshelfId); | ||
| if (bookshelfLikeRepository.existsByBookshelfIdAndUserId(bookshelfId, userId)) { | ||
| throw new AlreadyExistsBookshelfLikeException(bookshelf.getId()); | ||
| } | ||
| bookshelfLikeRepository.save(BookshelfLike.create(user, bookshelf)); | ||
| } |
There was a problem hiding this comment.
혹시.. 이미 좋아요가 눌린 상태에서 다시 요청 된다면 예외일까요??
There was a problem hiding this comment.
일단 그렇게 구현하였습니다. 프론트와 상의후에 덮어쓰기 형식으로 바꿀까 고민중입니다~
There was a problem hiding this comment.
동시성 문제는 추후에 추가하겠습니다!
| @@ -2,6 +2,8 @@ drop table if exists book_comments cascade; | |||
|
|
|||
| drop table if exists bookshelf_item cascade; | |||
|
|
|||
| drop table if exists bookshelf_likes cascade; | |||
| @ExtendWith(MockitoExtension.class) | ||
| class DefaultBookshelfLikeServiceTest { |
There was a problem hiding this comment.
좋아요처럼 동시성 문제가 발생할 수 있는 로직은 통합테스트로 직접 DB에서 테스트해보는건 어떨까요?
|
PR 포인트에 남겨둔것과 같이 좋아요 재요청시에 인스타그램처럼 덮어쓰기를 할까 고민중입니다! |
🍀 목적
좋아요 추가, 삭제 api 구현
❓ pr 포인트
#145