Sistema.ingresarUsuario - if anidados

[aitrusgit]

El metodo ingresarUsuario de Sistema tiene demasiados if anidados. Podrían delegar parte de es validación en el usuario mismo:

    if(usuario.contrasena != contr){
        throw new ContrasenaInvalidaException();
    }else{
        if(!usuario.validado){
            throw new UsuarioNoValidadoException();
        }else{
            return usuario;
        }
    }

por

       usuario.validarConstrasena(contr)
       usuario.validarIngresio()
       return usuario

donde

       def validarContrasena(String cntr) {
           if (cont != this.contrasena) {
                 throw new ContraenaInvalidaException()
           }
       }

       def validarIngreso() {
           if (! this.validado) {
                 throw new UsuarioNoValidoException(),
           }
       }

[aitrusgit]

Lo mismo para Sistema.cambiarContrasena