Skip to content

[FIX] Post 카운팅 버그 및 Redis 역직렬화 해결 #101

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
LJW22222 merged 10 commits into from
Sep 15, 2025
Merged

[FIX] Post 카운팅 버그 및 Redis 역직렬화 해결 #101

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
d003d8e
refactor: class -> record로 변환
LJW22222 Aug 20, 2025
97a171a
refactor: redis 역직렬화 수정 (EVERYTHING -> Record로 한정 )
LJW22222 Aug 20, 2025
93d41ab
refactor: class -> record로 변환
LJW22222 Aug 20, 2025
0ae5184
fix: Entity Lazy 역직렬화 에러 해결 ( graph fetch 도입 )
LJW22222 Aug 20, 2025
2e3712d
refactor: 필요 없는 DI 제거
LJW22222 Aug 20, 2025
92d85bc
fix: Post view count 원자적 업데이트 도입
LJW22222 Aug 20, 2025
c5ada65
fix: Post view count 원자적 업데이트 도입에 따른 test 파일 수정
LJW22222 Aug 20, 2025
213e9dd
fix: Post view count 원자적 업데이트 도입에 따른 test 파일 수정
LJW22222 Aug 20, 2025
46fe6b6
Merge Main
LJW22222 Sep 15, 2025
445c198
Merge Main
LJW22222 Sep 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/main/java/backend/airo/api/area_code/AreaCodeController.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public Response<List<MegaCodeResponse>> getMegaCodes(){
List<MegaCode> megaCodes = areaCodeUseCase.getMegaCodeList();
return Response.success(
megaCodes.stream().map(list ->
new MegaCodeResponse(list.getCtprvnCd(), list.getCtprvnNm())
new MegaCodeResponse(list.ctprvnCd(), list.ctprvnNm())
).toList()
);
}
Expand All @@ -39,7 +39,7 @@ public Response<List<MegaCodeResponse>> getMegaCodes(){
public Response<List<CityCodeResponse>> getCityCodes() {
List<CityCode> cityCodes = areaCodeUseCase.getCityCodeList();
return Response.success(cityCodes.stream().map(list ->
new CityCodeResponse(list.getCtprvnCd(), list.getCtprvnNm(), list.getMegaCodeId())
new CityCodeResponse(list.ctprvnCd(), list.ctprvnNm(), list.megaCodeId())
).toList());
}
}
5 changes: 2 additions & 3 deletions src/main/java/backend/airo/api/clutr_fatvl/ClutrFatvlController.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
import backend.airo.api.global.swagger.ClutrFatvlControllerSwagger;
import backend.airo.application.clure_fatvl.usecase.ClutrFatvlUseCase;
import backend.airo.cache.area_code.AreaCodeCacheService;
import backend.airo.cache.clutr_fatvl.ClutrFatvlCacheService;
import backend.airo.domain.clure_fatvl.ClutrFatvl;
import io.swagger.v3.oas.annotations.tags.Tag;
import lombok.RequiredArgsConstructor;
Expand Down Expand Up @@ -46,8 +45,8 @@ public Response<PageResponse<ClutrFatvListResponse>> getClureFatvlList(
List<ClutrFatvListResponse> content = clutrFatvls.getContent().stream().map(list ->
ClutrFatvListResponse.create(
list,
areaCodeCacheService.getMegaName(Long.valueOf(list.getAddress().megaCodeId())),
areaCodeCacheService.getCityName(Long.valueOf(list.getAddress().ctprvnCodeId()), Long.valueOf(list.getAddress().megaCodeId())))
areaCodeCacheService.getMegaName(Long.valueOf(list.address().megaCodeId())),
areaCodeCacheService.getCityName(Long.valueOf(list.address().ctprvnCodeId()), Long.valueOf(list.address().megaCodeId())))
).toList();
return Response.success(
new PageResponse<>(
Expand Down
24 changes: 12 additions & 12 deletions src/main/java/backend/airo/api/clutr_fatvl/dto/ClutrFatvInfoResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,18 +54,18 @@ public record ClutrFatvInfoResponse(

public static ClutrFatvInfoResponse create(ClutrFatvl clutrFatvlInfo) {
return ClutrFatvInfoResponse.builder()
.id(clutrFatvlInfo.getId())
.fstvlNm(clutrFatvlInfo.getFstvlNm())
.opar(clutrFatvlInfo.getOpar())
.fstvlCo(clutrFatvlInfo.getFstvlCo())
.period(clutrFatvlInfo.getPeriod())
.address(clutrFatvlInfo.getAddress())
.mnnstNm(clutrFatvlInfo.getMnnstNm())
.auspcInsttNm(clutrFatvlInfo.getAuspcInsttNm())
.suprtInsttNm(clutrFatvlInfo.getSuprtInsttNm())
.phoneNumber(clutrFatvlInfo.getPhoneNumber())
.homepageUrl(clutrFatvlInfo.getHomepageUrl())
.relateInfo(clutrFatvlInfo.getRelateInfo())
.id(clutrFatvlInfo.id())
.fstvlNm(clutrFatvlInfo.fstvlNm())
.opar(clutrFatvlInfo.opar())
.fstvlCo(clutrFatvlInfo.fstvlCo())
.period(clutrFatvlInfo.period())
.address(clutrFatvlInfo.address())
.mnnstNm(clutrFatvlInfo.mnnstNm())
.auspcInsttNm(clutrFatvlInfo.auspcInsttNm())
.suprtInsttNm(clutrFatvlInfo.suprtInsttNm())
.phoneNumber(clutrFatvlInfo.phoneNumber())
.homepageUrl(clutrFatvlInfo.homepageUrl())
.relateInfo(clutrFatvlInfo.relateInfo())
.build();
}

Expand Down
16 changes: 8 additions & 8 deletions src/main/java/backend/airo/api/clutr_fatvl/dto/ClutrFatvListResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,15 +23,15 @@ public record ClutrFatvListResponse(

public static ClutrFatvListResponse create(ClutrFatvl clutrFatvl, String megaName, String cityName) {
return ClutrFatvListResponse.builder()
.id(clutrFatvl.getId())
.name(clutrFatvl.getFstvlNm())
.startDate(clutrFatvl.getPeriod().start())
.endDate(clutrFatvl.getPeriod().end())
.id(clutrFatvl.id())
.name(clutrFatvl.fstvlNm())
.startDate(clutrFatvl.period().start())
.endDate(clutrFatvl.period().end())
.region(addMegaNameCityName(megaName, cityName))
.place(clutrFatvl.getOpar())
.progressCheck(clutrFatvl.getPeriod().progressCheck())
.periodCheck(clutrFatvl.getPeriod().periodCheck())
.ended(clutrFatvl.getPeriod().ended())
.place(clutrFatvl.opar())
.progressCheck(clutrFatvl.period().progressCheck())
.periodCheck(clutrFatvl.period().periodCheck())
.ended(clutrFatvl.period().ended())
.build();
}

Expand Down
1 change: 0 additions & 1 deletion src/main/java/backend/airo/api/post/PostController.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
import backend.airo.api.global.dto.Response;
import backend.airo.api.global.swagger.PostControllerSwagger;
import backend.airo.application.post.usecase.PostCacheUseCase;
import backend.airo.application.post.usecase.PostUseCase;
import backend.airo.domain.post.Post;
import backend.airo.api.post.dto.*;
import backend.airo.domain.user.User;
Expand Down
32 changes: 16 additions & 16 deletions src/main/java/backend/airo/api/post/dto/PostDetailResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,22 +75,22 @@ public static PostDetailResponse toResponse(Post post,
List<Image> imageList
) {
return new PostDetailResponse(
post.getId(),
post.getTitle(),
post.getContent(),
post.getSummary(),
post.getBusinessName(),
post.getStatus(),
post.getForWhatTag(),
post.getEmotionTags(),
post.getTravelDate(),
post.getLocation(),
post.getAddress(),
post.getViewCount(),
post.getLikeCount(),
post.getCommentCount(),
post.getIsFeatured(),
post.getPublishedAt(),
post.id(),
post.title(),
post.content(),
post.summary(),
post.businessName(),
post.status(),
post.forWhatTag(),
post.emotionTags(),
post.travelDate(),
post.location(),
post.address(),
post.viewCount(),
post.likeCount(),
post.commentCount(),
post.isFeatured(),
post.publishedAt(),
author,
imageList
);
Expand Down
22 changes: 10 additions & 12 deletions src/main/java/backend/airo/api/post/dto/PostResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,11 @@
package backend.airo.api.post.dto;

import backend.airo.domain.comment.Comment;
import backend.airo.domain.post.Post;
import backend.airo.domain.post.enums.PostStatus;
import io.swagger.v3.oas.annotations.media.Schema;

import java.time.LocalDate;
import java.time.LocalDateTime;
import java.util.List;

/**
* 게시물 응답 DTO
Expand Down Expand Up @@ -48,16 +46,16 @@ public record PostResponse(
) {
public static PostResponse fromDomain(Post post) {
return new PostResponse(
post.getId(),
post.getTitle(),
post.getSummary(),
post.getStatus(),
post.getTravelDate(),
post.getViewCount(),
post.getLikeCount(),
post.getCommentCount(),
post.getIsFeatured(),
post.getPublishedAt()
post.id(),
post.title(),
post.summary(),
post.status(),
post.travelDate(),
post.viewCount(),
post.likeCount(),
post.commentCount(),
post.isFeatured(),
post.publishedAt()
);
}
}
16 changes: 8 additions & 8 deletions src/main/java/backend/airo/api/post/dto/PostSummaryResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,14 +22,14 @@ public record PostSummaryResponse(
) {
public static PostSummaryResponse fromDomain(Post post) {
return new PostSummaryResponse(
post.getId(),
post.getTitle(),
post.getContent(),
post.getStatus(),
post.getViewCount(),
post.getEmotionTags() != null ?
new ArrayList<>(post.getEmotionTags()) : new ArrayList<>(),
post.getUserId()
post.id(),
post.title(),
post.content(),
post.status(),
post.viewCount(),
post.emotionTags() != null ?
new ArrayList<>(post.emotionTags()) : new ArrayList<>(),
post.userId()
);
}

Expand Down
20 changes: 10 additions & 10 deletions src/main/java/backend/airo/api/post/dto/PostThumbnailResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,16 +48,16 @@ public static PostThumbnailResponse fromDomain(Post post, ThumbnailResult thumbn
) : null;

return new PostThumbnailResponse(
post.getId(),
post.getTitle(),
post.getSummary(),
post.getStatus(),
post.getTravelDate(),
post.getViewCount(),
post.getLikeCount(),
post.getCommentCount(),
post.getIsFeatured(),
post.getPublishedAt(),
post.id(),
post.title(),
post.summary(),
post.status(),
post.travelDate(),
post.viewCount(),
post.likeCount(),
post.commentCount(),
post.isFeatured(),
post.publishedAt(),
thumbnailInfo
);
}
Expand Down
12 changes: 6 additions & 6 deletions src/main/java/backend/airo/api/shop/dto/ShopInfoResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,12 @@ public record ShopInfoResponse(

public static ShopInfoResponse create(Shop shop) {
return ShopInfoResponse.builder()
.id(shop.getId())
.shopName(shop.getShopName())
.roadAddr(shop.getAddress().road())
.lotAddr(shop.getAddress().lot())
.flrNo(shop.getFloorInfo().flrNo())
.hoNo(shop.getFloorInfo().hoNo())
.id(shop.id())
.shopName(shop.shopName())
.roadAddr(shop.address().road())
.lotAddr(shop.address().lot())
.flrNo(shop.floorInfo().flrNo())
.hoNo(shop.floorInfo().hoNo())
.build();
}

Expand Down
10 changes: 5 additions & 5 deletions src/main/java/backend/airo/api/shop/dto/ShopListResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@ public record ShopListResponse(

public static ShopListResponse create(Shop shop) {
return new ShopListResponse(
shop.getId(),
shop.getShopName(),
shop.getAddress().lot(),
shop.getAddress().road(),
shop.getShopType().getTypeName()
shop.id(),
shop.shopName(),
shop.address().lot(),
shop.address().road(),
shop.shopType().getTypeName()
);
}

Expand Down
32 changes: 12 additions & 20 deletions src/main/java/backend/airo/application/post/usecase/PostCacheUseCase.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
import backend.airo.domain.post.command.CreatePostCommandService;
import backend.airo.domain.post.command.DeletePostCommandService;
import backend.airo.domain.post.command.UpdatePostCommandService;
import backend.airo.domain.post.command.UpdatePostViewCountCommand;
import backend.airo.domain.post.enums.PostStatus;
import backend.airo.domain.post.exception.PostException;
import backend.airo.domain.post.query.GetPostListQueryService;
Expand All @@ -35,14 +36,15 @@
@Slf4j
@Service
@RequiredArgsConstructor
@Transactional(readOnly = true)
public class PostCacheUseCase {

private final CreatePostCommandService createPostCommandService;
private final UpdatePostCommandService updatePostCommandService;
private final DeletePostCommandService deletePostCommandService;
private final UpsertPointCommand upsertPointCommand;
private final CreatePointHistoryCommand createPointHistoryCommand;
private final UpdatePostViewCountCommand updatePostViewCountCommand;

private final GetPostListQueryService getPostListQueryService;
private final GetPostQueryService getPostQueryService;
private final GetUserQuery getUserQueryService;
Expand All @@ -52,7 +54,6 @@ public class PostCacheUseCase {



@Transactional
public Post createPost(PostCreateRequest request, Long userId) {
Post savedPost;

Expand All @@ -61,7 +62,7 @@ public Post createPost(PostCreateRequest request, Long userId) {
}else{
savedPost = createPostCommandService.handle(request, userId);

boolean handle = createPointHistoryCommand.handle(userId, 100L, savedPost.getId(), PointType.REPORT);
boolean handle = createPointHistoryCommand.handle(userId, 100L, savedPost.id(), PointType.REPORT);
if (handle) {
upsertPointCommand.handle(userId, 100L);
}
Expand All @@ -75,22 +76,15 @@ public Post createPost(PostCreateRequest request, Long userId) {


public PostDetailResponse getPostDetail(Long postId, Long requesterId) {
log.debug("게시물 조회: id={}, requesterId={}", postId, requesterId);

Post post;
try {
post = postCacheService.getPost(postId);
} catch (Exception e) {
log.warn("캐시에서 게시물 조회 실패, DB에서 직접 조회: postId={}, error={}",
postId, e.getMessage());
post = getPostQueryService.handle(postId);
}
log.info("게시물 조회: id={}, requesterId={}", postId, requesterId);

if(!isPostOwner(post, requesterId)) {
post.incrementViewCount();
Post post = postCacheService.getPost(postId);
if(!post.isPostOwner(requesterId)) {
updatePostViewCountCommand.handle(postId);
postCacheService.evictPostCaches(postId);
}

AuthorInfo authorInfo = getAuthorInfo(post.getUserId());
AuthorInfo authorInfo = getAuthorInfo(post.userId());
List<Image> imageList = new ArrayList<>(
getImageQueryService.getImagesBelongsPost(postId)
);
Expand Down Expand Up @@ -126,7 +120,6 @@ public Slice<PostSummaryResponse> getPostSlice(PostSliceRequest request) {



@Transactional
public Post updatePost(Long postId, Long requesterId, PostUpdateRequest request) {
log.info("게시물 수정 시작: id={}, requesterId={}", postId, requesterId);

Expand All @@ -144,7 +137,6 @@ public Post updatePost(Long postId, Long requesterId, PostUpdateRequest request)
}


@Transactional
public void deletePost(Long postId, Long requesterId) {
log.info("게시물 삭제 시작: id={}, requesterId={}", postId, requesterId);

Expand Down Expand Up @@ -175,12 +167,12 @@ private AuthorInfo getAuthorInfo(Long autherId) {

private void validatePostOwnership(Post post, Long requesterId) {
if (!isPostOwner(post, requesterId)) {
throw PostException.accessDenied(post.getId(), requesterId);
throw PostException.accessDenied(post.id(), requesterId);
}
}

private boolean isPostOwner(Post post, Long userId) {
return userId != null && userId.equals(post.getUserId());
return userId != null && userId.equals(post.userId());
}


Expand Down
Loading