기본형 집착

• 애플리케이션이 다루고 있는 도메인에 필요하 기본타입을 만들지 않고 프로그램 언어가 제공하는 기본타입을 사용하는 경우가 많다.

• 기본형으로 단위 또는 표기법을 표현하는데에 한계가 있다.

관련 리팩토링 기술

• 기본형을 객체로 바꾸기
• 타입 코드를 서브클래스로 바꾸기
• 조건부 로직을 다형성으로 바꾸기
• 클래스 추출
• 매개변수 객체 만들기

기본형 집착/ 기본형을 객체로 바꾸기

• 개발 초기에는 기본형으로 표현한 데이터가 나중에는 해당 데이터와 관련있는 다양한 기능을 필요로 하는 경우가 발생한다.

• 예) 숫자로 표현하던 온도의 단위 -> 화씨,섭씨로 변환하는 경우

• 기본형을 사용한 데이터를 감싸 줄 클래스를 만들면 해결 가능

• 지금은 단순히 Priority 지역변수의 값을 가져와 계산하는 방식이다. 하지만 이후에는 어떤값이 더 높은지 우선순위 분별 코드가 필요로 해질것이다.

• Priority class 에서는 데이터 타입의 안정성이 보장되지 않는다. 그냥 String 라 아무값이나 들어올수 있다.

• 타입의 안정성을 위해 리스트안에 존재하는 값만 들어올수 있도록 해준다.

• Value index를 구해주는 메서드와 현재 객체의 value값과 매개변수로 들어오는 값의 index를 비교하는 메서드도 만들어준다.

• Oder 클래스에서는 Priority 를 필드와 생성자로 받고 getter도 만들어준다.

• order 클래스에서 기존에 있던 String 타입은 지워준다.
• 테스트 코드도 무리없이 잘 진행된다.

=> 일반 문자열, 기본타입을 쓰던것을 객체타입으로 만들고 그 객체를 사용하도록 만들면 해당 데이터의 기능을 쓰기 쉽다.

기본형 집착/ 타입 코드를 서브 클래스로 바꾸기

• 크게 두가지가 있다. 첫번째는 아무것도 상속받지 않아 이 클래스를 상속을 통해 서브클래스로 뺴낼수 있는 클래스.

• 두번째는 이미 상속이 진행되있어서 간접적인 상속을 활용해야하는 클래스

• 일단 첫번쨰 클래스 타입이 추후에 각각의 타입마다 하는일이 많이 달라진다는 가정하에 타입을 서브클래스로 만들어보자

• 각각 Engineer , Salesman , Manager 클래스를 만든뒤 위와 같이 만들어 준다.

• 더 이상 Employee 에 type 이란 필드는 필요하지 않다. 그에따라 생성자 및 다른 코드들도 수정해주자.

• 각각 자식클래스에서 오버라이드한 메서드는 당연히 abstract로 바꿔준다.

• 기존에 사용하던 메서드 validate도 더이상 필요가 없으니 지워준다.

• 테스트 부분도 우리가 만든 static메서드로 교체 해준다.

두번째 클래스 타입인 경우

• 이미 상속을 받는 상태인 경우

• EmployeeType 라는 클래스를 만든뒤 각각 자식클래스들이 이 클래스를 상속 받게 한다. (자식클래스들도 새로)

• EmployeeType 를 적용시켜보자 . Emploee 에서는 참조를 하며 이전방법과 같이 메서드를 만들어준다.

• Employee도 부모클래스이지만 다른 부모클래스 EmployeeType 도 참조중인 상황

• 필요가 없는 필드와 메서드를 지워서 코드를 정리 해준다.

• capitalizedType() 는 다른부모 클래스에게 위임해준다.

• 다른부모 자식클래스에 오버라이드한 toString이 동작한다.

• 생성자 -> 필드(다른객체) 초기화 -> 그객체의 메서드

기본형 집착/ 조건부 로직을 다형성으로 바꾸기

• 복잡한 switch, if- else문의 반복은 상속이나 다형성을 활용해 해결할수 있다.

• 기본동작과 특수한 경우의 동작이 섞여있는경우에 상속 구조를 만들어 기본동작은 상위클래스에, 특수한경우는 하위클래스에 배치함으로써 각 타입에 따라 차이점을 강조 할수 있다.

예) 이러한경우에는 이렇게 동작하고 , 저러한 경우에는 저렇게 동작

• 이 예제인 경우에도 적용이 가능하다 (switch 문)

• 일단 상속을 받는 자식 클래스들을 만들어준다

• 부모 클래스의 메서드들도 수정 해준다

• type 필드는 지울 예정이다.

• type 라는 필드는 더이상 필요가 없으므로 지워주고 그에 해당하는 생성자도 지워준다.

