Merge pull request #28 from uju-in/LIME-20-change-pay-point

[LIME-20] 포인트 지급 처리를 AOP -> Event로 변경

Author: Yiseull

Diff for: lime-api/build.gradle

@@ -18,6 +18,8 @@ dependencies {
     implementation 'org.springframework.boot:spring-boot-starter-security'
     implementation 'org.springframework.boot:spring-boot-starter-web'
     implementation 'org.springframework.boot:spring-boot-starter-aop'
+    implementation 'org.springframework:spring-tx'
+
 
     implementation project(':lime-domain')
     implementation project(':lime-common')
@@ -34,7 +36,7 @@ dependencies {
     // 테스트 관련
     testImplementation(testFixtures(project(':lime-domain')))
     testImplementation 'org.springframework.boot:spring-boot-starter-test'
-    
+
     // Cache
     implementation 'org.springframework.boot:spring-boot-starter-cache'
     implementation "com.github.ben-manes.caffeine:caffeine:3.1.8"

Diff for: lime-api/src/main/java/com/programmers/lime/domains/comment/application/CommentService.java

@@ -2,6 +2,7 @@
 
 import org.springframework.context.ApplicationEventPublisher;
 import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
 
 import com.programmers.lime.common.cursor.CursorPageParameters;
 import com.programmers.lime.common.cursor.CursorSummary;
@@ -19,7 +20,7 @@
 import com.programmers.lime.domains.sse.SsePayload;
 import com.programmers.lime.error.BusinessException;
 import com.programmers.lime.error.ErrorCode;
-import com.programmers.lime.global.level.PayPoint;
+import com.programmers.lime.global.event.point.PointEvent;
 import com.programmers.lime.global.util.MemberUtils;
 
 import lombok.RequiredArgsConstructor;
@@ -37,8 +38,8 @@ public class CommentService {
 	private final MemberUtils memberUtils;
 	private final ApplicationEventPublisher applicationEventPublisher;
 
-	@PayPoint(5)
-	public Long createComment(
+	@Transactional
+	public void createComment(
 		final Long feedId,
 		final String content
 	) {
@@ -48,8 +49,7 @@ public Long createComment(
 		SsePayload ssePayload = CommentCreateEvent.toSsePayload(commentWriter.getNickname(), comment);
 
 		applicationEventPublisher.publishEvent(ssePayload);
-
-		return commentWriter.getId();
+		applicationEventPublisher.publishEvent(new PointEvent(commentWriter.getId(), 5));
 	}
 
 	public void modifyComment(
@@ -89,8 +89,8 @@ public CommentGetCursorServiceResponse getFeedComments(
 		return new CommentGetCursorServiceResponse(cursorSummary, totalCommentCount);
 	}
 
-	@PayPoint(20)
-	public Long adoptComment(
+	@Transactional
+	public void adoptComment(
 		final Long feedId,
 		final Long commentId
 	) {
@@ -111,7 +111,6 @@ public Long adoptComment(
 		}
 
 		commentModifier.adopt(comment);
-
-		return comment.getMemberId();
+		applicationEventPublisher.publishEvent(new PointEvent(comment.getMemberId(), 20));
 	}
 }

Diff for: lime-api/src/main/java/com/programmers/lime/domains/review/application/ReviewService.java

@@ -1,12 +1,13 @@
 package com.programmers.lime.domains.review.application;
 
-import java.util.UUID;
-
 import java.io.IOException;
 import java.util.List;
+import java.util.UUID;
 import java.util.stream.Collectors;
 
+import org.springframework.context.ApplicationEventPublisher;
 import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
 import org.springframework.util.StringUtils;
 import org.springframework.web.multipart.MultipartFile;
 
@@ -26,7 +27,7 @@
 import com.programmers.lime.domains.review.model.ReviewSortCondition;
 import com.programmers.lime.error.BusinessException;
 import com.programmers.lime.error.ErrorCode;
-import com.programmers.lime.global.level.PayPoint;
+import com.programmers.lime.global.event.point.PointEvent;
 import com.programmers.lime.global.util.MemberUtils;
 import com.programmers.lime.s3.S3Manager;
 
@@ -47,9 +48,10 @@ public class ReviewService {
 	private final MemberUtils memberUtils;
 	private final ReviewReader reviewReader;
 	private final S3Manager s3Manager;
+	private final ApplicationEventPublisher applicationEventPublisher;
 
-	@PayPoint(15)
-	public Long createReview(
+	@Transactional
+	public void createReview(
 		final Long itemId,
 		final ReviewContent reviewContent,
 		final List<MultipartFile> multipartReviewImages
@@ -58,8 +60,7 @@ public Long createReview(
 
 		Long memberId = memberUtils.getCurrentMemberId();
 		reviewAppender.append(itemId, memberId, reviewContent, reviewImageURLs);
-
-		return memberId;
+		applicationEventPublisher.publishEvent(new PointEvent(memberId, 15));
 	}
 
 	private List<String> uploadReviewImages(final List<MultipartFile> multipartReviewImages) {

Diff for: lime-api/src/main/java/com/programmers/lime/global/event/point/PointEvent.java

@@ -0,0 +1,12 @@
+package com.programmers.lime.global.event.point;
+
+import lombok.Getter;
+import lombok.RequiredArgsConstructor;
+
+@Getter
+@RequiredArgsConstructor
+public class PointEvent {
+
+	private final Long memberId;
+	private final int point;
+}

Diff for: lime-api/src/main/java/com/programmers/lime/global/event/point/PointEventListener.java

@@ -0,0 +1,22 @@
+package com.programmers.lime.global.event.point;
+
+import org.springframework.context.event.EventListener;
+import org.springframework.stereotype.Component;
+
+import com.programmers.lime.domains.member.domain.Member;
+import com.programmers.lime.domains.member.implementation.MemberReader;
+
+import lombok.RequiredArgsConstructor;
+
+@Component
+@RequiredArgsConstructor
+public class PointEventListener {
+
+	private final MemberReader memberReader;
+
+	@EventListener
+	public void earnPoint(final PointEvent pointEvent) {
+		final Member member = memberReader.read(pointEvent.getMemberId());
+		member.earnPoint(pointEvent.getPoint());
+	}
+}