점수 등록 API 수정 - Githubissues

Step3-kakao-tech-campus / Team3_BE

[카테캠 1기] 번개 지향 볼링 모집 커뮤니티 "번개볼링"의 백엔드 서버입니다.

2 stars 4 forks source link

점수 등록 API 수정 #70

Closed sososo0 closed 11 months ago

sososo0 commented 11 months ago

Summary

점수 등록 API 관련 버그 수정입니다.

Description

발견한 버그

점수만 등록했을 때
점수와 이미지(1개)를 등록했을 때

수정한 코드

기존의 코드에서 getSize() 부분을 제외하였습니다.

getSize가 file의 byte수를 체크하는 메소드이기 때문에, 잘못 사용된 코드라는 것을 알게 되어 해당 코드는 삭제했습니다.
```
if (image.getSize() > 1) {
        throw new CustomException(ErrorCode.SCORE_UPLOAD_FAILED, "점수는 1개씩 등록해주세요.");
}
```
image null 체크에서는 image를 보내지 않으면, 해당 필드를 포함하지 않는다고 하여, 해당 부분을 참고해서 코드를 수정하였습니다.
```
if (image == null) { 
... 
}
```

추가적으로 수정된 코드

위의 코드들을 수정하면서 점수 숫자 등록에 대한 유효성 검사 코드가 없어, 0~300 사이의 숫자가 아닌 다른 숫자를 넣고 postman에서 test를 해보았을 때, 500 error로 잡는 것을 발견했습니다.

따라서, scoreNum이 0~300 사이의 수가 아닐 때, 유효성 검사를 추가하였습니다.

if (scoreNum < 0 || scoreNum > 300) {
        throw new CustomException(ErrorCode.SCORE_UPLOAD_FAILED, "0~300 사이의 숫자만 입력해주세요.");
}

Related Issue

Issue Number: #69