"PHP" / Говнокод #3649 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 Vasiliy, (Updated )

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

  • ava mrbig66 # Ссылка на оригинал 0

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

    SElECT

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

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

  • ava telnet # Ссылка на оригинал 0

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

  • ava Spider # Ссылка на оригинал 0

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

    • ava Lure Of Chaos # Ссылка на оригинал 0 ⇈

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

      • ava Spider # Ссылка на оригинал 0 ⇈

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

        • ava Lure Of Chaos # Ссылка на оригинал 0 ⇈

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

          • ava Spider # Ссылка на оригинал 0 ⇈

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

            • ava Vasiliy # Ссылка на оригинал 0 ⇈

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

    • ava Vasiliy # Ссылка на оригинал 0 ⇈

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

      • ava Spider # Ссылка на оригинал 0 ⇈

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

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