단위테스트와 TDD는 다르다.

단위테스트는 프로덕션 코드에 대한 테스트코드를 작성하는 것이라면,

TDD는 실패하는 컴파일 되는 테스트코드를 기반으로 프로덕션 코드를 작성해 나가는 것을 의미한다.

레거시 코드 TDD로 리팩토링하기

단위가 큰 프로젝트일수록 TDD를 진행하기가 어렵다. 특히, 리팩토링을 하거나 새로운 기능을 추가하게 되는 경우가 많은데 이럴 경우 프로덕션 코드 하나에 따른 다수의 테스트코드 수정이 발생할 수 있다.

✨점진적 리팩토링 - 오버로딩

: 컴파일이 가능하게 기존 메소드를 유지한 채, 시그니처가 다른 수정 메소드를 하나 더 만든다. (메소드 오버로딩)

생성자나 변수의 경우에도 마찬가지이다. 컴파일이 가능하게 중복을 유지한 채로, 점진적인 수정을 진행한다. 리팩토링을 하면서 중간에 배포될 수도 있고, 기존 상황으로 되돌아가야할 수도 있다. 내가 생각할 땐, 어느시점에서든 ✨컴파일 되는 상황을 유지하는 것이 중요하다고 생각한다. (돌아가는 시스템이 가장 우선된다.)

컴파일되는 상황을 유지한 채(중복되는 상태인 과도기적 상태를 유지), 점진적으로 리팩토링하고, 테스트코드를 유지해나가는 연습이 필요하다. 이 경우에도 TDD의 원칙은 유지한채 작성해나갈 수 있다.

Getter를 지양하라 - 객체값으로 비교

생성자가 setting의 역할을 하기 때문에 불변객체가 되기위해 setter는 지양된다. 더불어, getter도 가능한 지양하도록 한다. equals, hashcode로 객체비교를 진행해야 한다. 하지만, 원시값을 포장하였는데 view에서 그 내용 값을 보여줘야 한다면 getter를 사용할 수 밖에 없지 않을까 하는 생각도 든다.

Cache - 적절한 자료구조✨를 생각하자!

Getter를 사용하지 않는다면 배열로 캐싱을 하여 사용할 때, 해당 인덱스에 내가 원하는 객체가 있는지 어떻게 비교하지? 고민이 많이 되었다. 그렇다면 더 적합한 자료구조가 있는지 생각해보자!

public class LottoNumber {
    public static final int MINIMUM_NUMBER = 1;
    public static final int MAXIMUM_NUMBER = 45;
    private static final LottoNumber[] cache = new LottoNumber[MAXIMUM_NUMBER];

    private final int number;

    static {
        for (int i = 0; i < MAXIMUM_NUMBER; i++) {
            cache[i] = new LottoNumber(i + MINIMUM_NUMBER);
        }
    }

    private LottoNumber(int number) {
        validateNumberRange(number);
        this.number = number;
    }

    public static LottoNumber valueOf(int i) {
        if (i <= MAXIMUM_NUMBER && i >= MINIMUM_NUMBER) {
            return LottoNumber.cache[i - MINIMUM_NUMBER];
        }
        return new LottoNumber(i);
    }
//...
}

Map은 캐싱을 할 때 많이 사용된다고 한다. Map<Integer, LottoNumber> cache 로 만든다면 인덱스 실수가 발생하지 않는다. 제대로 들어있는지 의심점이 적어진다. 뭔가 잘 안 풀린다면 적절한 자료구조를 사용하고 있는지 다시 생각해보자! 😀

public class LottoNumber {
    public static final int MINIMUM_NUMBER = 1;
    public static final int MAXIMUM_NUMBER = 45;
    private static final Map<Integer, LottoNumber> cache = new HashMap<>(MAXIMUM_NUMBER);

    private final int number;

    static {
        for (int i = MINIMUM_NUMBER; i <= MAXIMUM_NUMBER; i++) {
            cache.put(i, new LottoNumber(i));
        }
    }

    private LottoNumber(int number) {
        validateNumberRange(number);
        this.number = number;
    }

    public static LottoNumber valueOf(int i) {
        if (i <= MAXIMUM_NUMBER && i >= MINIMUM_NUMBER) {
            return cache.get(i);
        }
        return new LottoNumber(i);
    }
//...
}

new 생성자로 호출되면 생성되는 것을 막을 수 없다.

위와 같이 valueOf를 사용하여 정적 팩토리 메소드를 구현하였다면 생성자는 닫아주는 것이 잘못된 생성을 방지할 수 있는 방안이 될 수 있다. 1~45 값 이외의 객체 생성을 막으려면 생성자를 막고, 범위 이외에서 exception을 발생시키면 된다.

public class LottoNumber {
//...
    private static final LottoNumber[] cache = new LottoNumber[MAXIMUM_NUMBER];

    private final int number;

    static {
        for (int i = 0; i < MAXIMUM_NUMBER; i++) {
            cache[i] = new LottoNumber(i + MINIMUM_NUMBER);
        }
    }

    private LottoNumber(int number) {
        this.number = number;
    }

    public static LottoNumber valueOf(int i) {
        if (i > MAXIMUM_NUMBER || i < MINIMUM_NUMBER) {
            throw new CustomException("로또 넘버는 1~45 사이 정수이어야 합니다.");
        }
        return LottoNumber.cache[i - MINIMUM_NUMBER];
    }
//...
}

저작자표시 비영리 동일조건