[Clean Code] 14장. 오류 처리 리팩토링 실습

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25

// 실습 대상: 정수 오류 코드
public class FileProcessor {
    public static final int SUCCESS = 0;
    public static final int FILE_NOT_FOUND = 1;
    public static final int PERMISSION_DENIED = 2;
    public static final int INVALID_FORMAT = 3;

    public int processFile(String path) {
        if (!fileExists(path)) return FILE_NOT_FOUND;
        if (!hasReadPermission(path)) return PERMISSION_DENIED;
        if (!isValidFormat(path)) return INVALID_FORMAT;
        doProcess(path);
        return SUCCESS;
    }
}

// 호출부: 매번 정수 코드를 확인하고 분기해야 한다
int result = processor.processFile(path);
if (result == FileProcessor.FILE_NOT_FOUND) {
    System.out.println("파일을 찾을 수 없습니다: " + path);
} else if (result == FileProcessor.PERMISSION_DENIED) {
    System.out.println("읽기 권한이 없습니다: " + path);
} else if (result != FileProcessor.SUCCESS) {
    System.out.println("처리 실패: " + path);
}

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37

// 리팩토링 결과: 의미가 드러나는 예외 계층
public class FileProcessingException extends RuntimeException {
    public FileProcessingException(String message) { super(message); }
}

public class FileNotFoundException extends FileProcessingException {
    public FileNotFoundException(String path) {
        super("파일을 찾을 수 없습니다: " + path);
    }
}

public class InvalidFileFormatException extends FileProcessingException {
    public InvalidFileFormatException(String path, String expectedFormat) {
        super(String.format("파일 형식이 올바르지 않습니다: %s (기대 형식: %s)", path, expectedFormat));
    }
}

public class FileProcessor {
    public void processFile(String path) {
        if (!fileExists(path)) {
            throw new FileNotFoundException(path);
        }
        if (!isValidFormat(path)) {
            throw new InvalidFileFormatException(path, "CSV");
        }
        doProcess(path);
    }
}

// 호출부: 필요한 만큼만 구체적으로, 나머지는 상위 타입으로 처리
try {
    processor.processFile(path);
} catch (FileNotFoundException e) {
    System.out.println(e.getMessage());
} catch (FileProcessingException e) {
    logger.error("파일 처리 실패", e);
}

1
2
3
4
5
6
7
8

// 실습 대상: null 반환
public FileMetadata findMetadata(String path) {
    return metadataStore.get(path); // 없으면 null
}

// 호출부에서 null 확인을 빠뜨리면 NullPointerException이 발생한다
FileMetadata metadata = findMetadata(path);
System.out.println(metadata.getSize()); // path가 없으면 여기서 예외 발생

1
2
3
4
5
6
7
8
9

// 리팩토링 결과: Optional로 결과 없음을 타입으로 표현
public Optional<FileMetadata> findMetadata(String path) {
    return Optional.ofNullable(metadataStore.get(path));
}

// 호출부: 처리 경로를 명시적으로 선택해야 한다
long size = findMetadata(path)
    .map(FileMetadata::getSize)
    .orElseThrow(() -> new FileNotFoundException(path));