[Resolvido] Como melhorar a validação de login

[raphaelfogaca 0]

Olá galera, sou novo em desenvolvimento e estou tentando desenvolver o sistema da minha monografia na época da faculdade.

Escrevi um trecho para validar usuário e senha para ter acesso a aplicação, mas acredito que usando foreach para percorrer toda a lista de usuários não seja a melhor opção.

 

Como posso melhorar essa validação? Obrigado!!

 

  private void btn_Entrar(object sender, EventArgs e)
        {
            string login = textUsuario.Text;
            string senha = textSenha.Text;       
           
            UsuarioEntity uEntity = new UsuarioEntity();
            IList<Usuario> lista = uEntity.Usuarios();
                                  
                foreach (var item in lista) { 
                if ((item.UsuarioLogin == textUsuario.Text) && (item.UsuarioSenha == textSenha.Text))
                {
                    MessageBox.Show("Seja bem vindo ao Controle de Rotas");
                    panelLogin.Visible = false;
                    button4.Visible = true;
                    button5.Visible = true;
                    break;
                } else{
                    MessageBox.Show("Usuário e/ou Senha Incorreto(s)");
                    textUsuario.Text = "";
                    textSenha.Text = "";
                    break;
                }
            }            
        }

 

     

[quintelab 91]

Você não precisa (não deve) percorrer toda a lista, eu vi que antes de comparar o login e senha você trouxe todos os usuários do banco, isso não é necessário, o correto é procurar apenas 1 usuário que possua o login e senha informado.

 

Nessa linha aqui: 

IList<Usuario> lista = uEntity.Usuarios();

Usuarios() é um método que você criou ou é do EntiyFramework ?

[raphaelfogaca 0]

3 horas atrás, quintelab disse:

Você não precisa (não deve) percorrer toda a lista, eu vi que antes de comparar o login e senha você trouxe todos os usuários do banco, isso não é necessário, o correto é procurar apenas 1 usuário que possua o login e senha informado.

 

Nessa linha aqui: 

IList<Usuario> lista = uEntity.Usuarios();

Usuarios() é um método que você criou ou é do EntiyFramework ?

 

Usuarios() é um método que eu criei que retorna a lista de usuarios.ToList();

[quintelab 91]

O ideal seria criar outro método, algo como:

GetUsuario(string login, string password)

Nesse seu método você irá novamente buscar os usuários no banco porém adicionando duas condições (where)

Por exemplo:

SELECT * FROM USUARIOS WHERE PASSWORD = password (parametro que recebeu no método) AND LOGIN = login (parametro que recebeu no método)

 

Se estiver usando EntiyFramework a ideia é a mesma. Caso esse método retorne algo quer dizer que o usuário foi encontrado, caso contrário não existe o usuário.

[raphaelfogaca 0]

Deu certo!
Criei outro método para validar usuário e dentro do método faço o select para buscar somente o usuário específico.

 

Obrigado!

 

public bool ValidaUsuario(string login, string senha)
        {
            using (ControleRotasContext db = new ControleRotasContext())
            {
                var query = db.Usuarios.Where(o => o.UsuarioLogin == login && o.UsuarioSenha == senha).FirstOrDefault();
                if (query != null)
                {
                    return true;
                }
                else return false;
            }
        }