[Clean Code] 7장 오류 처리

Clean Code 스터디 내용 정리; 7장 오류 처리

 

오류 처리 코드로 인해 프로그램 논리를 이해하기 어려워진다면 깨끗한 코드라고 볼 수 없기 때문에, 이번 장에서는 우아하고 고상하게 오류를 처리하는 기법과 고려 사항 몇 가지를 다룬다.

미확인 예외를 사용한다.

 

• 오류 코드보다는 예외를 사용하여 프로그램 논리와 오류 처리 코드를 분리하는 게 좋다.
• 확인된 예외도 유용한 때가 있긴 하지만, 일반적인 애플리케이션에서는 이익보다는 연쇄적인 수정때문에 일어나는 비용이 더 크므로 쓰지 않는 편이 낫다.
• 예외를 던질 때는 오류 메시지에 전후 상황에 대한 정보를 담아 함께 넘겨준다.

try-catch-finally 부터 작성한다.

 

예외가 발생할 코드를 짤 때는, 강제로 예외를 일으키는 테스트 케이스를 작성한 후 테스트를 통과하게 코드를 작성하는 게 좋다. 이렇게 하면 try 블록부터 구현하게 되고, try-catch-finally 구조로 먼저 구현하면 catch 블록이 프로그램 상태를 일관성 있게 유지하도록 만들기 쉬워진다.

// 파일이 없으면 예외를 던지는지 알아보는 단위 테스트
@Test(expected = StorageException.class)
public void retrieveSectionShouldThrowOnInvalidFileName(){
  sectionStore.retrieveSection("invalid - file");
}

public List<RecordedGrip> retrieveSection(String sectionName){
  try{
    FileInputStream stream = new FileInputStream(sectionName);
  } catch(FileNotFoundException e){
    throw new StorageException("retrieval error",e);
  }
  return new ArrayList<RecordedGrip>();
}

예외를 처리하는 구체적인 방법

 

1. 감싸기 기법

 

외부 API를 사용할 때는 외부 API를 감싸는 클래스를 생성하여 외부 라이브러리와 프로그램 사이의 의존성을 줄인다.

// 외부 API ACMEPort의 예외 처리
public class LocalPort [
  private ACMEPort innerPort;

  public LocalPort(int portNumber){
    innerPort = new ACMEPort(portNumber);
  }

  public void open(){
    try{
      innerPort.open();
    } catch(DeviceResponseException e){
      throw new PortDeviceFailure(e);
    } catch(ATM1212UnlockedException(e){
      throw new PortDeviceFailure(e);
    }
  }
}


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

2. 특수 사례 패턴

 

클래스를 만들거나 객체를 조작해 예외적인 상황을 캡슐화해서 처리한다.

// 식비를 청구하지 않았다면(예외 발생 상황) 일일 기본 식비를 총계에 더한다.
try {
  MealExpenses expenses = expenseReportDAO.getMeals(employee.getID());
  m_total += expenses.getTotal();
} catch(MealExpensesNotFound e){
  m_total += getMealPerDiem();
}


// 청구한 식비가 없다면 일일 기본 식비를 반환하는 MealExpense 객체를 반환
public class PerDiemMealExpenses implements MealExpenses {
  public int getTotal() {
    // 기본값으로 일일 기본 식비를 반환
  }
}

MealExpenses expenses = expenseReportDAO.getMeals(employee.getID());
m_total += expenses.getTotal();

null 사용을 피한다.

• null을 반환하는 코드는 오류를 유발하기 쉽기 때문에 나쁘다.
• null을 반환하는 대신 예외를 던지거나 특수 사례 객체를 반환하자.
• 정상적인 인수로 null을 기대하는 API가 아니라면 메서드로 null을 전달하는 것을 마땅히 처리할 방법이 없기 때문에 애초에 null을 전달하지 않아야 한다.

// getEmployees()가 null도 반환하는 코드
List<Employee> employees = getEmployees();
if (employees != null) {
  for (Employee e : employees) {
    totalPay += e.getPay();
  }
}


// getEmployees()가 null이 아닌 빈 리스트를 반환하게 수정한 코드
public List<Employee> getEmployees() {
  if(..직원이 없다면..)
    return Collections.emptyList();
}

List<Employee> employees = getEmployees();
for (Employee e : employees) {
  totalPay += e.getPay();
}

깨끗한 코드는 읽기도 좋아야 하지만 안정성도 높아야 한다. 오류 처리를 프로그램 논리와 분리하면 독립적인 추론이 가능해지며 코드 유지보수성도 크게 높아져서 튼튼하고 깨끗한 코드를 작성할 수 있다.