728x90
반응형
호출하는 메소드의 기능이 너무 간단해서 내용이 뻔하다면 메소드 기능을 한개로 병합하고
삭제하자.
변경 전
String getRating()
{
return (isPassRate())? "Y" : "N";
}
boolean isPassRate()
{
return _score > 5;
}
변경 후
String getRating()
{
return (_score > 5)? "Y" : "N";
}
728x90
반응형
'IT > 리팩토링' 카테고리의 다른 글
[리팩토링] 임수변수 내용 직접 삽입 (0) | 2019.10.25 |
---|---|
[리팩토링] 메소드 추출 (0) | 2019.10.25 |
리팩토링 - 코드 개선 방법 (0) | 2019.10.25 |
[리팩토링] 클래스 멤버변수 이동 (0) | 2019.10.25 |
[리팩토링] 객체간 메소드 이동 (0) | 2019.10.25 |