0

Tenho a seguinte função que busca no BD um usuário que satisfaça o login e senha passados por parâmetro:

public Usuario getUsuarioLogin(String pLogin, String pSenha) {       
    String hql = " select u from Usuario u "
               + " where u.login =:pLogin "
               + " and u.senha =:pSenha ";

    Query query = this.getManager().createQuery(hql);

    query.setParameter("pLogin", pLogin);
    query.setParameter("pSenha", pSenha);       

    return (Usuario) query.getSingleResult();

}

Quando passo login e senha que existem no BD tudo bem, porém oque irá retornar caso a busca dê em nada, tentei comparar o valor retornado com null, porém ele não reconheceu a condição

1 Resposta 1

0

Segundo a documentação a forma feita para indicar a ausência de resultado é o lançamento de uma NoResultException.

Nesse caso você precisará utilizar um bloco try-catch para detectar um retorno sem resultados e prosseguir conforme desejado.

Ex.:

public Usuario getUsuarioLogin(String pLogin, String pSenha) {       
    String hql = " select u from Usuario u "
               + " where u.login =:pLogin "
               + " and u.senha =:pSenha ";

    Query query = this.getManager().createQuery(hql);

    query.setParameter("pLogin", pLogin);
    query.setParameter("pSenha", pSenha);       

    try {
        return (Usuario) query.getSingleResult();
    } catch (NoResultException e) {
        return null;
    }
}

Uma outra maneira é fazer a busca utilizando o método getResultList, mesmo que o resultado seja vazio, ele não irá lançar uma exception, daí você só verifica se a lista retornada é vazia, caso negativo retorna o primeiro item da lista:

public Usuario getUsuarioLogin(String pLogin, String pSenha) {       
    String hql = " select u from Usuario u "
               + " where u.login =:pLogin "
               + " and u.senha =:pSenha ";

    Query query = this.getManager().createQuery(hql);

    query.setParameter("pLogin", pLogin);
    query.setParameter("pSenha", pSenha); 

    List users = query.getResultList();

    if (users.isEmpty()) {
        return null;
    }
    return (Usuario) results.get(0);
}
2
  • 2
    Costuma ser ruim retornar null (o Uncle Bob, no seu livro Clean Code, é um que comenta sobre isso), você força um check desnecessário em outro(s) ponto(s) do programa e cria um ponto potencial de bug. Bem melhor seria, no catch, retornar simplesmente um Usuario vazio. 21/10/2018 às 1:20
  • 1
    Sim, força uma checagem externa, nesse caso em específico, acho que o melhor seria retornar um boolean informando sucesso ou falha, não teria o risco de uma NPE ou verificações em todos os pontos que o código é utilizado 21/10/2018 às 2:02

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .