Merge pull request #37 from Zipbap/refactor/#35

[Refactor]Recipe 조회수 반영 로직 개선

Author: wnd01jun

api-user/src/main/kotlin/zipbap/user/api/recipe/event/RecipeViewCountListener.kt

@@ -0,0 +1,20 @@
+package zipbap.user.api.recipe.event
+
+import org.springframework.stereotype.Component
+import org.springframework.transaction.annotation.Propagation
+import org.springframework.transaction.annotation.Transactional
+import org.springframework.transaction.event.TransactionPhase
+import org.springframework.transaction.event.TransactionalEventListener
+import zipbap.global.domain.recipe.RecipeRepository
+
+@Component
+class RecipeViewCountListener(
+        private val recipeRepository: RecipeRepository
+) {
+
+    @TransactionalEventListener(phase = TransactionPhase.AFTER_COMMIT)
+    @Transactional(propagation = Propagation.REQUIRES_NEW)
+    fun onRecipeViewed(event: RecipeViewedEvent) {
+        recipeRepository.updateViewCount(event.recipeId)
+    }
+}

api-user/src/main/kotlin/zipbap/user/api/recipe/event/RecipeViewedEvent.kt

@@ -0,0 +1,6 @@
+package zipbap.user.api.recipe.event
+
+data class RecipeViewedEvent(
+        val recipeId: String,
+        val viewerUserId: Long
+)

api-user/src/main/kotlin/zipbap/user/api/recipe/service/RecipeService.kt

@@ -1,5 +1,6 @@
 package zipbap.user.api.recipe.service
 
+import org.springframework.context.ApplicationEventPublisher
 import org.springframework.stereotype.Service
 import org.springframework.transaction.annotation.Transactional
 import zipbap.user.api.recipe.converter.RecipeConverter
@@ -17,14 +18,16 @@ import zipbap.global.domain.user.UserRepository
 import zipbap.global.global.util.CustomIdGenerator
 import zipbap.global.global.exception.GeneralException
 import zipbap.global.global.code.status.ErrorStatus
+import zipbap.user.api.recipe.event.RecipeViewedEvent
 
 @Service
 class RecipeService(
         private val recipeRepository: RecipeRepository,
         private val cookingOrderRepository: CookingOrderRepository,
         private val userRepository: UserRepository,
         private val categoryValidator: CategoryValidator,
-        private val recipeFileService: RecipeFileService
+        private val recipeFileService: RecipeFileService,
+        private val publisher: ApplicationEventPublisher
 ) {
 
     /**
@@ -206,12 +209,10 @@ class RecipeService(
             throw GeneralException(ErrorStatus.RECIPE_NOT_FOUND)
         }
 
-        // 조회수 증가
-        recipe.viewCount += 1
-        val savedRecipe = recipeRepository.save(recipe)
-
         val orders = cookingOrderRepository.findAllByRecipeId(recipe.id)
-        return RecipeConverter.toDto(savedRecipe, orders)
+
+        publisher.publishEvent(RecipeViewedEvent(recipeId, userId))
+        return RecipeConverter.toDto(recipe, orders)
     }
 
     /**

global/build.gradle.kts

@@ -45,8 +45,8 @@ dependencyManagement {
         // ⚠️ JDBC 드라이버는 애플리케이션 모듈(api-*)에 두는 걸 권장. (여기선 뺌)
 
         // Querydsl (APT는 여기서!)
-        implementation("com.querydsl:querydsl-jpa:5.0.0:jakarta")
-        kapt("com.querydsl:querydsl-apt:5.0.0:jakarta")
+        implementation("com.querydsl:querydsl-jpa:5.1.0:jakarta")
+        kapt("com.querydsl:querydsl-apt:5.1.0:jakarta")
         kapt("jakarta.annotation:jakarta.annotation-api")
         kapt("jakarta.persistence:jakarta.persistence-api")
 

global/src/main/kotlin/zipbap/global/domain/recipe/RecipeRepository.kt

@@ -1,6 +1,7 @@
 package zipbap.global.domain.recipe
 
 import org.springframework.data.jpa.repository.JpaRepository
+import org.springframework.data.jpa.repository.Modifying
 import org.springframework.data.jpa.repository.Query
 import org.springframework.data.repository.query.Param
 
@@ -35,4 +36,10 @@ interface RecipeRepository : JpaRepository<Recipe, String> {
 
     // ✅ 여기만 남기면 됨
     fun findAllByMyCategoryId(myCategoryId: String): List<Recipe>
+
+    @Modifying(clearAutomatically = true, flushAutomatically = true)
+    @Query("""
+        UPDATE Recipe r SET r.viewCount = r.viewCount + 1 WHERE r.id = :recipeId
+    """)
+    fun updateViewCount(@Param("recipeId") recipeId: String): Unit
 }