중복 모임 채팅방 생성 버그 픽스

[ksk0605]

PR의 목적이 무엇인가요?

새로운 채팅방을 생성할 때 targetId 와 ChatRoomType 을 기준으로 이미 존재하는 채팅방인지 검증하는 로직 누락으로 이미 존재하는 채팅방이 있음에도 중복 생성이 되는 문제 해결

이슈 ID는 무엇인가요?

• 중복 모임 채팅방 생성 버그 픽스 #751

설명

기존 ChatRoomWriter.append

public long append(long targetId, long darakbangId, ChatRoomType chatRoomType) {
ChatRoomEntity chatRoomEntity = ChatRoomEntity.builder()
	.targetId(targetId)
	.darakbangId(darakbangId)
	.type(chatRoomType)
	.build();
return chatRoomRepository.save(chatRoomEntity).getId();
}

변경된 ChatRoomWriter

private final ChatRoomValidator chatRoomValidator;

public long append(long targetId, long darakbangId, ChatRoomType chatRoomType) {
chatRoomValidator.validateAlreadyExists(targetId, chatRoomType);

ChatRoomEntity chatRoomEntity = ChatRoomEntity.builder()
	.targetId(targetId)
	.darakbangId(darakbangId)
	.type(chatRoomType)
	.build();
return chatRoomRepository.save(chatRoomEntity).getId();
}

검증 코드의 위치를 서비스가 아닌 구현레이어에 둔 이유

채팅방이 이미 존재하는지 검증하는 로직은 항상 생성하는 로직 이전에 이루어져야 한다고 판단하였습니다.

질문 혹은 공유 사항 (Optional)

[ay-eonii]

어푸루브 드릴게요

[hoyeonyy]

어푸푸르브 드릴꼐요