Remove Paramterは必要のないメソッドのパラメーターを削除するというリファクタリングです。
パラメーターを削除することで、コードが読みやすくなり、全体的にすっきりします。またバグも少なくなります。
パラメーターをどのようにするのが最適かは、常に状況によって変化します。
どうしても必要な時にはパラメーターを追加し、不要になったらさっさとパラメーターを削除しましょう。
リファクタリング前:
public int GetNumberOfMembers(bool includingUnregistered)
{
int number = customers.Count;
if(includingUnregistered)
{
number += unregisteredCustomers.Count;
}
return number;
}
リファクタリング後:
public int GetNumberOfMembers()
{
return customers.Count;
}
public int GetNumberOfUnregisteredMembers()
{
return unregisteredCustomers.Count;
}
|
Copyright(c) 2012 WoodenSoldier Software |