인프런 커뮤니티 질문&답변

김주영님의 프로필 이미지
김주영

작성한 질문수

스프링 MVC 2편 - 백엔드 웹 개발 활용 기술

정리

getOriginalFilename null check

해결된 질문

작성

·

960

0

public String convertedFile(@RequestParam MultipartFile attachFile) throws IOException {
    if (attachFile.isEmpty()) {
            return null;
        }
    String originalFilename = attachFile.getOriginalFilename();
    String ext = extractExt(originalFilename);
}

위처럼 MultipartFile를 isEmpty로 체크하고 getOriginalFilename로 originalFilename을 가져온다음 originalFilename을 extractExt 함수의 argument로 넣어려고하는데 null을 check하라고 나옵니다. 정확히는 아래처럼 null이 나올 수 있다고 뜨네요.

isEmpty로 위에서 한 번 체크하면 안뜨는 걸로 알고 있는데 뜨는 이유가 뭘까요

답변 1

2

안녕하세요, 인프런 AI 인턴이에요.

MultipartFile.getOriginalFilename() 메서드는 파일이 없는 경우 null 값을 반환합니다. 따라서 MultipartFile.isEmpty() 메서드로 체크한 후에도 null 값이 나올 수 있는 경우, null check가 필요합니다. 이러한 경우를 대비하여 null check 코드를 추가하는 것이 좋습니다.

해당 코드에서는 MultipartFile.isEmpty() 메서드를 통해 파일의 유무를 먼저 체크하고 있습니다. 만약 파일이 없는 경우, null 값으로 리턴하고 있습니다. 하지만, MultipartFile.getOriginalFilename() 메서드는 파일이 없는 경우 null 값을 반환합니다. 따라서, 파일 체크와 null check 모두를 수행하는 것이 좋습니다.

추가적으로, null check를 할 때 null 체크를 먼저 수행한 후 메서드를 호출하는 것이 좋습니다. 따라서, 아래와 같이 코드를 작성하면 null check를 수행할 수 있습니다.

if (attachFile != null && !attachFile.isEmpty()) {
String originalFilename = attachFile.getOriginalFilename();
if (originalFilename != null) {
String ext = extractExt(originalFilename);
}
else {
// originalFilename이 null인 경우 처리할 코드 작성
}
}
else {
// attachFile이 null 또는 비어있는 경우 처리할 코드 작성
}

이렇게 작성하면 attachFile이 null이거나 비어있는 경우를 먼저 처리하고, 그 후에 originalFilename이 null인 경우를 처리할 수 있습니다.

참고로, extractExt 함수에서도 null check가 필요합니다. 만약 originalFilename이 null인 경우 메서드를 호출하면 NullPointerException이 발생합니다. 따라서, 아래와 같이 null check를 수행하는 것이 좋습니다.

public String extractExt(String originalFilename) {
if (originalFilename == null) {
return null;
}

}

위 내용을 참고하여 코드를 수정하시면 될 것 같아요.

김주영님의 프로필 이미지
김주영

작성한 질문수

질문하기