tengo esta clase:
public class User { private String name; private int age; //getters setters }
Tengo un método que actualiza un objeto de usuario:
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); } }
Es un ejemplo simple, solo como ejemplo. ¿Cómo puedo refactorizar este código y eliminar la variable needUpdate
?
Si desea eliminar la variable local, perdería el estado que tenía, por lo que deberá invocar 2 veces los métodos needUpdateName()
y 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); } }
Otra forma más óptima sería introducir 2 nuevos métodos
boolean updateNameIfNeeded(User user, String newName)
boolean updateAgeIfNeeded(User user, Integer newAge)
Entonces serían algo como
private boolean updateNameIfNeeded(User user, String newName) { if (needUpdateName(user.getName())){ user.setName(newName); return true; } else { return false; } }
Igual para la edad
private boolean updateAgeIfNeeded(User user, Integer newAge) { if (needUpdateAge(user.getAge())){ user.setAge(newAge); return true; } else { return false; } }
Entonces su método principal se convertirá en
public void foo(User user) { if(updateNameIfNeeded(user, "new Name") || updateAgeIfNeeded(user, 42) ) { userRepository.update(user); } }
¡Esta solución es más óptima porque las funciones needUpdateName
y needUpdateAge
que hacen que las comprobaciones se invoquen una sola vez!