[#198/fix] in의 잘못된 사용으로 마지막 학습지를 받지 않은 구독자들도 구독을 해지하는 문제 해결

[belljun3395]

🎫 연관 이슈

resolved: #198

💁‍♂️ PR 내용

• in의 잘못된 사용으로 마지막 학습지를 받지 않은 구독자들도 구독을 해지하는 문제 해결

🙏 작업

• in의 잘못된 사용으로 마지막 학습지를 받지 않은 구독자들도 구독을 해지하는 문제를 발견하고 수정했습니다.

🙈 PR 참고 사항

📸 스크린샷

in 사용 쿼리 로그

in 사용 쿼리 결과

🤖 테스트 체크리스트

• 체크 미완료
• 체크 완료

[hun-ca]

이 파일엔 서비스도 있고 dto도 있고, 다 뭉쳐둔 이유가 뭔가요?

[belljun3395]

서비스는 이메일을 보내기 위해서 필요하고
dto라고 생각하시는 거는 이메일 전송 배치 라이터에서만 사용되는 모델이라서 함께 있습니다.
라이터에서만 사용돼서 함께 있어도 괜찮다고 생각하는데 분리하는 게 좋을까요?

[hun-ca]

서비스는 이메일을 보내기 위해서 필요하고 dto라고 생각하시는 거는 이메일 전송 배치 라이터에서만 사용되는 모델이라서 함께 있습니다. 라이터에서만 사용돼서 함께 있어도 괜찮다고 생각하는데 분리하는 게 좋을까요?

패키지 명은 서비스고 클래스는 라이터여서 약간 헷갈렸음

라이터에서만 사용하는거면 그냥 둬도 될거 같아요 근데 서비스라는 네이밍이 어울리나?

[belljun3395]

스프링 배치를 사용하면 라이터랑 리더 프로세서를 구현해야하는데 그때도 지금 리더 라이터 최대한 그대로 사용할 수 있도록 구현했고 그때는 서비스도 어울릴꺼 같슴다!

지금은 사진에 보이는 서비스를 컨트롤러에서 사용하고 있어요

[hun-ca]

스프링 배치를 사용하면 라이터랑 리더 프로세서를 구현해야하는데 그때도 지금 리더 라이터 최대한 그대로 사용할 수 있도록 구현했고 그때는 서비스도 어울릴꺼 같슴다!

지금은 사진에 보이는 서비스를 컨트롤러에서 사용하고 있어요

우리 지금 배치에는 UC 네이밍을 안쓰고 있죠

[belljun3395]

네네 배치는 UC는 아닌거 같아서요

[hun-ca]

Approve