Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[COZY-432] hotfix: isExit 삭제 #208

Merged
merged 1 commit into from
Dec 3, 2024
Merged

[COZY-432] hotfix: isExit 삭제 #208

merged 1 commit into from
Dec 3, 2024

Conversation

suuu0719
Copy link
Contributor

@suuu0719 suuu0719 commented Dec 3, 2024

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

#️⃣ 작업 내용

동작 확인

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

💬 리뷰 요구사항(선택)

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

@suuu0719 suuu0719 self-assigned this Dec 3, 2024
@suuu0719 suuu0719 merged commit 5cc39ba into develop Dec 3, 2024
@suuu0719 suuu0719 deleted the hotfix/COZY-432 branch December 3, 2024 17:55
Copy link

github-actions bot commented Dec 3, 2024

리뷰해드려요~

자동 메시지 처리 서비스 {name}

  • 입장 상태 변경 메소드에서 setNotExit() 메소드 제거
public void joinRoom(Long roomId, Long memberId) {
    // ...
    exitingMate.setEntryStatus(EntryStatus.JOINED);
    mateRepository.save(exitingMate);
    room.arrive();
    room.isRoomFull();
}
  • 초대 상태 변경 메소드에서 setNotExit() 메소드 제거
public void sendInvitation(Long inviteeId, Long inviterId) {
    // ...
    mate.setEntryStatus(EntryStatus.INVITED);
    mateRepository.save(mate);
}
  • 입장 요청 변경 메소드에서 setNotExit() 메소드 제거
public void requestToJoin(Long roomId, Long memberId) {
    // ...
    mate.setEntryStatus(EntryStatus.PENDING);
    mateRepository.save(mate);
}
  • isExit 변수 제거
public class Mate extends BaseTimeEntity {
    // ...
    // 제거: private boolean isExit = false;
    // ...
}
  • setExit() 메소드 제거
public void quit() {
    // 제거: this.isExit = true;
    setEntryStatus(EntryStatus.EXITED);
}
  • setNotExit() 메소드 제거
public void setNotExit() {
    // 제거: this.isExit = false;
}

이러한 변경 사항은 isExit 변수와 관련된 메소드를 제거하여 코드를 간결하게 만들고 입장 상태 변경 메소드에서 추가 변경 메소드 호출을 피할 수 있도록 합니다.

eple0329 pushed a commit that referenced this pull request Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant