fix : post project step order fix

[Eseas]

바로 MERGE하지 마세요! 검토 후 MERGE해야합니다!

✅ 최근 작업 주제 (하나 이상의 주제를 선택해주세요.)

• 기능 추가
• 리뷰 반영 수정
• 리팩토링
• 버그 수정
• 컨벤션 수정
• 브랜치 최신화

🏆 구현 목표

• 프로젝트 단계 등록 시 순서 +1로 저장되는 오류 수정

🔍 테스트 방법 (변경 사항을 확인하기 위한 테스트 방법을 기술합니다.)

• 스웨거 테스트

[github-actions]

Code review by ChatGPT

[github-actions]

코드 리뷰에 대해 몇 가지 피드백을 드리겠습니다.

1. 버그 리스크:

   • requestDto.getStepOrderNumber() - 1 변경으로 인해 stepOrderNumber가 0일 경우 getStepOrder 메서드는 -1을 사용하는데, 이 경우 인덱스 범위를 초과할 수 있습니다. orderList의 크기에 따라 NullPointerException이나 ArrayIndexOutOfBoundsException이 발생할 수 있습니다.

2. 유효성 검사:

   • requestDto.getStepOrderNumber()가 0보다 작은 경우에 대한 유효성 검사가 필요합니다. 이를 통해 예외 상황을 방지할 수 있습니다.
   • 예를 들어:

     Integer stepOrderNumber = requestDto.getStepOrderNumber();
     if (stepOrderNumber <= 0) {
         throw new BusinessException(ErrorCode.INVALID_STEP_ORDER);
     }

3. 주석:

   • 이러한 수정 사항에 대해 주석을 추가하여 코드의 가독성을 높이는 것이 좋습니다. 변경한 이유와 목적을 명확히 하여 추후 유지보수 시 혼동을 줄일 수 있습니다.

4. 메서드 이름:

   • getStepOrder 메서드의 구현이 어떻게 되어 있는지에 따라 몇 가지 개선이 가능합니다. 만약 메서드가 특정 인덱스에 대한 기본값을 리턴한다면 해당 기능이 명확하게 드러나도록 메서드 이름을 변경하는 것도 고려해볼 수 있습니다.

5. 예외 처리:

   • 오류 코드 세트에 ErrorCode.INVALID_STEP_ORDER와 같은 새로운 예외 처리를 추가하면 사용자가 문제를 더 쉽게 이해할 수 있도록 도와줄 수 있습니다.

이와 같은 점들을 고려하여 코드를 수정하면 더욱 안정적이고 가독성 높은 코드를 생성할 수 있을 것입니다.