Oque há de errado com alterar um objeto, que foi passado para outro método ?, ex:
public void doSomething(User user){
edit(user);
//...
EntityManager.persist(user);
}
public void edit(User user){
//...
user.setName("new name");
}
Em geral todos dizem que a forma abaixo seria a "mais correta"
public void doSomething(User user){
User updatedUser = edit(user);
//...
EntityManager.persist(updatedUser);
}
public User edit(User user){
//...
user.setName("new name");
return user;
}
Mas oque não entendo é porque retornar a mesma instância do objeto alterado ? Alguns dizem que isso é menos propenso a erros, porque na primeira forma você acaba perdendo o "rastro" de onde o objeto está sendo alterado. Sinceramente não esse argumento não me convenceu.
Existe alguma "boa prática" para esses casos, ou seria mesmo uma questão ideológica de desenvolvedor ?