From 3cca37ca400bba25d762775e80b8d8bc9f337091 Mon Sep 17 00:00:00 2001 From: JunHyeongChoi Date: Wed, 16 Oct 2024 17:09:27 +0900 Subject: [PATCH 1/2] =?UTF-8?q?fix:=20=EC=B9=B4=ED=85=8C=EA=B3=A1=EB=A6=AC?= =?UTF-8?q?=20=ED=95=84=ED=84=B0=20=EA=B4=80=EB=A0=A8=20=EC=97=90=EB=9F=AC?= =?UTF-8?q?=20=EC=88=98=EC=A0=95?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 카테고리 이름을 받아야 하는데 ID를 받아 처리하던 부분 수정 및 JOIN 관련 수정 --- .../dto/request/DiaryFilterRequest.java | 5 +++-- .../dto/request/DiaryManualCreateRequest.java | 2 -- .../com/potatocake/everymoment/entity/Diary.java | 7 +++++++ .../everymoment/entity/DiaryCategory.java | 9 +++++++++ .../everymoment/service/DiaryService.java | 9 ++++----- .../everymoment/service/DiarySpecification.java | 15 +++++++++------ .../everymoment/service/FriendDiaryService.java | 7 ++++--- 7 files changed, 36 insertions(+), 18 deletions(-) diff --git a/src/main/java/com/potatocake/everymoment/dto/request/DiaryFilterRequest.java b/src/main/java/com/potatocake/everymoment/dto/request/DiaryFilterRequest.java index 720a07e..1f31183 100644 --- a/src/main/java/com/potatocake/everymoment/dto/request/DiaryFilterRequest.java +++ b/src/main/java/com/potatocake/everymoment/dto/request/DiaryFilterRequest.java @@ -27,10 +27,11 @@ public List getEmojis() { : Collections.emptyList(); } - public List getCategories() { + public List getCategories() { return (category != null && !category.isEmpty()) ? Arrays.stream(category.split(",")) - .map(Long::parseLong) + .map(String::trim) + .filter(s -> !s.isEmpty()) .collect(Collectors.toList()) : Collections.emptyList(); } diff --git a/src/main/java/com/potatocake/everymoment/dto/request/DiaryManualCreateRequest.java b/src/main/java/com/potatocake/everymoment/dto/request/DiaryManualCreateRequest.java index 849298a..9df2ed2 100644 --- a/src/main/java/com/potatocake/everymoment/dto/request/DiaryManualCreateRequest.java +++ b/src/main/java/com/potatocake/everymoment/dto/request/DiaryManualCreateRequest.java @@ -13,7 +13,5 @@ public class DiaryManualCreateRequest { private boolean isBookmark; private boolean isPublic; private String emoji; - private List file; private String content; } - diff --git a/src/main/java/com/potatocake/everymoment/entity/Diary.java b/src/main/java/com/potatocake/everymoment/entity/Diary.java index 23d7583..c60f671 100644 --- a/src/main/java/com/potatocake/everymoment/entity/Diary.java +++ b/src/main/java/com/potatocake/everymoment/entity/Diary.java @@ -1,5 +1,6 @@ package com.potatocake.everymoment.entity; +import jakarta.persistence.CascadeType; import jakarta.persistence.Column; import jakarta.persistence.Entity; import jakarta.persistence.FetchType; @@ -9,6 +10,9 @@ import jakarta.persistence.JoinColumn; import jakarta.persistence.Lob; import jakarta.persistence.ManyToOne; +import jakarta.persistence.OneToMany; +import java.util.HashSet; +import java.util.Set; import lombok.AccessLevel; import lombok.AllArgsConstructor; import lombok.Builder; @@ -53,6 +57,9 @@ public class Diary extends BaseTimeEntity { @Builder.Default private boolean isPublic = false; + @OneToMany(mappedBy = "diary", cascade = CascadeType.ALL, orphanRemoval = true) + private Set diaryCategories = new HashSet<>(); + public void updateContent(String content) { if (content != null) { this.content = content; diff --git a/src/main/java/com/potatocake/everymoment/entity/DiaryCategory.java b/src/main/java/com/potatocake/everymoment/entity/DiaryCategory.java index 3a724d8..534c7dd 100644 --- a/src/main/java/com/potatocake/everymoment/entity/DiaryCategory.java +++ b/src/main/java/com/potatocake/everymoment/entity/DiaryCategory.java @@ -31,4 +31,13 @@ public class DiaryCategory { @ManyToOne(fetch = FetchType.LAZY) @JoinColumn(foreignKey = @ForeignKey(name = "fk_category_id"), nullable = false) private Category category; + + public void setDiary(Diary diary) { + this.diary = diary; + } + + public void setCategory(Category category) { + this.category = category; + } + } diff --git a/src/main/java/com/potatocake/everymoment/service/DiaryService.java b/src/main/java/com/potatocake/everymoment/service/DiaryService.java index 8293530..c4ac02a 100644 --- a/src/main/java/com/potatocake/everymoment/service/DiaryService.java +++ b/src/main/java/com/potatocake/everymoment/service/DiaryService.java @@ -100,7 +100,6 @@ public void createDiaryManual(Long memberId, DiaryManualCreateRequest diaryManua Member currentMember = memberRepository.findById(memberId) .orElseThrow(() -> new GlobalException(ErrorCode.MEMBER_NOT_FOUND)); - double longitude = diaryManualCreateRequest.getLocationPoint().getLongitude(); double latitude = diaryManualCreateRequest.getLocationPoint().getLatitude(); @@ -149,21 +148,21 @@ public MyDiariesResponse getMyDiaries(Long memberId, DiaryFilterRequest diaryFil Page diaryPage; - List categoryIds = diaryFilterRequest.getCategories(); + List categories = diaryFilterRequest.getCategories(); List emojis = diaryFilterRequest.getEmojis(); Specification spec = DiarySpecification.filterDiaries( diaryFilterRequest.getKeyword(), emojis, - categoryIds, + categories, diaryFilterRequest.getDate(), diaryFilterRequest.getFrom(), diaryFilterRequest.getUntil(), diaryFilterRequest.getIsBookmark()) .and((root, query, builder) -> builder.equal(root.get("member"), currentMember)); - diaryPage = diaryRepository.findAll(spec, PageRequest.of(diaryFilterRequest.getKey(), diaryFilterRequest.getSize())); - + diaryPage = diaryRepository.findAll(spec, + PageRequest.of(diaryFilterRequest.getKey(), diaryFilterRequest.getSize())); List diaryDTOs = diaryPage.getContent().stream() .map(this::convertToMyDiarySimpleResponseDto) diff --git a/src/main/java/com/potatocake/everymoment/service/DiarySpecification.java b/src/main/java/com/potatocake/everymoment/service/DiarySpecification.java index 3a9826d..95dd147 100644 --- a/src/main/java/com/potatocake/everymoment/service/DiarySpecification.java +++ b/src/main/java/com/potatocake/everymoment/service/DiarySpecification.java @@ -1,22 +1,23 @@ package com.potatocake.everymoment.service; +import com.potatocake.everymoment.entity.Category; import com.potatocake.everymoment.entity.Diary; import com.potatocake.everymoment.entity.DiaryCategory; import jakarta.persistence.criteria.CriteriaBuilder; import jakarta.persistence.criteria.CriteriaQuery; import jakarta.persistence.criteria.Join; +import jakarta.persistence.criteria.JoinType; import jakarta.persistence.criteria.Predicate; import jakarta.persistence.criteria.Root; +import java.time.LocalDate; import java.util.List; import org.springframework.data.jpa.domain.Specification; -import java.time.LocalDate; - public class DiarySpecification { public static Specification filterDiaries( - String keyword, List emojis, List categories, + String keyword, List emojis, List categories, LocalDate date, LocalDate from, LocalDate until, Boolean isBookmark) { return (Root root, CriteriaQuery query, CriteriaBuilder builder) -> { Predicate predicate = builder.conjunction(); @@ -29,15 +30,17 @@ public static Specification filterDiaries( } if (categories != null && !categories.isEmpty()) { - Join diaryCategoryJoin = root.join("diaryCategories"); - predicate = builder.and(predicate, diaryCategoryJoin.get("category").get("id").in(categories)); + Join diaryCategoryJoin = root.join("diaryCategories", JoinType.LEFT); + Join categoryJoin = diaryCategoryJoin.join("category", JoinType.LEFT); + predicate = builder.and(predicate, categoryJoin.get("categoryName").in(categories)); } LocalDate filterDate = (date != null) ? date : LocalDate.now(); predicate = builder.and(predicate, builder.equal(root.get("createAt").as(LocalDate.class), filterDate)); if (from != null && until != null) { - predicate = builder.and(predicate, builder.between(root.get("createAt"), from.atStartOfDay(), until.plusDays(1).atStartOfDay())); + predicate = builder.and(predicate, + builder.between(root.get("createAt"), from.atStartOfDay(), until.plusDays(1).atStartOfDay())); } if (isBookmark != null) { predicate = builder.and(predicate, builder.equal(root.get("isBookmark"), isBookmark)); diff --git a/src/main/java/com/potatocake/everymoment/service/FriendDiaryService.java b/src/main/java/com/potatocake/everymoment/service/FriendDiaryService.java index fb83595..2afa0a4 100644 --- a/src/main/java/com/potatocake/everymoment/service/FriendDiaryService.java +++ b/src/main/java/com/potatocake/everymoment/service/FriendDiaryService.java @@ -53,20 +53,21 @@ public FriendDiariesResponse getFriendDiaries(Long memberId, DiaryFilterRequest Page diaryPage; // 카테고리와 이모지가 여러 개 전달될 수 있으므로 이를 리스트로 변환 - List categoryIds = diaryFilterRequest.getCategories(); + List categories = diaryFilterRequest.getCategories(); List emojis = diaryFilterRequest.getEmojis(); Specification spec = DiarySpecification.filterDiaries( diaryFilterRequest.getKeyword(), emojis, - categoryIds, + categories, diaryFilterRequest.getDate(), diaryFilterRequest.getFrom(), diaryFilterRequest.getUntil(), diaryFilterRequest.getIsBookmark()) .and((root, query, builder) -> root.get("member").in(friendIdList)); - diaryPage = diaryRepository.findAll(spec, PageRequest.of(diaryFilterRequest.getKey(), diaryFilterRequest.getSize())); + diaryPage = diaryRepository.findAll(spec, + PageRequest.of(diaryFilterRequest.getKey(), diaryFilterRequest.getSize())); List friendDiarySimpleResponseList = diaryPage.getContent().stream() .map(this::convertToFriendDiariesResponseDTO) From 8d33ca53983422fceb3b181570b6debebf760788 Mon Sep 17 00:00:00 2001 From: JunHyeongChoi Date: Wed, 16 Oct 2024 17:22:26 +0900 Subject: [PATCH 2/2] =?UTF-8?q?feat:=20=EC=B9=9C=EA=B5=AC=20=EC=9A=94?= =?UTF-8?q?=EC=B2=AD=20=EB=AA=A9=EB=A1=9D=20API=20=EC=9D=91=EB=8B=B5=20?= =?UTF-8?q?=ED=95=84=EB=93=9C=EC=97=90=20=EB=B3=B4=EB=82=B8=20=EC=82=AC?= =?UTF-8?q?=EC=9A=A9=EC=9E=90=20=EC=A0=95=EB=B3=B4=20=EC=B6=94=EA=B0=80?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../dto/response/FriendRequestResponse.java | 2 ++ .../service/FriendRequestService.java | 28 ++++++++++++++++--- 2 files changed, 26 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/potatocake/everymoment/dto/response/FriendRequestResponse.java b/src/main/java/com/potatocake/everymoment/dto/response/FriendRequestResponse.java index 77f1771..45fe6d3 100644 --- a/src/main/java/com/potatocake/everymoment/dto/response/FriendRequestResponse.java +++ b/src/main/java/com/potatocake/everymoment/dto/response/FriendRequestResponse.java @@ -9,5 +9,7 @@ public class FriendRequestResponse { private Long id; private Long senderId; + private String nickname; + private String profileImageUrl; } diff --git a/src/main/java/com/potatocake/everymoment/service/FriendRequestService.java b/src/main/java/com/potatocake/everymoment/service/FriendRequestService.java index 0373b19..89f99a9 100644 --- a/src/main/java/com/potatocake/everymoment/service/FriendRequestService.java +++ b/src/main/java/com/potatocake/everymoment/service/FriendRequestService.java @@ -1,5 +1,6 @@ package com.potatocake.everymoment.service; +import static java.util.function.Function.identity; import static org.springframework.data.domain.Sort.Direction.DESC; import com.potatocake.everymoment.dto.response.FriendRequestPageRequest; @@ -14,6 +15,9 @@ import com.potatocake.everymoment.repository.MemberRepository; import com.potatocake.everymoment.util.PagingUtil; import java.util.List; +import java.util.Map; +import java.util.Set; +import java.util.stream.Collectors; import lombok.RequiredArgsConstructor; import org.springframework.data.domain.Pageable; import org.springframework.data.domain.ScrollPosition; @@ -106,11 +110,27 @@ private Window fetchFriendRequestWindow(Long key, int size, Long } private List convertToFriendRequestResponses(List requests) { + Set senderIds = requests.stream() + .map(request -> request.getSender().getId()) + .collect(Collectors.toSet()); + + Map senderMap = memberRepository.findAllById(senderIds) + .stream() + .collect(Collectors.toMap(Member::getId, identity())); + return requests.stream() - .map(request -> FriendRequestResponse.builder() - .id(request.getId()) - .senderId(request.getSender().getId()) - .build()) + .map(request -> { + Member sender = senderMap.get(request.getSender().getId()); + if (sender == null) { + throw new GlobalException(ErrorCode.MEMBER_NOT_FOUND); + } + return FriendRequestResponse.builder() + .id(request.getId()) + .senderId(sender.getId()) + .nickname(sender.getNickname()) + .profileImageUrl(sender.getProfileImageUrl()) + .build(); + }) .toList(); }