[2019.02.09] 클린 코드 (JUnit 리팩토링 사례 분석)

JUnit 리팩토링 사례 분석

이제는 더 이상 필요없는 접두어를 제거하자

지저분한 조건문은 메소드로 boolean 을 반환 받도록 분리하자

특정 메소드에 사용되는 함수들이 있다면

해당 함수들의 사용 방식에 일괄성을 주자

(모두 반환하는 데이터가 있게 하도록 하거나 모두 없게 하거나 하자)

숨겨진 시간 결합을 끊자.

숨겨진 시간 결합

findA가 멤버변수 A를 찾아내서 할당하는 함수고,

findB가 멤버변수 A를 이용해서 B를 찾아내는 함수라면,

다음 코드는 숨겨진 시간 결합이 존재한다.

findA();

findB();

아래와 같이 바꿔서 숨겨진 시간 결합을 깰 수 있다.

A = findA();

B = findB(A);

좀 더 개선하면 findB를 다음과 같이 고치고 findAAndB함수를 사용해라.

public void findAAndB(){

findA();

findB에 있던 Body 영역 코드

}

※ JUnit