회원 탈퇴 시 bad SQL grammar 오류 발생 이슈 해결

[LJW25]

📄 Summary

#797

🙋🏻 More

슬랙에 오류 알림 떴길래..그냥 간단히 예외처리 추가함..

여행 없으면 삭제 요청하지 않도록 if문 추가했습니다.
이 과정에서 애매한게 생겼는데 MemberDeleteEvent도 발행을 안하는게 맞는건지?
MemberDeleteEvent라는 네이밍만 놓고 보면 여행 존재 여부와 상관없이 계정이 삭제되면 발행되야할것같은데, 실제로 해당 이벤트가 발행되었을 때 동작하는 것은 해당 계정의 여행 삭제밖에 없어서, 그 코드에서 오류가 납니다.
즉 여행이 없으면 해당 이벤트를 발행하지 않아야 오류가 발생하지 않아요.
예외처리를 이벤트를 받는쪽에서 해도 되긴 한데, 그럼 이벤트 발행 이유 자체가 없어져서 낭비라고 판단했습니다.

네이밍을 바꿀까요? 아니면 다른 좋은 의견 있으시면 남겨주시면 감사하겠습니다.

[github-actions]

📝 Jacoco Test Coverage

Total Project Coverage	81.76%	🍏

File	Coverage [20.09%]
LoginService.java	20.09%	❌

[jjongwa]

흠 저는 일단 이벤트 발행하고 리스너에서 처리하는게 낫다는 생각이긴 한데
(Member Delete할때 다른 처리가 추가적으로 필요해질 수도 있으니)

앞으로 이쪽 부분이 크게 변경될 일은 없을거 같아서 네이밍 변경도 좋습니다.

[mcodnjs]

음음 .. 저도 리스너에서 처리하는 거에 조금 더 선호가 있긴 한데 ㅎㅎ.. 크게 상관없을거 같습니다!
만약 네이밍을 변경한다하면, 어떤 네이밍이 좋을까요? (정말 몰라서)

도메인 이벤트 발행 (멤버 삭제) -> 발행된 이벤트를 통한 상태 변화 (여행들 삭제)

여기에 기인했을 때 MemberDeleteEvent를 벗어난다면 조오금 어색할 수도 있을 것 같다는 생각이 들었어용

[hgo641]

저도 MemberDeleteEvent 그대로 발행하고 리스너 측에서 멤버가 생성한 trip이 없다면 무시하는 쪽으로 가는 게 좋을 것 같아요!!ㅎㅎ 이유는 디노랑 같습니닷