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!