인프런 커뮤니티 질문&답변

친절한 답변 감사드립니다! 조언해주신 것 바탕으로 코드를 수정해봤습니다 ㅎㅎ

• UserOrGuestResolver 클래스 만들기 (UserResolver 참고)

• UserOrGuest 개념 객체 만들기: UserOrGuestResolver에서 사용 + 앞으로 인증/미인증 사용자 모두에 대해서 필요할 것 같아서 만들어 봤습니다. 처음에는 어노테이션으로 만들었다가 개념 자체가 다른 요구사항에서도 자주 사용될 수도 있을 것 같아 클래스로 만드는 것으로 변경을 했네요.

• UserService 코드 수정하기

fun getCouponsForProducts(productIds: Collection<Long>, user: UserOrGuest): List<Coupon> {
        val productTargets = couponTargetRepository.findByTargetTypeAndTargetIdInAndStatus(
            CouponTargetType.PRODUCT,
            productIds,
            EntityStatus.ACTIVE,
        )
        val categoryTargets = couponTargetRepository.findByTargetTypeAndTargetIdInAndStatus(
            CouponTargetType.PRODUCT_CATEGORY,
            productCategoryRepository.findByProductIdInAndStatus(productIds, EntityStatus.ACTIVE).map { it.categoryId },
            EntityStatus.ACTIVE,
        )

        val applicableCouponIds = (productTargets + categoryTargets).map { it.couponId }.toSet()

        // 비로그인 사용자의 경우, 모든 쿠폰을 반환
        if (user.isGuest) {
            return couponRepository.findByIdInAndStatus(applicableCouponIds, EntityStatus.ACTIVE)
                .map {
                    Coupon(
                        id = it.id,
                        name = it.name,
                        type = it.type,
                        discount = it.discount,
                        expiredAt = it.expiredAt,
                    )
                }
        }

        val downloadedCouponIds = ownedCouponRepository.findByUserIdAndStatusAndState(
            user.userId!!,
            EntityStatus.ACTIVE,
            OwnedCouponState.DOWNLOADED,
        )
            .map { it.couponId }
            .toSet()

        val finalCouponIds = applicableCouponIds - downloadedCouponIds

        if (finalCouponIds.isEmpty()) {
            return emptyList()
        }

        return couponRepository.findByIdInAndStatus(finalCouponIds, EntityStatus.ACTIVE)
            .map {
                Coupon(
                    id = it.id,
                    name = it.name,
                    type = it.type,
                    discount = it.discount,
                    expiredAt = it.expiredAt,
                )
            }
    }

최종적으로 CouponService.getCouponsForProducts 의 코드는 이렇게 생겼는데요. 코드를 짜고 보니깐 구현 부분이 계속 들어가 있어서 정확히 이 메서드가 어떤 일을 하는지 한 번에 파악이 쉽지 않은 것 같다는 느낌이 들기도 하더라구요. 이럴 때 재민 님이 말씀하신 Implement Layer를 도입해봐도 괜찮을 것 같다는 생각이 들었습니다😀

많이 고민해보시고 코드도 작성 해보셨군요!

적어주신 내용도 고민해보셔도 좋을 것 같습니다!

다른 관점 고민 포인트로 툭툭 던져드리면 요런 포인트가 있을듯하네요ㅎㅎ

• user.isGuest 가 비즈니스 구현 코드에 분기로 침투 된게 아쉽지 않나? 다른 전략은 무엇이 있을까 + 비슷한 관점으로 UserOrGuest 를 비즈니스 구현 코드에서 알게하는게 맞을까!?

   

물론 별개로 지금 작성해주신 코드에서 구현영역을 도구로 나눠보시면 일차 정리는 깔끔해보입니다!

모쪼록 남은 강의도 화이팅입니다!