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

안녕하세요 질문 감사드립니다

우선 현재 예제 기준으로 couponMap 은 상태를 포함하여 조회(findAllById 사용)하고 있지 않습니다!
그래서 적어주신 것 처럼 어드민에서 상태를 변경한다 하여도 예외가 발생하지 않는 형태입니다

다만 누군가 HardDelete를 해버리면 문제는 발생 할 수 있긴합니다만, 이건 의도가 아닌 이상 발생하지 않게 잘 관리 해야할 것 같습니다 (디비 권한 부터..)

별개로 적어주신 것 기준으로 보면 기획 의도와 비즈니스 상황에 따라 다르다고 봐야할 것 같습니다
다만 기획과 운영팀과 협의하여 기준을 명확히 정리해야하는 부분이라고 생각합니다

1번의 경우 현실적으로 운영을 하다보면 운영팀 입장에선 상태 변경이 필요하다보니 불변을 강제하기 어려울 수 있을 것 같습니다 😢 (외부 가맹점에서 클레임이 있을 수도 있구요, 정말 설정 오류일 수도 있구요)

2번의 경우 쿠폰을 비활성화 했다고 해서 소유 쿠폰이 제거되면 안 될 것 같습니다! 이건 고객 입장에서 민원으로 직결 될 수 있는 부분 같습니다

결과적으로 두개의 선택지 모두 다시 재고해봐야하는 전략 같습니다
그래서 현재 예제에서는 무조건 Coupon 은 메타데이터 성으로 상태 상관없이 가져오도록 되어있습니다

다만 강의에서 설명했듯 이런 부분이 애매하다 판단되거나 이슈가 있다면 OwnedCoupon 이 메타 데이터를 직접 가지고 있게하는 것도 방법이 될 수 있을 것 같습니다!

모쪼록 답이 되었길 바랍니다! 감사합니다!

안녕하세요, 인프런 AI 인턴입니다. 비오님!

쿠폰 데이터를 처리하면서 !! 연산자를 사용하는 부분에 대한 질문을 주셨네요. !! 연산자는 Kotlin에서 null 가능성을 명시적으로 제거하기 위해 사용되지만, 실제로 null이 발생할 가능성이 있으면 오류가 발생할 수 있는 위험이 있는 방식입니다.

고려해야 할 사항

1. 기획의 일관성 유지:

   • 기획에 따라 쿠폰의 상태가 변하지 않는다는 전제가 있다면, !! 연산자를 사용하는 것은 문제없을 수 있습니다. 하지만 이 전제가 깨질 가능성이 있다면, 오류를 방지하기 위한 보완이 필요합니다.
     

2. 쿠폰의 상태 변화:

   • 쿠폰이 비활성화되거나 상태가 변경될 가능성이 있는 경우, 소유 쿠폰에서도 이 변경 사항을 반영하는 것이 안전한 설계입니다.
     
   • 기획에 따라 쿠폰의 비활성화 기준을 명확히 하고, 비활성화 시 소유 쿠폰에서 어떻게 처리할지를 정의해 두는 것이 좋습니다.
     

3. 비즈니스 로직의 명확화:

   • 모든 소유 쿠폰을 불변 처리하여 변경되지 않도록 통제하는 방법이 운영상 관리하기 쉬울 수 있다면 이 방향으로 진행할 수 있습니다. 다만, 이는 기능적으로 제한을 가할 수 있습니다.
     

관련 예제 및 해결 방법

• 쿠폰 관리 방법에 따라 다양한 구현이 가능합니다.
  
  • CouponService에서 이미 다운로드 한 쿠폰 안 내려주기 - 쿠폰의 상태 및 사용자 소유 쿠폰 관리에 대한 논의 예제를 참고하면 좋습니다.
    
  • Controller에서 비즈니스 로직 흐름이 나타나는 것에 대하여.. - 비즈니스 로직의 흐름을 어떻게 관리할지에 대한 논의가 있어 도움이 될 수 있습니다.
    

현재 상황에서 비즈니스 로직에 맞는 최적의 해결책은 각 서비스의 요구사항에 따라 달라질 수 있음을 유념하셔야 합니다. 각각의 선택지는 나름의 장단점이 있으니 기획 및 요구사항과 맞추어 결정하는 것이 중요합니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.