1. PHP / Говнокод #11721

    +45

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    function Login($user_login, $user_password) {
        if (isset($user_loginl) && isset($user_password)) {
            $user = $this->mongo_db->get_where('users', array(
                'EMAIL' => $user_email,
                'PASSWORD' => $user_password)
            );            
            if (isset($user->email)) {
                return true;
            }
        }
        return false;
    }

    Из категорий, нет ошыбок но почемуто не работает!

    Запостил: dzen, 07 Сентября 2012

    Комментарии (13) RSS

    • ava Steve_Brown 07.09.2012 12:09 # +2

      >user_loginl
      Это, что ли? Ну да, динамические языки не придираются к опечаткам...
      Ответить

      • ava sayidandrtfm 07.09.2012 12:15 # +2

        Логинил логинил да не вылогонил.
        Ответить

      • ava bormand 07.09.2012 12:38 # +2

        В общем случае придираются. То, что пыхеры не читают нотисы - это их личные проблемы.

        В данном же случае эта ошибка, действительно, не обнаружится.
        Ответить

        • ava Lowezar 07.09.2012 13:04 # +2

          Дзен с такими "ошыбка"ми ему не познать.
          Ответить

          • ava dzen 07.09.2012 13:27 # 0

            код не мой, нашол в одной из моделей CodeIgniter, автор кода неизвестен
            Ответить

    • ava guest 07.09.2012 13:28 # 0

      'EMAIL' => $user_email,
      Так нужен же $user_login !
      Ответить

    • ava Lure Of Chaos 07.09.2012 13:31 # 0

      ко всему прочему - как я понимаю, isset($user_password) всегда будет true
      Ответить

      • ava bormand 07.09.2012 13:37 # +2

        Нет, я тоже так думал. Потом в одном из тредов выяснилось, что если $user_password содержит null - isset тоже вернет false.

        Ссылка на тот тред:
        http://govnokod.ru/11659#comment152129
        Ответить

        • ava Vasiliy 07.09.2012 16:23 # +2

          на null есть отдельная функция is_null
          Ответить

          • ava Lowezar 08.09.2012 08:53 # +1

            Да, но всё равно isset вернёт таки false если там null. :) У Борманда нет ссылки на офф, поэтому дополню: http://www.php.net/manual/en/function.isset.php
            Returns TRUE if var exists and has value other than NULL, FALSE otherwise.
            Ответить

          • ava Hamer 14.09.2012 03:24 # 0

            Если переменной нет, пых возвращает нулл при попытке обращения.
            var_dump($undeclared_var); // NULL

            соответственно isset($undeclared_var) вернет FALSE.
            Ответить

    Добавить комментарий