berryberrybin
commented
1 year ago 리팩토링 1
-
[x] (1) PostLike 취소, 등록 메소드가 분리되어 API를 각각 호출하는편이 좋은가?
- 이미 좋아요 클릭한 상태인지에 따라 좋아요 등록, 취소 한번에 처리
- 좋아요 이미 클릭되어 있을 경우 좋아요 취소 처리
- 좋아요 클릭되어 있지 않을 경우 좋아요 등록 처리
- 이미 좋아요 클릭한 상태인지에 따라 좋아요 등록, 취소 한번에 처리
-
[x] post 목록 조회 화면에서 좋아요 수를 위해 Post에 Like필드 추가를 통해 양방향 관계 설정이 베스트인가?
- 단방향 매핑일때는 PostLike(N) : Post(1)에서 기본적으로 PostLike(다)인 쪽에서 PK를 가지고 주인 역할을 하게됨
Post.PostLikes의 size를 가져오기 위해 양방향 매핑하게 되었음- 양방향 매핑 없이 좋아요수를 가져올 수 있도록 코드 수정
Domain
-
Post : PostLike 필드 삭제
@Getter @Entity @NoArgsConstructor(access = AccessLevel.PROTECTED) public class Post extends DateTimeEntity { @Id @GeneratedValue(generator = "UUID") @Column(name = "post_id", columnDefinition = "BINARY(16)") @GenericGenerator(name = "UUID", strategy = "org.hibernate.id.UUIDGenerator") private UUID id; private String title; private String content; @ManyToOne(fetch = FetchType.LAZY) @JoinColumn(name = "user_id") private User user; // 삭제 // @OneToMany(fetch = FetchType.LAZY, mappedBy = "post", cascade = CascadeType.ALL, orphanRemoval = true) // private List<PostLike> likes = new ArrayList<>(); @Builder public Post(User user, String title, String content) { this.user = user; this.title = title; this.content = content; this.viewCount = 0; } // 삭제 // public Integer getLikeCount() { return likes.size();} // public void addLike(PostLike postLike) { this.likes.add(postLike); } // public void cancelLike(PostLike like) { this.likes.remove(like); }
}
- PostLike : 기존과 동일
```java
@Getter
@Entity
@Table(uniqueConstraints = { @UniqueConstraint(name = "POST_LIKE_UNIQUE", columnNames = {"user_id", "post_id"})})
@NoArgsConstructor(access = AccessLevel.PROTECTED)
public class PostLike {
@Id
@GeneratedValue(generator = "UUID")
@Column(name = "post_like_id", columnDefinition = "BINARY(16)")
@GenericGenerator(name = "UUID", strategy = "org.hibernate.id.UUIDGenerator")
private UUID id;
@ManyToOne(fetch = FetchType.LAZY)
@JoinColumn(name = "user_id")
private User user;
@ManyToOne(fetch = FetchType.LAZY)
@JoinColumn(name = "post_id")
private Post post;
@Builder
public PostLike(Post post, User user) {
this.post = post;
this.user = user;
}
}
Mapper
-
PostInfoMapper
PostInfo of(Post post);➔PostInfo of(Post post, long likeCount);로 변경- Post에는 없는 likeCount를 PostInfo에 매핑하기 위해 파라미터 추가
@Mapper public interface PostInfoMapper { PostInfoMapper INSTANCE = Mappers.getMapper(PostInfoMapper.class);
@Mapping(target = "userId", source = "post.user.id") @Mapping(target = "postId", source = "post.id") PostInfo of(Post post, long likeCount); }
Service
-
PostCommandService
- postLike 등록, 삭제 관련 메서드 삭제
- savePostLike 1개의 메서드로 통일
- post.addLike 및 post.cancleLike를 통해 like를 수정하는 것이 아닌 postLikeRepository를 통해 postLike 수정
- mapper변경으로 파라미터 인자 2개
Post와likeCount필요하여 변경 - likeCount는
postLikeRepository.findPostLikeCount(postId);통해 가져오거나 게시글 처음 등록할때는 0으로 설정@Service @Transactional @RequiredArgsConstructor public class PostCommandService { private final PostRepository postRepository; private final UserRepository userRepository; private final PostLikeRepository postLikeRepository;
// postLike 등록, 삭제 관련 메서드를 1개의 메서드로 통일 및 내부 코드 리팩토링 // public void cancelLikePost(PostCommand.CancelLikePostCommand cancelLikePostCommand) {...} // public void likePost(PostCommand.LikePostCommand likePostCommand) { ... }
public void savePostLike(PostCommand.LikePostCommand likePostCommand){ UUID userId = likePostCommand.getUserId(); User user = userRepository.findById(userId) .orElseThrow(() -> new BusinessException(ErrorCode.NOT_FOUND, "user.byId", List.of(userId.toString())));
UUID postId = likePostCommand.getPostId(); Post post = postRepository.findById(postId) .orElseThrow(() -> new BusinessException(ErrorCode.NOT_FOUND, "post.byId", List.of(postId.toString()))); postLikeRepository.findByUserIdAndPostId(userId, postId) .ifPresentOrElse( postLike -> postLikeRepository.deleteById(postLike.getId()), // 존재하면 좋아요 취소 () -> postLikeRepository.save(new PostLike(post, user)) // 없으면 좋아요 생성 );} }
-
registerPost : 게시글 등록시 likecount는 0부터 시작
public PostInfo registerPost(PostCommand.RegisterPostCommand registerPostCommand) { UUID userId = registerPostCommand.getUserId(); User user = userRepository.findById(userId) .orElseThrow(() -> new BusinessException(ErrorCode.NOT_FOUND, "user.byId", List.of(userId.toString()))); Post post = Post.builder() .user(user) .title(registerPostCommand.getTitle()) .content(registerPostCommand.getContent()) .build(); // mapper변경으로 파라미터 인자 2개 `Post`와 `likeCount` 필요 //return PostInfoMapper.INSTANCE.of(postRepository.save(post)); return PostInfoMapper.INSTANCE.of(postRepository.save(post) , 0); }
- retrievePost : 단건 게시글 조회시 mapper변경으로 파라미터 2개
```java
public PostInfo retrievePost(PostQuery.RetrievePostByPostId retrievePostQuery) {
UUID postId = retrievePostQuery.getPostId();
Post post = postRepository.findById(postId)
.orElseThrow(() -> new BusinessException(ErrorCode.NOT_FOUND, "post.byId", List.of(postId.toString())));
// mapper 수정으로 인해 변경
// return PostInfoMapper.INSTANCE.of(post);
long postLikeCount = postLikeRepository.findPostLikeCount(postId);
return PostInfoMapper.INSTANCE.of(post, postLikeCount);
}
public Slice<SimplePostInfo> retrievePosts(PostQuery.RetrievePosts retrievePostsQuery) {
UUID boardId = retrievePostsQuery.getBoardId();
if (!boardRepository.existsById(boardId)) {
throw new BusinessException(ErrorCode.NOT_FOUND, "board.byId", List.of(boardId.toString()));
}
if (StringUtils.hasText(retrievePostsQuery.getTag())) {
return postRepository.findAllByBoardIdAndTag(retrievePostsQuery.getBoardId(),
retrievePostsQuery.getTag(),
retrievePostsQuery.getPageable())
// .map(SimplePostInfoMapper.INSTANCE::of);
.map(post -> SimplePostInfoMapper.INSTANCE.of(post, postLikeRepository.findPostLikeCount(post.getId())));
}
return postRepository.findAllByBoardId(boardId, retrievePostsQuery.getPageable())
// .map(SimplePostInfoMapper.INSTANCE::of);
.map(post -> SimplePostInfoMapper.INSTANCE.of(post, postLikeRepository.findPostLikeCount(post.getId())));
}
Repository
- postId를 통해 PostLike 개수 찾는 QueryDsl 코드 추가
- QueryDsl 사용을 위해 PostLikeRepositoryCustom과 PostLikeRepositoryCustomImpl 생성
- findPostLikeCount()를 통해
해당 post의 좋아요 수반환
public interface PostLikeRepository extends JpaRepository<PostLike, UUID>, PostLikeRepositoryCustom {
Optional<PostLike> findByUserIdAndPostId(UUID userId, UUID postId);
boolean existsByUserIdAndPostId(UUID userId, UUID postId);
}- PostLikeRepositoryCustom
- findPostLikeCount :
public interface PostLikeRepositoryCustom { long findPostLikeCount(UUID postId); }
- findPostLikeCount :
- PostLikeRepositoryCustomImpl
public class PostLikeRepositoryCustomImpl implements PostLikeRepositoryCustom {
private final JPAQueryFactory jpaQueryFactory;
public PostLikeRepositoryCustomImpl(EntityManager entityManager){
this.jpaQueryFactory = new JPAQueryFactory(entityManager);
}
@Override
public long findPostLikeCount(UUID postId){
return jpaQueryFactory
.select(postLike.count())
.from(postLike)
.where(postLike.post.id.eq(postId)).fetchOne();
}
}Controller
@RestController
@RequestMapping("/api/v1/users/{userId}")
@RequiredArgsConstructor
public class PostCommandController {
private final PostCommandService postCommandService;
// likePost와 cancleLikePost 1개로 통합
@PostMapping("/like/posts/{postId}")
public ResponseEntity<CommonResponse<?>> savePostLike(@PathVariable UUID userId,
@PathVariable UUID postId) {
PostCommand.LikePostCommand command = PostCommand.LikePostCommand.builder()
.userId(userId)
.postId(postId)
.build();
postCommandService.savePostLike(command);
return ResponseEntity
.status(HttpStatus.OK)
.body(CommonResponse.success("포스트 좋아요 변경 성공"));
}
}
기존 코드
List<PostLike> likes의 size를 구함Domain
Post (게시글)
}
Service
Mapper
Controller