feat: 리뷰 작성 시 필요한 태그 조회

oliviarla · oliviarla · commit 6f8603cf215b · 2024-01-04T14:20:45.000+09:00
diff --git a/perfume-api/src/docs/asciidoc/index.adoc b/perfume-api/src/docs/asciidoc/index.adoc
@@ -347,6 +347,14 @@ include::{snippets}/get-reviews/http-request.adoc[]
 include::{snippets}/get-reviews/http-response.adoc[]
 include::{snippets}/get-reviews/response-fields.adoc[]
 
+== 리뷰 작성을 위한 태그 조회
+=== 요청
+include::{snippets}/get-all-tags/http-request.adoc[]
+
+=== 응답
+include::{snippets}/get-all-tags/http-response.adoc[]
+include::{snippets}/get-all-tags/response-fields.adoc[]
+
 == 리뷰 생성
 === 요청
 include::{snippets}/create-review/http-request.adoc[]
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/in/http/ReviewController.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/in/http/ReviewController.java
@@ -15,6 +15,7 @@
 import io.perfume.api.review.adapter.in.http.dto.GetReviewsRequestDto;
 import io.perfume.api.review.adapter.in.http.dto.GetReviewsResponseDto;
 import io.perfume.api.review.adapter.in.http.dto.ReviewLikeResponseDto;
+import io.perfume.api.review.adapter.in.http.dto.TagResponseDto;
 import io.perfume.api.review.application.facade.ReviewDetailFacadeService;
 import io.perfume.api.review.application.in.comment.CreateReviewCommentUseCase;
 import io.perfume.api.review.application.in.comment.DeleteReviewCommentUseCase;
@@ -23,7 +24,9 @@
 import io.perfume.api.review.application.in.review.CreateReviewUseCase;
 import io.perfume.api.review.application.in.review.DeleteReviewUseCase;
 import io.perfume.api.review.application.in.review.GetReviewInViewUseCase;
+import io.perfume.api.review.application.in.tag.GetTagUseCase;
 import java.time.LocalDateTime;
