Spiga

一天一个重构方法（2）：内联方法

2009-05-06 13:22:28

Inline Method：将方法内联化

一个方法，其本体如果比其名称更清楚易懂。在方法调用点插入方法本体，然后移除该方法

int GetRating()
{
	return (MoreThanFiveLateDeliveries()) ? 2 : 1;
}

bool MoreThanFiveLateDeliveries()
{
	return _numberOfLateDeliveries > 5;
}

你会发现通过下面重构合并上面的两个方法代码会更具可读性：

int GetRating()
{
    return (_numberOfLateDeliveries > 5) ? 2 : 1;
}

有时候你会遇到某个方法，其内部代码和方法名称同样清晰易懂。你就可以去掉这个方法，直接使用其中的代码。间接性可能带来帮助，但非必须的间接性总是让人不舒服。