feat: 내서제 도서 삭제 기능을 개발합니다. (#95)

minwoo1999 · web-flow · commit a27f21488341 · 2025-08-16T13:29:00.000+09:00
* feat: apis - 내서재 삭제 기능 개발(#92) * feat: domain - 내서재 삭제 기능 개발(#92) * feat: infra - 내서재 삭제 기능 개발(#92) * [BOOK-261] refactor: apis,domain recordingDomainservice 분리 (#95) * [BOOK-261] refactor: aps - 책 기록 먼저 삭제(#95)
diff --git a/apis/src/main/kotlin/org/yapp/apis/book/controller/BookController.kt b/apis/src/main/kotlin/org/yapp/apis/book/controller/BookController.kt
@@ -6,8 +6,10 @@ import org.springframework.data.domain.Sort
 import org.springframework.data.web.PageableDefault
 import org.springframework.http.ResponseEntity
 import org.springframework.security.core.annotation.AuthenticationPrincipal
+import org.springframework.web.bind.annotation.DeleteMapping
 import org.springframework.web.bind.annotation.GetMapping
 import org.springframework.web.bind.annotation.ModelAttribute
+import org.springframework.web.bind.annotation.PathVariable
 import org.springframework.web.bind.annotation.PutMapping
 import org.springframework.web.bind.annotation.RequestBody
 import org.springframework.web.bind.annotation.RequestMapping
@@ -70,4 +72,13 @@ class BookController(
         return ResponseEntity.ok(response)
     }
 
+    @DeleteMapping("/my-library/{userBookId}")
+    override fun deleteBookFromMyLibrary(
+        @AuthenticationPrincipal userId: UUID,
+        @PathVariable userBookId: UUID,
+    ): ResponseEntity<Unit> {
+        bookUseCase.deleteBookFromMyLibrary(userId, userBookId)
+        return ResponseEntity.noContent().build()
+    }
+
 }
diff --git a/apis/src/main/kotlin/org/yapp/apis/book/controller/BookControllerApi.kt b/apis/src/main/kotlin/org/yapp/apis/book/controller/BookControllerApi.kt
@@ -140,4 +140,24 @@ interface BookControllerApi {
         @PageableDefault(size = 10, sort = ["createdAt"], direction = Sort.Direction.DESC)
         pageable: Pageable
     ): ResponseEntity<UserBookPageResponse>
+
+    @Operation(summary = "내 서재에 저장한 도서 삭제", description = "내 서재에 저장한 도서를 삭제합니다.")
+    @ApiResponses(
+        value = [
+            ApiResponse(
+                responseCode = "204",
+                description = "성공적으로 도서를 삭제했습니다."
+            ),
+            ApiResponse(
+                responseCode = "404",
+                description = "해당하는 도서를 찾을 수 없습니다.",
+                content = [Content(schema = Schema(implementation = ErrorResponse::class))]
+            )
+        ]
+    )
+    @DeleteMapping("/my-library/{userBookId}")
+    fun deleteBookFromMyLibrary(
+        @AuthenticationPrincipal userId: UUID,
+        @Parameter(description = "삭제할 도서 ID") @PathVariable userBookId: UUID
+    ): ResponseEntity<Unit>
 }
diff --git a/apis/src/main/kotlin/org/yapp/apis/book/service/UserBookService.kt b/apis/src/main/kotlin/org/yapp/apis/book/service/UserBookService.kt
@@ -83,4 +83,9 @@ class UserBookService(
             completedCount = userBookStatusCountsVO.completedCount
         )
     }
+
+    fun deleteUserBook(userBookId: UUID, userId: UUID) {
+        validateUserBookExists(userBookId, userId)
+        userBookDomainService.deleteById(userBookId)
+    }
 }
diff --git a/apis/src/main/kotlin/org/yapp/apis/book/usecase/BookUseCase.kt b/apis/src/main/kotlin/org/yapp/apis/book/usecase/BookUseCase.kt
@@ -13,6 +13,7 @@ import org.yapp.apis.book.dto.response.UserBookResponse
 import org.yapp.apis.book.service.BookManagementService
 import org.yapp.apis.book.service.BookQueryService
 import org.yapp.apis.book.service.UserBookService
+import org.yapp.apis.readingrecord.service.ReadingRecordService
 import org.yapp.apis.user.service.UserService
 import org.yapp.domain.userbook.BookStatus
 import org.yapp.domain.userbook.UserBookSortType
