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

안녕하세요 선생님 !
섹션 6. 이전 메시지 조회 강의에서 질문 있습니다.

1. 선생님 코드의 getChatHistory() 메서드에서
chatParticipants를 chatRoom으로 조회해서
for문을 돌면서 현재 로그인한 멤버가 chatParticipants에 있는지 확인하고 있는데,
findByChatRoomAndMember 메서드를 통해
참여자인지 확인해도 로직에 상관이 없을까요 ?

2. 현재 로직은 getHistory 메서드가 실행되면
해당 참여자가 채팅방에 참여하기 전의 메시지들까지 전부 표시될것 같은데, 참여자가 채팅방에 참여한 후 부터의 메시지만 보여주고 싶으면 참여시간을 따로 저장한 다음 findByChatRoomAndCreatedTimeGreaterThanEqualOrderByCreatedTimeAsc(ChatRoom chatRoom, LocalDateTime participatedTime);
이런식으로 코드를 짜면 될까요 ? 좋은 방법인지 모르겠습니다.

선생님 코드
public List<ChatMessageDto> getChatHistory(Long roomId){

// 내가 해당 채팅방의 참여자가 아닐경우 에러

ChatRoom chatRoom = chatRoomRepository.findById(roomId).orElseThrow(()-> new EntityNotFoundException("room cannot be found"));

Member member = memberRepository.findByEmail(SecurityContextHolder.getContext().getAuthentication().getName()).orElseThrow(()->new EntityNotFoundException("member cannot be found"));

List<ChatParticipant> chatParticipants = chatParticipantRepository.findByChatRoom(chatRoom);

boolean check = false;

for(ChatParticipant c : chatParticipants){

if(c.getMember().equals(member)){

check = true;

}

}

if(!check)throw new IllegalArgumentException("본인이 속하지 않은 채팅방입니다.");

// 특정 room에 대한 message조회

List<ChatMessage> chatMessages = chatMessageRepository.findByChatRoomOrderByCreatedTimeAsc(chatRoom);

List<ChatMessageDto> chatMessageDtos = new ArrayList<>();

for(ChatMessage c : chatMessages){

ChatMessageDto chatMessageDto = ChatMessageDto.builder()

.message(c.getContent())

.senderEmail(c.getMember().getEmail())

.build();

chatMessageDtos.add(chatMessageDto);

}

return chatMessageDtos;

}