简单的重构示例

Question

给出以下代码：

public static bool UpdateUser(string userId, 
                                      string jobTitle)
{    
        return GetProvider().UpdateUser
            (userId, jobTitle);
}

需要更改它，并且可能不会返回布尔值，例如：

    UserProfile userProfile = new UserProfile();
    userProfile.Initialize(user.UserName, true);
    userProfile.ProfileJobTitle = jobTitle;
    userProfile.Save();

您是否应该确定它确实会返回布尔值或只是完全更改方法？

解决这类问题的正确方法是什么？

Answer 1

我相信取决于您和您要执行的操作，如果您不介意所返回的值（布尔），则可以忽略它，但是如果介意它返回false，则可以显示一条消息（即“保存未成功”或其他）。

Answer 2

大概您问题中的第二段代码是GetProvider::UpdateUser() 。

确保它确实返回布尔值的明显替代方案是使它在失败时引发异常。 根据您现有的代码有多少不返回值，这可能会节省大量的工作。 特别是，当前返回的没有值的任何内容都可以保留。 任何返回“成功”的东西都可以简单地删除返回值。 返回失败的事物将需要修改以引发异常。

如果您这样做，困难的部分将是确保所有其他代码都是异常安全的。 根据您对异常的其他使用（如果有），几乎可以肯定无论如何都要这样做，但这可能会导致问题，因此您需要尽快而不是稍后来清除其他代码中的问题。