[클린코드] 15일차 - 오류 처리 (부제: 프로그램 논리와 분리해 독자적인 사안으로 고려)

[클린코드] 15일차 - 오류 처리 (부제: 프로그램 논리와 분리해 독자적인 사안으로 고려)

클린코드 14일차 (p. 123~128(6장) /129~131(7장) )

클린코드 15일차 (p. 132~141(7장) )

7 오류 처리

오류 처리는 프로그램에 반드시 필요한 요소 중 하나. 뭔가 잘못되면 바로 잡을 책임은 프로그래머에게 있다.

깨끗한 코드와 오류 처리는 확실히 연관성이 있다.

하지만 오류 처리 코드로 인해 프로그램 논리를 이해하기 어렵다면 깨끗한 코드가 아니다.

 

1. 오류 코드보다 예외를 사용하라

- 오류가 발생하면 예외를 던지는 편이 낫다. 

- 그러면 호출자 코드가 더 깔끔해진다.

 

2. Try-Catch-Finally 문부터 작성하라

- try-catch-finally 문에서 try블록에 들어가는 코드를 실행하면 실행이 중단된 후 catch 블록으로 넘어갈 수 있다.

- 어떤 면에서 try 블록은 트랜잭션과 비슷하다.

- try 블록에서 어떤 일이 생기든지 catch 블록은 프로그램 상태를 일관성 있게 유지해야한다.

 

- 즉, 예외가 발생할 코드를 짤 때는 try-catch-finally 문이 Good!

 

- 강제로 예외를 일으키는 테스트 케이스를 작성한 후 테스트를 통과하게 코드를 작성하는 방법 권장

- 자연스럽게 try 블록의 트랜잭션 범위부터 구현하게 되므로 범위 내 트랜잭션 본질을 유지하기 쉬워진다.

 

3. undefined 예외를 사용하라

- 확인된 예외는 OCP(Open Closed Principle)을 위반한다.

 

4. 예외에 의미를 제공하라

- 예외를 던질 떄는 전후 상황을 충분히 덧붙인다.

- 오류 메시지에 정보를 담아 예외와 함께 던진다. (+ 실패한 연산 이름과 실패 유형도 언급)

- 어플리케이션이 로깅 기능을 사용한다면 catch 블록에서 오류를 기록하도록 ..!

 

5. 호출자를 고려해 예외 클래스를 정의하라

- 오류가 발생한 위치로 분류가 가능하다.

- 예를 들면, 오류가 발생한 컴포넌트로 분류 / 유형으로 분류 / 디바이스 실패 / 네트워크 실패 / 프로그래밍 오류 등 으로 분류

위 경우 예외에 대응하는 방식이 예외 유형과 무관하게 거의 동일하다. 그래서 간결하게 고치기 쉽다..!!

 

위 경우 예외에 대응하는 방식이 예외 유형과 무관하게 거의 동일하다. 그래서 간결하게 고치기 쉽다..!!

바로 호출하는 라이브러리 API를 감싸면서 예외 유형 하나를 반환하면 된다. 

*호출하는 라이브러리 API : LocalPort 클래스를 만들어 ACMEPort 클래스가 던지는 예외 잡아 반환하는 Wrapper 클래스 만듬

실제로 외부 API를 사용할 때는 감싸는 기법 선택 !

실제로 외부 API를 사용할 때는 감싸는 기법이 최선이라고 한다!!!

외부 API를 감싸면 외부 라이브러리와 프로그램 사이에서 의존성이 크게 줄어든다. 

 

6. 정상 흐름을 정의하라

- 특수 사례 패턴 Special Case Pattern 

- 클래스를 만들거나 객체를 조작해 특수 사례를 처리하는 방식이다.

- 그러면 클라이언트 코드가 예외적인 상황을 처리할 필요가 없어진다. (클래스나 객체가 예외적인 상황을 캡슐화해서 처리하므로.)

 

7. null을 반환하지 마라

- null을 반환하는 코드는 일거리를 늘릴 뿐만 아니라 호출자에게 문제를 떠넘긴다.

- 누구 하나라도 null 확인을 빼먹는다면 애플리케이션이 통제 불능에 빠질지도 모르ㄴ다... 

 

- Bad: null을 반환하는 getEmployees()

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

// 위에서 getEmployees는 null도 반환한다. BUT, 반드시 null을 반환할 필요가 있을까???

 

- Better: 빈 리스트를 반환하는 getEmployees()

// getEmployees를 변경해 빈 리스트를 반환한다면 코드가 훨 깔끔해질 것 !
public List<Employee> getEmployees() {
    if ( ... 직원이 없다면 ... ){
    	return Collections.emptyList();
    }
}

 

8. null을 전달하지 마라

- null을 반환하는 방식도 나쁘지만 메서드로 null을 전달하는 방식은 더 나쁘다. 

 

 

결론
- 깨끗한 코드는 읽기도 좋아야 하지만 안정성도 높아야 한다. 이 둘은 상충하는 목표가 아니다.

- 오류 처리를 프로그램 논리와 분리해 독자적인 사안으로 고려하자...!

- 프로그램 논리와 분리 시 독립적인 추론이 가능해지며 코드 유지보수성이 높아진다..!