重构方法并移除额外的局部变量（移除绑定）

Question

我有这门课：

public class User {
    private String name;
    private int age;
    //getters setters
}

我有一个更新用户对象的方法：

public void foo(User user) { 
    boolean needUpdate = false;
    
    if(needUpdateName(user.getName())) {
        user.setName("new name"); 
        needUpdate = true;
    }
    
    if(needUpdateAge(user.getAge())) {
        user.setAge(42);
        needUpdate = true; 
    }
    
    if(needUpdate) {
        userRepository.update(user);
    } 
}

这是一个简单的例子，仅作为示例。如何重构此代码并删除 needUpdate 变量？

Answer 1

如果你想删除局部变量，你会失去你所拥有的状态，所以你将不得不调用 2 次方法 needUpdateName() 和 needUpdateAge()

public void foo(User user) { 
    
    if(needUpdateName(user.getName()) || needUpdateAge(user.getAge()) ) {

       if(needUpdateName(user.getName())) {
         user.setName("new name"); 
       }
       
       if(needUpdateAge(user.getAge())) {
         user.setAge(42); 
       }
       userRepository.update(user);
    } 
}

另一种更优化的方法是引入 2 种新方法

• boolean updateNameIfNeeded(User user, String newName)
• boolean updateAgeIfNeeded(User user, Integer newAge)

所以他们会是这样的

private boolean updateNameIfNeeded(User user, String newName) {
        if (needUpdateName(user.getName())){
           user.setName(newName);
           return true;
        } else {
           return false;
        }
}

年龄相同

 private boolean updateAgeIfNeeded(User user, Integer newAge) {
        if (needUpdateAge(user.getAge())){
           user.setAge(newAge);
           return true;
        } else {
           return false;
        }
}

那么你的主要方法就会变成

public void foo(User user) { 

  if(updateNameIfNeeded(user, "new Name") || updateAgeIfNeeded(user, 42) ) {
      userRepository.update(user);
  } 
}

此解决方案更为理想，因为进行检查的 needUpdateName 和 needUpdateAge 函数仅被调用一次！