🚧 [STMT-146] 파일 업로드 기능 코드 리뷰 반영

[05AM]

💁 해결하려는 문제를 적어주세요

• 파일 업로드 기능 코드 리뷰 반영 및 리팩터링

🤔 어떤 방식으로 해결했는지 적어주세요

• BusinessException을 상속받는 NotImplementedException를 구현하여 구현되지 않은 메서드 사용 시 예외 던지기
• 파일 업로드 시 IOException 놓쳤던 로그 출력
• 유효한 이미지 확장자를 별도의 enum 타입으로 구현
• FileUtil 클래스 리팩터링: 확장자 검사를 문자열 추출에서 content type 검사로 변경
  • originalFileName에서 문자열을 추출하는 것보다는 contentType을 가져와 검사하는 것이 낫다고 판단하여 수정하였습니다.
  • null 검사 등의 로직은 동일하게 진행되고 있습니다. 혹시 추가 검증이 필요하다 생각되시면 편하게 리뷰 달아주세요!
• file upload 정상작동 테스트용 API 구현
  • 실제 서비스에서 사용되는 api는 아니지만 정상 작동을 테스트하기 위해 구현했습니다.
  • 이후 admin만 사용 가능한 api로 변경 가능하고, 사용할 일이 없을 시에는 삭제하겠습니다.

🙋 중점적으로 리뷰 했으면 하는 부분이 있다면 적어주세요

• GlobalExceptionController의 상수 WARN_LOG_MESSAGE를 가져와서 사용해도 되는지, 안되는 이유가 있다면 어떻게 구현하면 좋을지.
  • 개인적으로 생각해본 방법은 common의 Constant 클래스를 구현하는 방법을 생각했습니다.

public class GlobalExceptionHandler {

    public static final String ERROR_LOG_MESSAGE = "[ERROR] {} : {}";
    public static final String WARN_LOG_MESSAGE = "[WARN] {} : {}";

• 지금 구현한 방식으로 예외를 찍는다면 중복 코드가 많아질 것 같은데 더 좋은 방법은 없을지

catch (IOException e) {
log.warn(WARN_LOG_MESSAGE, e.getClass().getSimpleName(), e.getMessage());
log.debug(e.getMessage(), e);

throw new BusinessException(ErrorCode.UPLOAD_FILE_FAIL_ERROR);
}

• ValidImageContentType의 패키지 위치
  • 현재는 FileUtil과 관련된 파일이니 utils에 넣었는데 비즈니스 규칙과 연관된 클래스인듯도 하여 어디에 위치시켜야 할지 고민이 됩니다.