Блог

Я хочу входить в систему своих пользователей, только если их учетная запись активирована, но по какой-то причине запрос, который должен вернуть мне пользователя (я называю это Role в моей программе) при предоставлении правильных учетных данных, вообще ничего не возвращает.

Я на 100% уверен, что учетные данные верны, также я уверен, что в базе данных нет дубликатов и синтаксис запроса правильный.

Итак, почему этот метод внутри одного из моих EJB не возвращает true?:

 // Checks if the account is not activated!
    public boolean isActivated(String email, String password) {
        // 1-Send query to database to see if that user exist
        Query query = em
                .createQuery("SELECT r FROM Role r WHERE r.email=:emailparam AND r.password=:passwordparam");
        query.setParameter("emailparam", email);
        query.setParameter("passwordparam", password);

        List<Object> tmpList = query.getResultList();
        if (tmpList.isEmpty() == false) {
            System.out.println("IS NOT EMPTY");
            Role role = (Role) tmpList.get(0);
            if (role.getAccountStatus().trim()
                    .equalsIgnoreCase(AccountStattus.ACTIVATED.toString())) {
                // The account is activated!
                System.out.println("ACTIVATED!!!!!!!");
                return true;
            }
        }
        // The account is not activated!
        System.out.println("NOT ACTIVATED!!!!!!!");
        return false;
    }
  

Единственное сообщение, которое я вижу в консоли, НЕ АКТИВИРОВАНО !!!!

Обновить

Чтобы подтвердить, что список пуст на 100%, я изменил свой код, чтобы он выглядел следующим образом:

 // Checks if the account is not activated!
    public boolean isActivated(String email, String password) {
        // 1-Send query to database to see if that user exist
        System.out.println("HERE:"   email password);
        Query query = em
                .createQuery("SELECT r FROM Role r WHERE r.email=:emailparam AND r.password=:passwordparam");
        query.setParameter("emailparam", email);
        query.setParameter("passwordparam", password);

        List<Object> tmpList = query.getResultList();
        Iterator<Object> it = tmpList.iterator();
        int elements = 0;
        while(it.hasNext()) { 
               elements  ;
             }


        if (elements > 0) {
            System.out.println("IS NOT EMPTY");
            Role role = (Role) tmpList.get(0);
            if (role.getAccountStatus().trim()
                    .equalsIgnoreCase(AccountStattus.ACTIVATED.toString())) {
                // The account is activated!
                System.out.println("ACTIVATED!!!!!!!");
                return true;
            }
        }
        // The account is not activated!
        System.out.println("NOT ACTIVATED!!!!!!!");
        return false;
    }
  

Метод по-прежнему возвращает false, когда я ввожу правильные учетные данные. Что не так?

Шаги по отладке,

1. Запустите тот же запрос, используя любой SQL-клиент
2. Проверьте свой Role класс. У него должны быть свойства email и password с соответствующими параметрами получения / установки

И, пожалуйста, не говорите мне, что пароль в базе данных является hashed-version .

Несколько основных предложений по рефакторингу,

1. Напишите свой запрос таким образом, чтобы он был удобен для восприятия,

    Query query = em.createQuery("from Role r where r.email=:email and r.password=:password")
                   .setParameter("email", email)
                   .setParameter("password", password);
     

2. Измените свое условие на это,

    if (!list.isEmpty()) {...}
     

Ваш список запросов должен быть пустым. Это либо проблема с базой данных, к которой вы подключены (возможно, не с той, которую, по вашему мнению, вы используете), запросом, параметрами или базой данных.