• 오버라이드를 당하는(?) 메서드들도 한눈에 보기좋게 abstract로 만들어준다

• 자식클래스 중에서 중복되는 메서드는 위로 (부모클래스로) 올려준다.

• 테스트 코드에서도 잘 동작 하는것을 볼수 있다.

결론

처음 switch 문에서 type에 따라 실행코드를 다르게 했다면 부모-자식 상속관계로 나눠줌으로써 type이라는 필드를 없애며 복잡한 switch 문을 정리

반복되는 Switch 문

• 위에서 함

반복문을 파이프라인으로 바꾸기

• 반복문을 필터나 맵핑과 같은 파이프라인 기능을 사용해 적용시켜주면 보다 빠르게 어떤 작업을 하는지 파악할 수 있다.

• 반복문을 보면 한눈에 보기가 쉽지않다. 한줄한줄 해석해서 읽어야함.
• 이 반복문을 파이프라인으로 바꿔 좀더 명확한 코드로 만들어주자

• 첫 if문부터 바꿔주자 . stream/filter/lamda

• map 으로는 해당 데이터 안의 데이터 타입을 바꿔준다.

• 마지막으로 collect를 사용해 List에 해당 값들을 넣는것을 표현해준다. 이러면 기존코드들(주석처리된)은 더 명확하게 대체 될수 있다.

성의없는 요소

• 가끔 예상하고 만들어 놓은 요소들이 부응하지 못하는 경우가 있는데 그런 경우에 해당 요소들을 제거 해야한다.

관련 리팩토링 기술

• 함수 인라인
• 클래스 인라인
• 불필요한 상속 구조는 계층 합치기

추측성 일반화

• 나중에 이런저런 기능이 생길것을 예상하여 여러 기능을 만들어 놨지만 결국에 쓰이지 않는 코드가 발생할 경우..

• YAGNI(u r not gonna need it) 원칙을 따르자

추측성 일반화 / 죽은 코드 제거하기

• 사용하지 않는 코드가 애플리케이션 성능이나 기능에 영향을 끼치지는 않는다.

• 나중에 필요해질 코드라 하더라도 지금 쓰이지 않는 코드라면 (주석으로 감싸는게 아니라) 삭제 해야한다.

임시필드

• 클래스에 있는 어떤 필드가 특정한 경우에만 값을 갖는 경우

관련 리팩토링 기술

• 해당 변수 옮기기 => 클래스 추출하기

• 해당 변수를 사용하는 함수 옮기기 => 함수 옮기기

• 특정한 경우에 해당하는 클래스를 만들어 해당조건을 제거 => 특이 케이스 추가하기

임시필드 / 특이케이스 추가하기

• 어떠한 경우 이런 로직이 실행되고 어떠한 경우에는 저런 로직이 발생하는 경우

• customer 클래스중 필드인 name이 unknown 인 경우 어떠한 로직이 발생한다. "unknown 인경우" 가 반복된다. 이 반복되는것을 따로 빼서 클래스로 만들어 주자.

• 메서드 추출로 반복되는 부분을 추출해준다. isUnknown 을 보면 customer의 정보만 필요함으로 메서드의 위치 customer이 더 적절함으로 메서드위치를 옮겨주자.

• Customer의 isUnknown 은 이름이 있으니까 false 그 반대 unknownClass는 true

• 매개변수 Site의 값에 따라 좌지우지 되야한다. Site 값을 설정해주자

• unknown 이면 unkownCustomer 아니면 customer

• 캡슐화

• 더 간략하게, 더 깔끔하게 줄여보자. 냄새를 맡자 냄새

• 여기서 주목해야 할점은 Site 값이 들어올때 이미 unknownCustomer / Customer 인지 분별이 되서 넘어온다는점!

• 기존 코드에서는 unknownCustomer 일때 특정한 값을 변수에 대입해서 반환한다. Customer 일때는 클래스의 필드 값반환.

• unknownCustomer 클래스에 아예 고정값을 넣어 버려서 두번쨰 사진처럼 코드를 한줄로 줄일수 있다.

• 두번째 코드변경이다. 첫번째 변경사항과 매우 흡사하다. 기존 billPlan() 부분도 커스터머 종류에 따라 값을 다르게 출력( 하나는 고정값)

• customer 일경우 그 클래스의 필드값 반환 / unknownCustomer일 경우 객체반환 ->
  unknownCustomer 필드 초기화시 아예 필드에 객체값을 넣어준다.

• 기존의 코드는 unknown일경우 0반환 / customer 일경우 해당 필드 값 반환

• customer일 경우 초기화된 필드값을 그대로 반환 하고 / unknownCustomer 일경우 도 따로 초기화된 필드값을 반환 -> 그 값이 상속을 받아 값을 따로 설정