▣ 개요
→ 아주 간단한 연산을 하는 임시변수가 존재하는데 이 임시변수가 다른 리펙토링을 방해한다면 원래 연산으로 바꿔버린다.
▣ Inline Temp의 사용 필요성
→ 보통 메소드 호출의 결과값이 임시변수에 대입되는 경우에 사용된다.
→ 임시변수가 Extract Methods 같은 리펙토링에 방해가 될시에도 사용된다.
▣ 주의사항
→ 지나치게 복잡한 연산의 경우는 좀 더 생각해보고 변경하길 바란다. 특히 연산량이 많고, 많이 사용되는 임시변수는 더더욱 그렇다.
▣ 예시
double basePrice = anOrder.BasePrice();
return (basePrice > 1000);
▼
return (anOrder.BasePrice() > 1000);
→ 불필요한 임시변수를 제거하고 연산을 직접 넣었다.
▣ 개요
→ 지난 포스팅때의 Extract Methods 와는 반대개념. 이번에는 보기에도 너무나도 명확한 메소드를 삭제하고 코드조각으로 만들어 버린다.
▣ Inline Method의 사용 필요성
1) 메소드의 몸체가 메소드의 이름만큼 명확할때
2) 불필요한 인디렉션 (간접접근)을 줄이고자 할때
3) 메소드가 잘못 나뉘어져 있을때 그것을 합칠시 사용.
▣ 주의사항
→ 다형성을 가지고 있는 메소드 (오버리이딩 메소드)에는 사용해서는 안된다.
→ 코드조각으로 바꾼 다음 잊지 않고 기존 메소드는 제거한다.
→ 재귀가 사용되는 등의 복잡성을 가진 함수는 적용하지 않는 게 좋을 수도 있다.
▣ 예시
int GetRating()
{
return (MoveThanFiveLateDeliveries()) ? 2 : 1;
}
bool MoveThanFiveLateDeliveries()
{
return m_numberOfLateDeliveries > 5;
}
▼
int GetRating()
{
return (m_numberOfLateDeliveries > 5) ? 2 : 1;
}
→ 지나치게 간략하고 동작히 명확한 메소드를 제거하였다.
▣ Extract Methods 사용 필요성
1) 지나치게 긴 메소드에 대한 쪼개기 용도
2) 메소드를 잘게 쪼갬으로써 다른 메소드에서 사용 가능성 용이
3) 메소드를 나눔으로써 일련의 주석을 읽는것 같은 효과 : 가독성 쉬워짐
4) 메소드 오버라이딩 용이
→ Extract Methods 수행시 메소드의 이름을 잘 지어야 한다. 만약 이해하기 쉬운 이름을 지을 수 없다면 코드를 뽑아내지 않는 것이 나을지도 모른다.
▣ 주의사항
→ 메소드 이름을 정할시 어떻게 하는지를 나타내는 이름을 정하지 말고, 무엇을 하는지를 나타내는 이름을 정하라.
→ 뽑아내고자 하는 코드 조각 내에 원래 사용하던 지역변수가 있을시에는 새로운 메소드의 파라미터 혹은 지역변수로 바꾼다.
→ 뽑아낸 코드 조각에서 지역변수가 있고, 만약 그 변수가 코드 조각에서 수정되고, 코드조각 이후에 쓰이게 될시에는 이 뽑아낸 코드를 질의(query)로 봐야 한다.
: 질의(query) 는 쉽게 말해 하나의 리턴값을 주는 메소드를 의미한다.
→ 만약 위와 같이 값이 수정되는 지역변수가 둘 이상 존재시에는 다른 방법을 써야 한다.
▣ 예시
★ 코드조각 내 지역변수가 없을시
void printOwing()
{
Enumeration e = m_orders.elements();
double outstanding = 0.0;
// banner
System.out.println("*************************");
System.out.println("***** Customer Owes *****");
System.out.pringln("*************************");
// outstanding 계산
while (e.hasMoreElements())
{
Order each = (Order)e.nextElement();
outstanding += each.getAmount();
}
// 상세 정보 표시
System.out.println("name:" + m_name);
System.out.println("amount:" + outstanding);
}
void printOwing()→ 단순하게 필요한 코드조각만 뽑아서 새로운 메소드로 만들면 된다.
{
Enumeration e = m_orders.elements();
double outstanding = 0.0;
printBanner();
// calculate outstanding
while (e.hasMoreElements())
{
Order each = (Order)e.nextElement();
outstanding += each.getAmount();
}
// print details
System.out.println("name:" + m_name);
System.out.println("amount:" + outstanding);
}
void printBanners()
{
// banner
System.out.println("*************************");
System.out.println("***** Customer Owes *****");
System.out.pringln("*************************");
}
void printOwing()
{
Enumeration e = m_orders.elements();
double outstanding = 0.0;
printBanner();
// calculate outstanding
while (e.hasMoreElements())
{
Order each = (Order)e.nextElement();
outstanding += each.getAmount();
}
// print details
System.out.println("name:" + m_name);
System.out.println("amount:" + outstanding);
}
void printOwing()→ 단순히 코드 조각 내에서 수정하지 않고 읽기만 하는 지역변수일 경우 (수정 전의 outstanding 변수) 에는 새로운 메소드의 매개변수로 줌으로써 해결할 수 있다.
{
Enumeration e = m_orders.elements();
double outstanding = 0.0;
printBanner();
// calculate outstanding
while (e.hasMoreElements())
{
Order each = (Order)e.nextElement();
outstanding += each.getAmount();
}
printDetails(outstanding);
}
void printDetails(double _outstanding)
{
// print details
System.out.println("name:" + m_name);
System.out.println("amount:" + _outstanding);
}
void printOwing()
{
Enumeration e = m_orders.elements();
double outstanding = 0.0;
printBanner();
// calculate outstanding
while (e.hasMoreElements())
{
Order each = (Order)e.nextElement();
outstanding += each.getAmount();
}
printDetails(outstanding);
}
void printOwing()→ 변수 e 값은 뽑아낸 코드 내에서만 사용되므로 매개변수로 사용하여도 문제가 없다.
{
printBanner();
double outstanding = getOutstanding();
printDetails(outstanding);
}
double getOutstanding()
{
Enumeration e = m_orders.elements();
double outstanding = 0.0;
// calculate outstanding
while (e.hasMoreElements())
{
Order each = (Order)e.nextElement();
outstanding += each.getAmount();
}
return outstanding;
}
→ outstanding의 경우에는 코드 조각 내부에서 계산이 되어 그 이후에서도 사용되기 때문에 내부에서 선언되어 값을 리턴하는 형태로 수정되었다.
최근 덧글