Spring Event Listener 기반 이벤트 처리 질문

Question

@Transactional public CommentCreateResponse create(final AuthPrincipal authPrincipal, final CommentCreateRequest request) { validateComponentId(request); validateParentId(request); final Comment comment = commentRepository.save(CommentCreationStrategy.createBy(request, authPrincipal.id())); eventPublisher.publishEvent(CommentCreateEvent.from(comment)); return CommentCreateResponse.from(comment); }@Async // 재시도, 예외 처리 필요 @TransactionalEventListener(phase = TransactionPhase.AFTER_COMMIT) public void handleCommentCreateEvent(final CommentCreateEvent event) { final Comment comment = findCommentById(event.commentId()); eventPublisher.publishEvent(CommentImageMoveEvent.from(comment)); eventPublisher.publishEvent(ComponentCommentCountIncreaseEvent.from(comment)); eventPublisher.publishEvent(CommentReplyCountIncreaseEvent.from(comment)); eventPublisher.publishEvent(CommentReplyNotificationEvent.from(comment)); }현재 댓글 생성 시 아래 4개 이벤트가 수행됩니다.컴포넌트(게시글)의 댓글 수 컬럼 1 증가 (ComponentCommentCountIncreaseEvent)댓글의 이미지 S3 업로드 (CommentImageMoveEvent)(대댓글인 경우) 부모 댓글의 대댓글 수 컬럼 1 증가 (CommentReplyCountIncreaseEvent)(대댓글인 경우) 부모 댓글 작성자에게 알림 (CommentReplyNotificationEvent)handleCommentCreateEvent() 메서드 내부에서 eventPublisher.publishEvent() 를 계속 호출하여 일일히 이벤트를 호출하는데 이를 컬렉션 + for 문 형태로 리펙토링하고 싶은데 감이 안잡히네요 ㅜㅜ

Xellos Choi · Accepted Answer

질문 주신 부분에 대해서 생각나는 방법을 추천해 드리자면이벤트를 해야 하는 일들을 enum class로 작성하시고 해당 enum을 기준으로 For 문을 돌면서 Reflection 또는 Factory 패턴으로 가져와 할 수 있을 꺼 같네요사족을 하나 달자면 굳이 위의 방법을 써야 하나 하는 의문이 있긴 하네요 이벤트가 매우 많이 늘어나거나(for 문을 많이 돌아야 할) 할 케이스가 그리 많아 보이지 않는데 추상화를 한다는 건 코드의 복잡도 만 높일 뿐일 것 같아요 4건정도의 이벤트라면 그냥 위의 처럼 처리하는게 코드가 더 읽기 쉬울 꺼 같아 보이네요

Xellos Choi · Answer

내부 이벤트들을 eventPublisher.publish()로 호출하고 있는데 차라리 Bean으로 등록해서 하나의 인터페이스 타입 컬렉션으로 관리하는건 어떻게 생각하시나요?내부적으로 이벤트에 등록된 4개의 서비스를 eventPublisher.publish 이용하셨다는 말은 아마도 4개의 서비스가 이미 존재하고 있는 서비스이며 현재 create 서비스와 관심을 분리(Create는 후 이후 4개의 서비스의 성공 여부와 관계없이 처리한다.)하기 위함으로 생각이 되는데요~ 그걸 Bean으로 등록해서 하나의 인터페이스 타입 컬렉션으로 관리한다는 이야기가 이해가 잘 안되네요ㅠㅠ이미 존재하는 bean들을 Map 이나 이런거에 담아서 사용하시겠다는건지? 정확한 내용의 이해가 잘 되지 않습니다."또한, 내부 이벤트가 추가될 가능성이 있다고 해도 지금처럼 유지하는게 좋다고 생각하시나요?": 위에서 말씀드렸다시피 만약에 이벤트가 추가 될 가능성이 있다고 한다면 개인적으로는 Enum으로 관리하시는 걸 추천 드리긴 합니다만 ( properties로 관리해도 되고, Custom Annotation으로 관리해도 되고 관리의 방법은 여러가지가 있을 수 있습니다.)위의 관리에 대한 내용은 Coding Convention에 대한 내용에 가깝기에 프로젝트를 함께 하시는 분들과 의사결정을 하시는게 좋을 것 같네요

Spring Event Listener 기반 이벤트 처리 질문

2개의 답변

Q&A 목록