@@ -26,7 +27,8 @@ class BookUseCase(
     private val bookQueryService: BookQueryService,
     private val userService: UserService,
     private val userBookService: UserBookService,
-    private val bookManagementService: BookManagementService
+    private val bookManagementService: BookManagementService,
+    private val readingRecordService: ReadingRecordService
 ) {
     fun searchBooks(
         request: BookSearchRequest
@@ -84,6 +86,16 @@ class BookUseCase(
         return userBookService.findUserBooksByDynamicConditionWithStatusCounts(userId, status, sort, title, pageable)
     }
 
+    @Transactional
+    fun deleteBookFromMyLibrary(
+        userId: UUID,
+        userBookId: UUID
+    ) {
+        userService.validateUserExists(userId)
+        readingRecordService.deleteAllByUserBookId(userBookId)
+        userBookService.deleteUserBook(userBookId, userId)
+    }
+
     private fun mergeWithUserBookStatus(
         searchedBooks: List<BookSummary>,
         userId: UUID?
diff --git a/apis/src/main/kotlin/org/yapp/apis/readingrecord/service/ReadingRecordService.kt b/apis/src/main/kotlin/org/yapp/apis/readingrecord/service/ReadingRecordService.kt
@@ -47,6 +47,9 @@ class ReadingRecordService(
         return page.map { ReadingRecordResponse.from(it) }
     }
 
+    fun deleteAllByUserBookId(userBookId: UUID) {
+        readingRecordDomainService.deleteAllByUserBookId(userBookId)
+    }
     fun updateReadingRecord(
         readingRecordId: UUID,
         request: UpdateReadingRecordRequest
diff --git a/apis/src/main/kotlin/org/yapp/apis/readingrecord/usecase/ReadingRecordUseCase.kt b/apis/src/main/kotlin/org/yapp/apis/readingrecord/usecase/ReadingRecordUseCase.kt
@@ -1,6 +1,5 @@
 package org.yapp.apis.readingrecord.usecase
 
-import org.springframework.data.domain.Page
 import org.springframework.data.domain.Pageable
 import org.springframework.transaction.annotation.Transactional
 import org.yapp.apis.book.service.UserBookService
diff --git a/domain/src/main/kotlin/org/yapp/domain/readingrecord/ReadingRecordDomainService.kt b/domain/src/main/kotlin/org/yapp/domain/readingrecord/ReadingRecordDomainService.kt
@@ -146,6 +146,9 @@ class ReadingRecordDomainService(
         return buildReadingRecordInfoVO(savedReadingRecord)
     }
 
+    fun deleteAllByUserBookId(userBookId: UUID) {
+        readingRecordRepository.deleteAllByUserBookId(userBookId)
+    }
     fun deleteReadingRecord(readingRecordId: UUID) {
         val readingRecord = readingRecordRepository.findById(readingRecordId)
             ?: throw ReadingRecordNotFoundException(
diff --git a/domain/src/main/kotlin/org/yapp/domain/readingrecord/ReadingRecordRepository.kt b/domain/src/main/kotlin/org/yapp/domain/readingrecord/ReadingRecordRepository.kt
@@ -9,6 +9,7 @@ interface ReadingRecordRepository {
 
     fun save(readingRecord: ReadingRecord): ReadingRecord
 
+    fun deleteAllByUserBookId(userBookId: UUID)
 
     fun findById(id: UUID): ReadingRecord?
 
diff --git a/domain/src/main/kotlin/org/yapp/domain/userbook/UserBookDomainService.kt b/domain/src/main/kotlin/org/yapp/domain/userbook/UserBookDomainService.kt
@@ -6,11 +6,12 @@ import org.yapp.domain.userbook.vo.HomeBookVO
 import org.yapp.domain.userbook.vo.UserBookInfoVO
 import org.yapp.domain.userbook.vo.UserBookStatusCountsVO
 import org.yapp.globalutils.annotation.DomainService
+import org.yapp.domain.readingrecord.ReadingRecordRepository
 import java.util.*
 
 @DomainService
 class UserBookDomainService(
-    private val userBookRepository: UserBookRepository
+    private val userBookRepository: UserBookRepository,
 ) {
     fun upsertUserBook(
         userId: UUID,
@@ -77,6 +78,10 @@ class UserBookDomainService(
         return userBookRepository.existsByIdAndUserId(userBookId, userId)
     }
 
+    fun deleteById(userBookId: UUID) {
+        userBookRepository.deleteById(userBookId)
+    }
+
     fun findBooksWithRecordsOrderByLatest(userId: UUID): List<HomeBookVO> {
         val resultTriples = userBookRepository.findRecordedBooksSortedByRecency(userId)
 
@@ -103,7 +108,8 @@ class UserBookDomainService(
         return userBooks.map { userBook ->
             HomeBookVO.newInstance(
                 userBook = userBook,
-                lastRecordedAt = userBook.updatedAt ?: throw IllegalStateException("UserBook의 updatedAt이 null입니다: ${userBook.id}"),
+                lastRecordedAt = userBook.updatedAt
+                    ?: throw IllegalStateException("UserBook의 updatedAt이 null입니다: ${userBook.id}"),
                 recordCount = 0
             )
         }
diff --git a/domain/src/main/kotlin/org/yapp/domain/userbook/UserBookRepository.kt b/domain/src/main/kotlin/org/yapp/domain/userbook/UserBookRepository.kt
@@ -12,6 +12,7 @@ interface UserBookRepository {
     fun existsByIdAndUserId(id: UUID, userId: UUID): Boolean
     fun findById(id: UUID): UserBook?
     fun save(userBook: UserBook): UserBook
+    fun deleteById(id: UUID)
     fun findAllByUserId(userId: UUID): List<UserBook>
     fun findAllByUserIdAndBookIsbn13In(userId: UUID, bookIsbn13s: List<String>): List<UserBook>
     fun findUserBooksByDynamicCondition(
diff --git a/infra/src/main/kotlin/org/yapp/infra/readingrecord/repository/JpaReadingRecordRepository.kt b/infra/src/main/kotlin/org/yapp/infra/readingrecord/repository/JpaReadingRecordRepository.kt
@@ -11,6 +11,8 @@ interface JpaReadingRecordRepository : JpaRepository<ReadingRecordEntity, UUID>,
 
     fun findAllByUserBookId(userBookId: UUID): List<ReadingRecordEntity>
 
+    fun deleteAllByUserBookId(userBookId: UUID)
+
     fun findAllByUserBookId(userBookId: UUID, pageable: Pageable): Page<ReadingRecordEntity>
 
     fun findAllByUserBookIdIn(userBookIds: List<UUID>): List<ReadingRecordEntity>
diff --git a/infra/src/main/kotlin/org/yapp/infra/readingrecord/repository/impl/ReadingRecordRepositoryImpl.kt b/infra/src/main/kotlin/org/yapp/infra/readingrecord/repository/impl/ReadingRecordRepositoryImpl.kt
@@ -21,6 +21,10 @@ class ReadingRecordRepositoryImpl(
         return savedEntity.toDomain()
     }
 
+    override fun deleteAllByUserBookId(userBookId: UUID) {
+        jpaReadingRecordRepository.deleteAllByUserBookId(userBookId)
+    }
+
     override fun findById(id: UUID): ReadingRecord? {
         return jpaReadingRecordRepository.findByIdOrNull(id)?.toDomain()
     }
diff --git a/infra/src/main/kotlin/org/yapp/infra/userbook/repository/impl/UserBookRepositoryImpl.kt b/infra/src/main/kotlin/org/yapp/infra/userbook/repository/impl/UserBookRepositoryImpl.kt
@@ -38,6 +38,10 @@ class UserBookRepositoryImpl(
         return savedEntity.toDomain()
     }
 
+    override fun deleteById(id: UUID) {
+        jpaUserBookRepository.deleteById(id)
+    }
+
     override fun findAllByUserId(userId: UUID): List<UserBook> {
         return jpaUserBookRepository.findAllByUserId(userId).map { it.toDomain() }
     }

Original file line number	Diff line number	Diff line change
`@@ -83,4 +83,9 @@ class UserBookService(`
`83`	`83`	`completedCount = userBookStatusCountsVO.completedCount`
`84`	`84`	`)`
`85`	`85`	`}`
	`86`	`+`
	`87`	`+ fun deleteUserBook(userBookId: UUID, userId: UUID) {`
	`88`	`+ validateUserBookExists(userBookId, userId)`
	`89`	`+ userBookDomainService.deleteById(userBookId)`
	`90`	`+ }`
`86`	`91`	`}`
Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,9 @@ class ReadingRecordService(`
`47`	`47`	`return page.map { ReadingRecordResponse.from(it) }`
`48`	`48`	`}`
`49`	`49`
	`50`	`+ fun deleteAllByUserBookId(userBookId: UUID) {`
	`51`	`+ readingRecordDomainService.deleteAllByUserBookId(userBookId)`
	`52`	`+ }`
`50`	`53`	`fun updateReadingRecord(`
`51`	`54`	`readingRecordId: UUID,`
`52`	`55`	`request: UpdateReadingRecordRequest`
Original file line number	Diff line number	Diff line change
`@@ -146,6 +146,9 @@ class ReadingRecordDomainService(`
`146`	`146`	`return buildReadingRecordInfoVO(savedReadingRecord)`
`147`	`147`	`}`
`148`	`148`
	`149`	`+ fun deleteAllByUserBookId(userBookId: UUID) {`
	`150`	`+ readingRecordRepository.deleteAllByUserBookId(userBookId)`
	`151`	`+ }`
`149`	`152`	`fun deleteReadingRecord(readingRecordId: UUID) {`
`150`	`153`	`val readingRecord = readingRecordRepository.findById(readingRecordId)`
`151`	`154`	`?: throw ReadingRecordNotFoundException(`
Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,7 @@ interface ReadingRecordRepository {`
`9`	`9`
`10`	`10`	`fun save(readingRecord: ReadingRecord): ReadingRecord`
`11`	`11`
	`12`	`+ fun deleteAllByUserBookId(userBookId: UUID)`
`12`	`13`
`13`	`14`	`fun findById(id: UUID): ReadingRecord?`
`14`	`15`