clean code 제 7장 오류처리

7장 오류처리입니다

깨끗하게 오류를 처리하는 방법을 소개하고 있습니다.

 

1. 오류 코드보다 예외처리

 

BAD 예시

if (handle != DeviceHandle.INVALID):
    함수1()
    
    if ():
        함수2()
    else:
        log("오류2")
        
    log("오류1")
}

 

이런 코드 보다는

try catch 문으로 작성하는게 좋음

 

 

 

2. 오류 메세지

 

예외를 던질때 어떤 오류인지 추가로 정보를 제공하면 좋음.

ex. log("Device response exception")

 

 

3. 호출자를 고려

 

예외를 리턴하는 클래스를 정의하여 에러처리

 

BAD 예시

try {
    port.open();
} catch (e1) {
    reportPortError(e);
    log("Device ~ e");
} catch (e2){
    report~Error(e);
    log("~");
}
...

 

이렇게 각각 에러마다 에러 정보를 전달하는 부분을 작성해주기 보다는

 

아래처럼

LocalPort port = new LocalPort(12);
try {
    port.open();
} catch (PortDeviceFailure e) {
    reportError(e);
    logger.log(e.getMessage() , 3);
} finally {
     ..
}

 

라이브러리 API를 감싸면서 하나의 예외 유형과 log를 반환하는 class를 만들어줍니다.

 

-> 그렇게되면 에러마다 log 작성을 하지 않아도 됨.

 

 

 

 

4. catch에서 논리 제어 x (정상 흐름을 정의하라)

 

BAD 예시

 

식비 비용 청구하는 코드입니다.

 

try {

    MealExpenses expense = expenseReportDAO.getMeals(employee.getID()); 

    m_total += expenses.getTotal();

} catch (MealExpensesNotFound e) { 

    m_total += getMealPerDiem();

}

 

try {

    // 식비 냈는지 정보 가져옴

    //직원이 청구한 식비를 총계에 더함()

} catch () { // 식비 비용 청구 하지 않음

    // 일일 기본 식비를 총계에 더함()

}

 

 

이거 보다는

 

MealExpenses expense = expenseReportDAO.getMeals(employee.getID());

m_total += expenses.getTotal();

 

expenseReportDAO를 수정하여 항상 MealExpenses 객체를 반환하도록 함.

청구한 식비가 없다면 MealExpenses 객체 내부에서 일일 기본 식비를 반환하게 함.

 

이런식으로 catch에서 흐름을 정의하기보단 정상 흐름으로 코드를 정의하면 좋다고 함.

 

 

 

5. null 반환 x

 

if something != null { }

이런식으로 체크해야하는데

혹여라도 빠뜨리면 큰 문제가 될 수 있으니 지양하면 좋다고 함.

 

 

 

 

6. null 전달 x 

 

null 전달시 NullPointerException 발생 주의

 

assert 문으로 처리하면 좋음

 

public class MetricsCalculator {
	public double xProjection(Point p1, Point p2) {
    	assert p1 != null : "p1 is null";
        assert p2 != null : "p2 is null";
        return (p2.x - p1.x) * 1.5;
    }
}

 

 

 

감사합니다