[리팩토링워크북]3장_측정할 수 있는 냄새-연습문제3. 주석-

연습문제 3. 주석
이 코드를 검토해 보자. (코드는 http://www.xp123.com/rwb에서 구할 수 있다.)
(소스 코드)
public class Matcher { public Matcher(){} public boolean match(int[] expected, int[] actual, int clipLimit, int delta){ //기준 값보다 긴 값 잘라내기! for(int i=0; i<actual.length; ++i) if(actual[i] > clipLimit) actual[i] = clipLimit; //실제 값들과 기대 값들의 총 개수가 같은지 검사하기! if(actual.length != expected.length) return false; //각 값들이 예상 범위에 있는지 검사하기! for(int i=0; i<actual.length; ++i) if(Math.abs(expected[i] - actual[i]) > delta) return false; return true; } }
(테스트 코드)
import junit.framework.TestCase; public class MatcherTest extends TestCase{ public MatcherTest(String name) {super(name);} public void testMatch(){ Matcher matcher= new Matcher(); int[] expected = new int[]{10, 50, 30, 98}; int clipLimit = 100; int delta = 5; int[] actual = new int[]{12, 55, 25, 110}; assertTrue(matcher.match(expected, actual, clipLimit, delta)); actual = new int[]{10, 60, 30, 98}; assertTrue(!matcher.match(expected, actual, clipLimit, delta)); actual = new int[]{10, 50, 30}; assertTrue(!matcher.match(expected, actual, clipLimit, delta)); } }
(리팩토링 코드)
public class Matcher { public Matcher(){} public boolean match(int[] expected, int[] actual, int clipLimit, int delta){ clipValue(actual, clipLimit); if(isDifferentLength(expected, actual)) return false; for(int i=0; i<actual.length; ++i) if(isWithinSection(expected, actual, delta, i)) return false; return true; } private boolean isWithinSection(int[] expected, int[] actual, int delta, int i) { return Math.abs(expected[i] - actual[i]) > delta; } private boolean isDifferentLength(int[] expected, int[] actual) { return actual.length != expected.length; } private void clipValue(int[] actual, int clipLimit) { for(int i=0; i<actual.length; ++i) if(actual[i] > clipLimit) actual[i] = clipLimit; } }

-match 메소드 내부 코드를 사람이 이해하기 쉬운 의미 단위로 코드를 리팩토링하여 별도의 주석없이도 코드 이해가 쉽도록 변경

-사용한 리팩토링 기법: Extract-Method