+import java.util.List;
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -44,6 +47,7 @@
 @RequiredArgsConstructor
 public class ReviewController {
 
+  private final GetTagUseCase getTagUseCase;
   private final CreateReviewUseCase createReviewUseCase;
   private final DeleteReviewUseCase deleteReviewUseCase;
   private final ReviewDetailFacadeService reviewDetailFacadeService;
@@ -70,6 +74,11 @@ public ResponseEntity<CustomPage<GetReviewsResponseDto>> getReviews(
     return ResponseEntity.ok(new CustomPage<>(responseItems, results));
   }
 
+  @GetMapping("/tags")
+  public List<TagResponseDto> getAllTags() {
+    return getTagUseCase.getAll().stream().map(TagResponseDto::from).toList();
+  }
+
   @PreAuthorize("isAuthenticated()")
   @PostMapping
   public CreateReviewResponseDto createReview(
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/in/http/dto/TagResponseDto.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/in/http/dto/TagResponseDto.java
@@ -0,0 +1,9 @@
+package io.perfume.api.review.adapter.in.http.dto;
+
+import io.perfume.api.review.application.in.dto.TagResult;
+
+public record TagResponseDto(Long id, String name) {
+  public static TagResponseDto from(TagResult tagResult) {
+    return new TagResponseDto(tagResult.id(), tagResult.name());
+  }
+}
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagEntity.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagEntity.java
@@ -1,4 +1,4 @@
-package io.perfume.api.review.adapter.out.persistence.repository.tag;
+package io.perfume.api.review.adapter.out.persistence.repository.reviewtag;
 
 import io.perfume.api.base.BaseTimeEntity;
 import jakarta.persistence.EmbeddedId;
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagId.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagId.java
@@ -1,4 +1,4 @@
-package io.perfume.api.review.adapter.out.persistence.repository.tag;
+package io.perfume.api.review.adapter.out.persistence.repository.reviewtag;
 
 import jakarta.persistence.Column;
 import jakarta.persistence.Embeddable;
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagJpaRepository.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagJpaRepository.java
@@ -1,4 +1,4 @@
-package io.perfume.api.review.adapter.out.persistence.repository.tag;
+package io.perfume.api.review.adapter.out.persistence.repository.reviewtag;
 
 import org.springframework.data.repository.CrudRepository;
 
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagMapper.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagMapper.java
@@ -1,4 +1,4 @@
-package io.perfume.api.review.adapter.out.persistence.repository.tag;
+package io.perfume.api.review.adapter.out.persistence.repository.reviewtag;
 
 import io.perfume.api.review.domain.ReviewTag;
 import jakarta.validation.constraints.NotNull;
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagPersistenceAdapter.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagPersistenceAdapter.java
@@ -1,19 +1,19 @@
-package io.perfume.api.review.adapter.out.persistence.repository.tag;
+package io.perfume.api.review.adapter.out.persistence.repository.reviewtag;
 
 import io.perfume.api.base.PersistenceAdapter;
-import io.perfume.api.review.application.out.tag.TagRepository;
+import io.perfume.api.review.application.out.tag.ReviewTagRepository;
 import io.perfume.api.review.domain.ReviewTag;
 import java.util.List;
 import java.util.stream.StreamSupport;
 
 @PersistenceAdapter
-public class TagPersistenceAdapter implements TagRepository {
+public class ReviewTagPersistenceAdapter implements ReviewTagRepository {
 
   private final ReviewTagJpaRepository reviewTagRepository;
 
   private final ReviewTagMapper reviewTagMapper;
 
-  public TagPersistenceAdapter(
+  public ReviewTagPersistenceAdapter(
       ReviewTagJpaRepository reviewTagRepository, ReviewTagMapper reviewTagMapper) {
     this.reviewTagRepository = reviewTagRepository;
     this.reviewTagMapper = reviewTagMapper;
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagQueryPersistenceAdapter.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/reviewtag/ReviewTagQueryPersistenceAdapter.java
@@ -0,0 +1,41 @@
+package io.perfume.api.review.adapter.out.persistence.repository.reviewtag;
+
+import static io.perfume.api.review.adapter.out.persistence.repository.reviewtag.QReviewTagEntity.reviewTagEntity;
+
+import com.querydsl.jpa.impl.JPAQueryFactory;
+import io.perfume.api.base.PersistenceAdapter;
+import io.perfume.api.review.application.out.tag.ReviewTagQueryRepository;
+import io.perfume.api.review.domain.ReviewTag;
+import java.util.List;
+import lombok.RequiredArgsConstructor;
+
+@PersistenceAdapter
+@RequiredArgsConstructor
+public class ReviewTagQueryPersistenceAdapter implements ReviewTagQueryRepository {
+
+  private final ReviewTagMapper reviewTagMapper;
+
+  private final JPAQueryFactory jpaQueryFactory;
+
+  @Override
+  public List<ReviewTag> findReviewTags(Long reviewId) {
+    return jpaQueryFactory
+        .selectFrom(reviewTagEntity)
+        .where(reviewTagEntity.id.reviewId.eq(reviewId))
+        .fetch()
+        .stream()
+        .map(reviewTagMapper::toDomain)
+        .toList();
+  }
+
+  @Override
+  public List<ReviewTag> findReviewsTags(List<Long> reviewIds) {
+    return jpaQueryFactory
+        .selectFrom(reviewTagEntity)
+        .where(reviewTagEntity.id.reviewId.in(reviewIds))
+        .fetch()
+        .stream()
+        .map(reviewTagMapper::toDomain)
+        .toList();
+  }
+}
diff --git a/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/tag/TagQueryPersistenceAdapter.java b/perfume-api/src/main/java/io/perfume/api/review/adapter/out/persistence/repository/tag/TagQueryPersistenceAdapter.java
@@ -1,29 +1,24 @@
 package io.perfume.api.review.adapter.out.persistence.repository.tag;
 
-import static io.perfume.api.review.adapter.out.persistence.repository.tag.QReviewTagEntity.reviewTagEntity;
 import static io.perfume.api.review.adapter.out.persistence.repository.tag.QTagEntity.tagEntity;
 
 import com.querydsl.jpa.impl.JPAQueryFactory;
 import io.perfume.api.base.PersistenceAdapter;
 import io.perfume.api.review.application.out.tag.TagQueryRepository;
-import io.perfume.api.review.domain.ReviewTag;
 import io.perfume.api.review.domain.Tag;
 import java.util.List;
+import lombok.RequiredArgsConstructor;
 
+@RequiredArgsConstructor
 @PersistenceAdapter
 public class TagQueryPersistenceAdapter implements TagQueryRepository {
 
-  private final TagMapper tagMapper;
-
-  private final ReviewTagMapper reviewTagMapper;
-
   private final JPAQueryFactory jpaQueryFactory;
+  private final TagMapper tagMapper;
 
-  public TagQueryPersistenceAdapter(
-      TagMapper tagMapper, ReviewTagMapper reviewTagMapper, JPAQueryFactory jpaQueryFactory) {
-    this.tagMapper = tagMapper;
-    this.jpaQueryFactory = jpaQueryFactory;
-    this.reviewTagMapper = reviewTagMapper;
+  @Override
+  public List<Tag> findAll() {
+    return jpaQueryFactory.selectFrom(tagEntity).fetch().stream().map(tagMapper::toDomain).toList();
   }
 
   @Override
@@ -32,26 +27,4 @@ public List<Tag> findByIds(List<Long> ids) {
         .map(tagMapper::toDomain)
         .toList();
   }
-
-  @Override
-  public List<ReviewTag> findReviewTags(Long reviewId) {
-    return jpaQueryFactory
-        .selectFrom(reviewTagEntity)
-        .where(reviewTagEntity.id.reviewId.eq(reviewId))
-        .fetch()
-        .stream()
-        .map(reviewTagMapper::toDomain)
-        .toList();
-  }
-
-  @Override
-  public List<ReviewTag> findReviewsTags(List<Long> reviewIds) {
-    return jpaQueryFactory
-        .selectFrom(reviewTagEntity)
-        .where(reviewTagEntity.id.reviewId.in(reviewIds))
-        .fetch()
-        .stream()
-        .map(reviewTagMapper::toDomain)
-        .toList();
-  }
 }
diff --git a/perfume-api/src/main/java/io/perfume/api/review/application/facade/ReviewDetailFacadeService.java b/perfume-api/src/main/java/io/perfume/api/review/application/facade/ReviewDetailFacadeService.java
@@ -231,7 +231,7 @@ public long toggleLikeReview(long userId, long reviewId, LocalDateTime now) {
   @Override
   public ReviewResult create(Long authorId, CreateReviewCommand command) {
     final ReviewResult reviewDetailResult = reviewService.create(authorId, command);
-    reviewTagService.addTags(reviewDetailResult.id(), command.keywords());
+    reviewTagService.addReviewTags(reviewDetailResult.id(), command.keywords());
     reviewThumbnailService.addThumbnails(
         reviewDetailResult.id(), command.thumbnailIds(), command.now());
     return reviewDetailResult;
diff --git a/perfume-api/src/main/java/io/perfume/api/review/application/in/dto/TagResult.java b/perfume-api/src/main/java/io/perfume/api/review/application/in/dto/TagResult.java
@@ -0,0 +1,9 @@
+package io.perfume.api.review.application.in.dto;
+
+import io.perfume.api.review.domain.Tag;
+
+public record TagResult(Long id, String name) {
+  public static TagResult from(Tag tag) {
+    return new TagResult(tag.getId(), tag.getName());
+  }
+}
diff --git a/perfume-api/src/main/java/io/perfume/api/review/application/in/tag/GetTagUseCase.java b/perfume-api/src/main/java/io/perfume/api/review/application/in/tag/GetTagUseCase.java
@@ -0,0 +1,8 @@
+package io.perfume.api.review.application.in.tag;
+
+import io.perfume.api.review.application.in.dto.TagResult;
+import java.util.List;
+
+public interface GetTagUseCase {
+  List<TagResult> getAll();
+}
diff --git a/perfume-api/src/main/java/io/perfume/api/review/application/out/tag/ReviewTagQueryRepository.java b/perfume-api/src/main/java/io/perfume/api/review/application/out/tag/ReviewTagQueryRepository.java
@@ -0,0 +1,11 @@
+package io.perfume.api.review.application.out.tag;
+
+import io.perfume.api.review.domain.ReviewTag;
+import java.util.List;
+
+public interface ReviewTagQueryRepository {
+
+  List<ReviewTag> findReviewTags(Long reviewId);
+
+  List<ReviewTag> findReviewsTags(List<Long> reviewIds);
+}
diff --git a/perfume-api/src/main/java/io/perfume/api/review/application/out/tag/ReviewTagRepository.java b/perfume-api/src/main/java/io/perfume/api/review/application/out/tag/ReviewTagRepository.java
@@ -3,7 +3,7 @@
 import io.perfume.api.review.domain.ReviewTag;
 import java.util.List;
 
-public interface TagRepository {
+public interface ReviewTagRepository {
 
   ReviewTag save(ReviewTag tags);
 
diff --git a/perfume-api/src/main/java/io/perfume/api/review/application/out/tag/TagQueryRepository.java b/perfume-api/src/main/java/io/perfume/api/review/application/out/tag/TagQueryRepository.java
@@ -1,14 +1,10 @@
 package io.perfume.api.review.application.out.tag;
 
-import io.perfume.api.review.domain.ReviewTag;
 import io.perfume.api.review.domain.Tag;
 import java.util.List;
 
 public interface TagQueryRepository {
+  List<Tag> findAll();
 
   List<Tag> findByIds(List<Long> ids);
-
-  List<ReviewTag> findReviewTags(Long reviewId);
-
-  List<ReviewTag> findReviewsTags(List<Long> reviewIds);
 }
diff --git a/perfume-api/src/main/java/io/perfume/api/review/application/service/GetTagService.java b/perfume-api/src/main/java/io/perfume/api/review/application/service/GetTagService.java
@@ -0,0 +1,20 @@
+package io.perfume.api.review.application.service;
+
+import io.perfume.api.review.adapter.out.persistence.repository.tag.TagQueryPersistenceAdapter;
+import io.perfume.api.review.application.in.dto.TagResult;
+import io.perfume.api.review.application.in.tag.GetTagUseCase;
+import java.util.List;
+import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
+
+@Service
+@RequiredArgsConstructor
+public class GetTagService implements GetTagUseCase {
+
+  private final TagQueryPersistenceAdapter tagQueryPersistenceAdapter;
+
+  @Override
+  public List<TagResult> getAll() {
+    return tagQueryPersistenceAdapter.findAll().stream().map(TagResult::from).toList();
+  }
+}
diff --git a/perfume-api/src/main/java/io/perfume/api/review/application/service/ReviewTagService.java b/perfume-api/src/main/java/io/perfume/api/review/application/service/ReviewTagService.java
@@ -1,9 +1,9 @@
 package io.perfume.api.review.application.service;
 
-import io.perfume.api.auth.adapter.out.redis.RedisRepository;
 import io.perfume.api.review.application.in.dto.ReviewTagResult;
+import io.perfume.api.review.application.out.tag.ReviewTagQueryRepository;
+import io.perfume.api.review.application.out.tag.ReviewTagRepository;
 import io.perfume.api.review.application.out.tag.TagQueryRepository;
-import io.perfume.api.review.application.out.tag.TagRepository;
 import io.perfume.api.review.domain.ReviewTag;
 import io.perfume.api.review.domain.Tag;
 import java.time.LocalDateTime;
@@ -20,27 +20,32 @@
 @Service
 @RequiredArgsConstructor
 public class ReviewTagService {
-  private final RedisRepository redisRepository;
+
+  private final ReviewTagQueryRepository reviewTagQueryRepository;
   private final TagQueryRepository tagQueryRepository;
-  private final TagRepository tagRepository;
+  private final ReviewTagRepository reviewTagRepository;
+
+  public List<Tag> getTags() {
+    return tagQueryRepository.findAll();
+  }
 
-  public List<Tag> addTags(Long reviewId, List<Long> tagIds) {
+  public List<Tag> addReviewTags(Long reviewId, List<Long> tagIds) {
     var now = LocalDateTime.now();
     var tags = tagQueryRepository.findByIds(tagIds);
     var entities = tags.stream().map(tag -> ReviewTag.create(reviewId, tag.getId(), now)).toList();
 
-    tagRepository.saveAll(entities);
+    reviewTagRepository.saveAll(entities);
 
     return tags;
   }
 
   public void deleteReviewTag(Long reviewId, LocalDateTime now) {
     var reviewTags =
-        tagQueryRepository.findReviewTags(reviewId).stream()
+        reviewTagQueryRepository.findReviewTags(reviewId).stream()
             .peek(reviewTag -> reviewTag.markDelete(now))
             .toList();
 
-    tagRepository.saveAll(reviewTags);
+    reviewTagRepository.saveAll(reviewTags);
   }
 
   public List<ReviewTagResult> getReviewTags(final Long reviewId) {
@@ -52,7 +57,7 @@ public List<ReviewTagResult> getReviewsTags(final List<Long> reviewIds) {
   }
 
   private List<ReviewTagResult> getReviewTags(final List<Long> reviewIds) {
-    final List<ReviewTag> reviewTags = tagQueryRepository.findReviewsTags(reviewIds);
+    final List<ReviewTag> reviewTags = reviewTagQueryRepository.findReviewsTags(reviewIds);
     final List<Long> tagIds = reviewTags.stream().map(ReviewTag::getTagId).toList();
     final Map<Long, ReviewTag> reviewTagMap =
         reviewTags.stream()
diff --git a/perfume-api/src/test/java/io/perfume/api/review/adapter/in/http/ReviewControllerTest.java b/perfume-api/src/test/java/io/perfume/api/review/adapter/in/http/ReviewControllerTest.java
diff --git a/perfume-api/src/test/java/io/perfume/api/review/adapter/out/persistence/repository/tag/ReviewTagQueryPersistenceAdapterTest.java b/perfume-api/src/test/java/io/perfume/api/review/adapter/out/persistence/repository/tag/ReviewTagQueryPersistenceAdapterTest.java
diff --git a/perfume-api/src/test/java/io/perfume/api/review/application/service/ReviewTagServiceTest.java b/perfume-api/src/test/java/io/perfume/api/review/application/service/ReviewTagServiceTest.java
diff --git a/perfume-api/src/test/java/io/perfume/api/review/stub/StubReviewTagRepository.java b/perfume-api/src/test/java/io/perfume/api/review/stub/StubReviewTagRepository.java

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-package io.perfume.api.review.adapter.out.persistence.repository.tag;`
	`1`	`+package io.perfume.api.review.adapter.out.persistence.repository.reviewtag;`
`2`	`2`
`3`	`3`	`import io.perfume.api.base.BaseTimeEntity;`
`4`	`4`	`import jakarta.persistence.EmbeddedId;`