[COZY-449] fix: 쪽지 내용 앞뒤 공백 trim 처리

[veronees]

⚒️develop의 최신 커밋을 pull 받았나요?

네

#️⃣ 작업 내용

어떤 기능을 구현했나요?
기존 기능에서 어떤 점이 달라졌나요?
자세한 로직이 필요하다면 함께 적어주세요!
코드에 대한 설명이라면, 코맨트를 통해서 어떤 부분이 어떤 코드인지 설명해주세요!

• 쪽지 내용 앞뒤 공백 trim 처리

동작 확인

기능을 실행했을 때 정상 동작하는지 여부를 확인하고 스샷을 올려주세요

수정 전 쪽지방 목록 조회

수정 후 쪽지방 목록 조회

---

쪽지 작성 시

db 저장 상태 - trim 처리

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
고민사항도 적어주세요.

[github-actions]

리뷰해드려요~

• ChatCommandService.java

• 리뷰 1: 메세지 내용을 깨끗하게 처리하기 위해 trim() 메소드를 사용했습니다.

• 리뷰 2: 제거된 코드 - 아무것도 없음

• ChatRoomConverter.java

• 리뷰 1: 메세지 내용을 깨끗하게 처리하기 위해 trim() 메소드를 사용했습니다.

• 리뷰 2: 제거된 코드 - 아무것도 없음

추가적으로 제안하고 싶은 것은 없습니다. 위에서 변경된 부분은 코드의 일반적인 유지보수 및 개선 사항으로 보입니다.

[veronees]

조회에서 trim 처리는 db 밀게되면 지우겠습니다

[genius00hwan]

출시전에 지우면 되겠네여

[github-actions]

리뷰해드려요~

ChatCommandService.java

1. 변경사항: ChatConverter.toEntity(chatRoom, sender, content)에서 content에 대해 trim() 메소드를 사용하여 공백 문자를 제거하였습니다.

2. 개선 제안: 메세지 컨텐츠를 정규화하는 코드가 없습니다. 예를 들어 특수문자 또는 유저 정보를 포함하는 문자열을 제거하는 코드가 없습니다.

private void saveChat(ChatRoom chatRoom, Member sender, String content) {
    content = content.trim();
    content = content.replaceAll("[^a-zA-Z0-9가-힣]", ""); // 정규화 코드 추가
    Chat chat = ChatConverter.toEntity(chatRoom, sender, content);
    chatRepository.save(chat);
}

ChatRoomConverter.java

1. 변경사항: ChatRoomDetailResponseDTO.builder().lastContent(content)에서 content에 대해 trim() 메소드를 사용하여 공백 문자를 제거하였습니다.

2. 개선 제안: 메세지 컨텐츠를 정규화하는 코드가 없습니다. 예를 들어 특수문자 또는 유저 정보를 포함하는 문자열을 제거하는 코드가 없습니다.

public static ChatRoomDetailResponseDTO toChatRoomDetailResponseDTO(String nickname, String content, Long chatRoomId, Integer persona, Long memberId, boolean hasNewChat) {
    content = content.trim();
    content = content.replaceAll("[^a-zA-Z0-9가-힣]", ""); // 정규화 코드 추가
    return ChatRoomDetailResponseDTO.builder()
            .nickname(nickname)
            .lastContent(content)
            .chatRoomId(chatRoomId)
            .persona(persona)
            .memberId(memberId)
            .hasNewChat(hasNewChat)
            .build();
}

[genius00hwan]

LGTM 빠른 반영 수고하셨습니다

[jpark0506]

LGLGTM~