DAY 29

🔖 오늘 읽은 범위 : 14. 점진적인 개선(270~295p)

🤓 책에서 기억하고 싶은 내용

String 인수

• 한 번에 하나씩 고치면서 테스트 수행, 하나라도 실패할시 수정 후 진행
• 각 인수 유형을 처리하는 코드를 모두 ArgumentMarchaler 클래스에 넣고 나서 ArgumentMarshaler 파생 클래스를 만들어 코드를 분리 → 프로그램 구조를 조금씩 변경하는 동안에도 시스템의 정상 동작을 유지하기 쉬움

  • 추상 메서드 set을 만듬

    private abstract class ArgumentMarshaler {
        protected boolean booleanValue = false;
        private String stringValue;
        private int integerValue;
        public void setBoolean(boolean value) {
            booleanValue = value;
        }
        public boolean getBoolean() {
            return booleanValue;
        }
        public void setString(String s) {
            stringValue = s;
        }
        public String getString() {
            return stringValue == null ? "" : stringValue;
        }
        public void setInteger(int i) {
            integerValue = i;
        }
        public int getInteger() {
            return integerValue;
        }
        public abstract void set(String s);
    }

  • BooleanArgumentMarshaler 클래스에 set 메서드를 구현

    private class BooleanArgumentMarshaler extends ArgumentMarshaler {
        public void set(String s) {
            booleanValue = true;
        }
    }

  • setBoolean 호출을 set호출로 바꿈

    private void setBooleanArg(char argChar, boolean value) {
        booleanArgs.get(argChar).set("true");
    }

  • set 기능을 BooleanArgumentMarshaler 로 옮겻으므로 BooleanArgumentMarshaler.setBoolean 메서드를 제거

• 컴파일을 위해 get 추상메서드 추가
• 알고리즘 처음에 나오는 인수 유형별 맵 세 개 없앰 → ArgumentMarshaler로 맵 변경
• 여전히 지저분 → args 배열을 list로 변환한 후 Iterator를 set 함수로 전달

🤔 떠오르는 생각

• 코드 리뷰는 2회독으로 미룬다. 다음은 오지 않는다고했는데..

🔎 질문

📝 소감 3줄 요약

• 한 번에 하나씩 고치면서 테스트 수행, 하나라도 실패할시 수정 후 진행
• 하나의 클래스에 수정할 내용을 모두 넣고 분리하는 식으로 하여 기존 시스템 운영을 유지시킬 수 있다
• 인수개수를 최소화 하기위해 args배열을 list로 변환하여 set함수로 전달