代码重构-1 对参数中有 bool值的拆分

最近对一个同事的代码进行重构

第1步

对参数中有 bool值的拆分

原代码如下:

private bool CheckIsInFreeTimes(GetDataForValidateLotteryResultModel data, bool isHelp)
{
bool isFree = false;

if (data.FreeTimes > 0 && (isHelp == false && data.AlreadyTakeNum < data.FreeTimes || isHelp == true && GetFriendTakeTimes(data) < data.FreeTimes))
{
isFree = true;
}

return isFree;
}

 

重构后

private static bool CheckIsInFreeTimesBySelf(GetDataForValidateLotteryResultModel data)
{
if (data.FreeTimes <= 0) return false;
if (data.AlreadyTakeNum >= data.FreeTimes) return false;
return true;

}

private static bool CheckIsInFreeTimesByHelper(GetDataForValidateLotteryResultModel data)
{
if (data.FreeTimes <= 0) return false;
if (GetFriendTakeTimes(data) >= data.FreeTimes) return false;
return true;

}

重构后,代码清晰明了