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

    +131

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    $user_name=$_POST['user_name'];
$query = mysql_query("SElECT login FROM user") or die(mysql_error());
$existing_users = mysql_fetch_array($query);


if (in_array($user_name, $existing_users))
  {
				   echo "no";
			   }
		  else 
		       {
				   echo "yes";
			   }

    Проверка зареган ли такой пользователь

    Запостил: Vasiliy, 06 Июля 2010

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

    • ava mrbig66 06.07.2010 18:52 # +1

      ПЦ. Создатели языка SQL плачут кровавыми слезами.

      SElECT

      ВебКил, залогиньтесь.

      Функцию COUNT() Денис Попов ещё не изобрёл.
      Ответить

    • ava telnet 06.07.2010 18:59 # +2

      Зато надёжная защита от SQL-инъекции (был когда-то такой боянчик, искать лень). Правда, вероятность того, что код будет работать как надо, обратно пропорциональна числу пользователей в таблице (1 пользователь - гарантированно заработает, 2 пользователя - 50 на 50, 3 - 1/3 и т.д.). Ещё расстановка отступов доставила.
      Ответить

      • ava zantor 06.07.2010 21:32 # 0

        Надежная защита? Спасибо, поржал :) Бинды -- вот защита
        Ответить

    • ava Lure Of Chaos 07.07.2010 00:44 # +8

      проверка залогинен ли вебкилл
      Ответить

    • ava Spider 07.07.2010 12:57 # −1

      Ребята, вы так интересно обсуждаете защиту от SQL инъекций. А вы в курсе, что этот код нерабочий?
      Ответить

      • ava Lure Of Chaos 07.07.2010 14:08 # 0

        чуть чуть рабочий. mysql_fetch_array вообще то не возвращает массив значений по колонке, а вернет массив с первым значением логина. что бы хоть какая то польза от сабжа была, там бы надо или where в скуле дать, или циклом бежать
        Ответить

        • ava Spider 07.07.2010 16:44 # +1

          "чуть чуть рабочий" - это сильно! :)
          Если калькулятор будет правильно считать только пример вида 2+2 - вы тоже скажете, что он "чуть чуть рабочий"?
          Ответить

          • ava Lure Of Chaos 07.07.2010 17:31 # +1

            ну если бы он выдавал фатальную ошибку - тогда был бы не рабочим :) а так, что-то делает, хотя не все и не так, как нужно
            Ответить

            • ava Spider 07.07.2010 18:50 # 0

              В моём понимании нерабочий — значит работает не так, как задумывал автор
              Ответить

              • ava Vasiliy 08.07.2010 06:06 # 0

                значит он рабочий откуда Вы знаете о чем думал автор. Может он Попов Женька который. а у него весь код классика ГК
                Ответить

      • ava Vasiliy 07.07.2010 16:23 # −1

        На своем сайте проверил ??? этот код рабочий только когда пользователь зареган 1. Я подозреваю что автор забыл where дописать но и это не делает код менее говнистым
        Ответить

        • ava Spider 07.07.2010 16:41 # +2

          > На своем сайте проверил ???
          <sarcasm>да, без проверки, конечно, никак не понять что этот код делает </sarcasm>
          Ответить

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