Merge pull request #379 from Team-Ampersand/delete-music-list-cache

Delete music list cache
Team-Ampersand · Jul 7, 2024 · 2ffdb41 · 2ffdb41
2 parents b3435c8 + 82629b9
commit 2ffdb41
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/src/main/kotlin/com/dotori/v2/domain/music/service/impl/DeleteMusicServiceImpl.kt b/src/main/kotlin/com/dotori/v2/domain/music/service/impl/DeleteMusicServiceImpl.kt
@@ -4,6 +4,7 @@ import com.dotori.v2.domain.member.enums.MusicStatus
 import com.dotori.v2.domain.music.domain.entity.Music
 import com.dotori.v2.domain.music.domain.repository.MusicRepository
 import com.dotori.v2.domain.music.exception.MusicNotFoundException
+import com.dotori.v2.domain.music.presentation.data.res.MusicListResDto
 import com.dotori.v2.domain.music.service.DeleteMusicService
 import com.dotori.v2.global.config.redis.service.RedisCacheService
 import org.springframework.data.repository.findByIdOrNull
@@ -16,15 +17,13 @@ class DeleteMusicServiceImpl(
     private val musicRepository: MusicRepository,
     private val redisCacheService: RedisCacheService
 ) : DeleteMusicService {
+
     override fun execute(musicId: Long) {
         val music: Music = musicRepository.findByIdOrNull(musicId) ?: throw MusicNotFoundException()
 
-        val key = "musicList:${music.createdDate.toLocalDate()}"
-
-        if(redisCacheService.getFromCache(key) != null) {
-            redisCacheService.deleteFromCache(key)
-        }
+        val date = music.createdDate.toLocalDate().toString()
 
+        redisCacheService.putToCacheMusic(date, MusicListResDto(mutableListOf()))
         musicRepository.delete(music)
         music.member.updateMusicStatus(MusicStatus.CAN)
     }

diff --git a/src/main/kotlin/com/dotori/v2/domain/music/service/impl/DeleteMyMusicServiceImpl.kt b/src/main/kotlin/com/dotori/v2/domain/music/service/impl/DeleteMyMusicServiceImpl.kt
@@ -6,6 +6,7 @@ import com.dotori.v2.domain.music.domain.entity.Music
 import com.dotori.v2.domain.music.domain.repository.MusicRepository
 import com.dotori.v2.domain.music.exception.MusicNotFoundException
 import com.dotori.v2.domain.music.exception.NotMyMusicException
+import com.dotori.v2.domain.music.presentation.data.res.MusicListResDto
 import com.dotori.v2.domain.music.service.DeleteMyMusicService
 import com.dotori.v2.global.config.redis.service.RedisCacheService
 import com.dotori.v2.global.util.UserUtil
@@ -28,12 +29,9 @@ class DeleteMyMusicServiceImpl(
 
         validMusic(music, member)
 
-        val key = "musicList:${music.createdDate.toLocalDate()}"
-
-        if(redisCacheService.getFromCache(key) != null) {
-            redisCacheService.deleteFromCache(key)
-        }
+        val date = music.createdDate.toLocalDate().toString()
 
+        redisCacheService.putToCacheMusic(date, MusicListResDto(mutableListOf()))
         musicRepository.delete(music)
         music.member.updateMusicStatus(MusicStatus.CAN)
     }

diff --git a/src/main/kotlin/com/dotori/v2/domain/music/service/impl/FindMusicsServiceImpl.kt b/src/main/kotlin/com/dotori/v2/domain/music/service/impl/FindMusicsServiceImpl.kt
@@ -23,7 +23,10 @@ class FindMusicsServiceImpl(
         val cachedData = redisCacheService.getFromCacheMusic(date.toString())
 
         if(cachedData != null) {
-            return cachedData as MusicListResDto
+            val response = cachedData as MusicListResDto
+            if(response.content.isNotEmpty()) {
+                return response
+            }
         }
 
         val response = MusicListResDto(