Skip to content

fix : 체크리스트 요청 반려 시 더이상 요청을 보낼 수 없던 문제 해결 #313

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.

Sign up for GitHub

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

Closed
Eseas wants to merge 1 commit into from
Closed

fix : 체크리스트 요청 반려 시 더이상 요청을 보낼 수 없던 문제 해결 #313

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Common/src/main/java/com/seveneleven/entity/project/Checklist.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ public Checklist deleteChecklist() {
}

public void setChecklistApplication() {
if(checklistStatus == ChecklistStatus.APPLICATION_WAITING) {
if(checklistStatus == ChecklistStatus.APPLICATION_WAITING || checklistStatus == ChecklistStatus.REJECTED) {
checklistStatus = ChecklistStatus.APPROVE_WAITING;
return;
}
Copy link

@github-actions github-actions bot Feb 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드 패치에 대한 간략한 리뷰를 제공하겠습니다.

  1. 버그 위험:

    • setChecklistApplication 메서드는 checklistStatus가 APPLICATION_WAITING 또는 REJECTED일 경우에만 상태를 APPROVE_WAITING으로 변경합니다. 이러면 오히려 REJECTED 상태에서도 APPROVE_WAITING으로 상태가 변경될 수 있다는 점이 문제입니다. 만약 REJECTED 상태에서 승인 대기 상태로 넘어가는 것이 비즈니스 로직에 맞지 않다면, 이는 치명적인 오류를 유발할 수 있습니다.

  2. 개선 제안:

    • REJECTED 상태에서의 처리 로직을 명확히 해야 합니다. 예를 들어, REJECTED 상태일 때는 로그를 남기거나 예외를 던지는 등의 조치를 고려할 수 있습니다.
    • 메서드의 명칭이나 주석을 추가하여 이 메서드의 의도를 더욱 명확히 할 수 있습니다. 현재 메서드 이름만으로는 어떤 조건에서 상태를 변경하는지 명확하지 않습니다.
    • 나중에 유지보수나 기능 확장성을 고려해 상태 전이가 가능한 세부 규칙을 분리하여 관리하는 방법도 고려해볼 수 있습니다.

  3. 스타일:

    • 전반적으로 코드 스타일은 깔끔하지만, 조건문의 가독성을 높이기 위해 괄호를 추가하는 것도 좋은 습관입니다.

결론적으로, 코드 변경이 비즈니스 로직에 어떤 영향을 미치는지 충분히 검토한 후, 추가 테스트를 통해 예상되는 문제를 방지하는 것이 중요합니다.

Expand Down
Loading