[Clean Code] 7장. 오류 처리

Rose·2024년 2월 7일

클린코드

CleanCode

목록 보기

6/8

TIL(Today I Learned)

내용 3줄 정리 💁🏻‍♀️

깨끗한 코드는 읽기도 좋아야 하지만 안정성도 높아야 한다.
오류 처리를 프로그램 논리와 분리하면 독립적인 추론이 가능해지며 유지보수성도 크게 높아진다.
오류 코드보다 try-catch-finall 문을 활용하여 예외처리를 하고, null은 반환하거나 전달하지 않는편이 좋다.

기억하고싶은 내용 📝

깨끗한 코드와 오류 처리는 확실히 연관성이 있다. 상당수 코드 기반은 전적으로 오류 처리 코드에 좌우된다.
오류 처리는 중요하다. 하지만 오류 처리 코드로 인해 프로그램 논리를 이해하기 어려워진다면 깨끗한 코드라 부르기 어렵다.
오류 코드보다 예외를 사용하라.
Try-Catch-Finally 문부터 작성하라. (132p)

try블록에 들어가는 코드를 실행하면 어느 시점에서든 실행이 중단된 후 catch 블록으로 넘어갈 수 있다.
try블록에서 무슨 일이 생기든지 catch 블록은 프로그램 상태를 일관성 있게 유지해야 한다.
try블록에서 무슨 일이 생기든지 호출자가 기대하는 상태를 정의하기 쉬워진다.
미확인 예외를 사용하라. (133p)
예외에 의미를 제공하라. (135p)
- 오류 메시지에 정보를 담아 예외와 함께 던진다. 실패한 연산 이름과 실패 유형도 언급한다.
호출자를 고려해 예외 클래스를 정의하라. (135p)
- 흔히 예외 클래스가 하나만 있어도 충분한 코드가 많다. 예외 클래스에 포함된 정보로 오류를 구분해도 괜찮은 경우가 그렇다.
- 한 예외는 잡아내고 다른 예외는 무시해도 괜찮은 경우라면 여러 예외 클래스를 사용한다.
null을 반환하지 마라. (138p)
- NullPointerException이 발생할 수 있다.
- 메서드에서 null을 반환하는 대신 예외를 던지거나 특수 사례 객체를 반환한다.
- 사용하려는 외부 API가 null을 반환한다면 감싸기 메서드를 구현해 예외를 던지거나 특수 사례 객체를 반환하는 방식을 고려한다.
null을 전달하지 마라. (140p)
다음은 두 지점 사이의 거리를 구하는 간단한 메서드이다.
```
public class MetricsCalculator
{ 
	public double xProjection(Point p1, Point p2) {
   	return (p2.x - p1.x) * 1.5;
   }
   ...
} 
```
➡️ 누군가 인수로 null을 전달하는 경우 NullPointerException 발생

solution 1. 새로운 예외 유형을 만들어 던지기
```
public class MetricsCalculator 
{
	public double xProjection(Point p1, Point p2) {
    	if (p1 == null || p2 == null) {
        	throw InvalidArgumentException(
            	"Invalid argument for MetricsCalculator.xProjection");
        }
        return (p2.x - p1.x) * 1.5;
    }
}
```
하지만 위 코드는 InvalidArgumentException 을 잡아내는 처리기가 필요하다.

solution 2. assert문 사용하기
```
public class MetricsCalculator 
{
	public double xProjection(Point p1, Point p2) {
    	assert p1 != null : "p1 should not be null";
        assert p2 != null : "p2 should not be null";
        return (p2.x - p1.x) * 1.5;
    }
}
```
문서화가 잘 되어 코드 읽기는 편하지만 문제를 해결하지 못한다.

대다수 프로그래밍 언어는 호출자가 실수로 넘기는 null을 적절히 처리하 는 방법이 없다. 애초에 null을 넘기지 못하도록 금지하는 정책이 합리적이다. 즉, 인수로 null이 넘어오면 코드에 문제가 있다는 말이다. 이런 정책을 따르면 그만큼 부주의한 실수를 저지를 확률도 작아진다.