Estou escrevendo um método para desativar usuários, fiz desta forma:
[code]public boolean desativarUsuario(){
listaUsuarios = new ArrayList();
if(usuario.getLogin().equals(UsuarioDAO.getInstance().findByUsuario(login))){
ativar_desativar = true;
listaUsuarios.add(usuario);
}else{
return false;
}
return ativar_desativar;
}[/code]
Alguém teria uma sugestão de melhoria?
Nao entendi porque vc esta populando esse array de usuarios. E o porque dessa variabel ativar_desativar.
Nao poderia ser algo mais ou menos assim ??
if(usuario.getLogin().equals(UsuarioDAO.getInstance().findByUsuario(login))){
return true;
}
return false;
Se eu entendi, esse array vai conter os usuarios que foram desativados, eh isso? Se for, acho que nao devia ficar dentro desse metodo.
Sim, correto, o array irá conter os usuário desativados.