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

    +185.5

    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
    14. 14
    <?
//------проверяем на совпадение логинов--------
$query = "SELECT login FROM clients";
$result = mysql_query($query, $dbConn);
	while ($row = mysql_fetch_assoc($result)){
		foreach ($row as $val){
			if ($val==$login){
				$flag_login='true';
			}// if	
		} // завершение цикла foreach
	} // завершение цикла while 
	//----------------------------------------
if ($flag_login=='true'){$output.= "<li class='text_blue'>клиент с таким логином уже сужествует, введите другой</li><br>"; $z='1';};
?>

    (говно)проверка на совпадение логинов

    Запостил: xXx_totalwar, 18 Января 2010

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

    • ava Dreyk 18.01.2010 18:36 # +9

      ололо. вот это классический говнокод. классический не в смысле боян, а в смысле вот для таких шедевров и предназначен сей ресурс!
      Ответить

      • ava xXx_totalwar 18.01.2010 18:44 # 0

        ага)) отличные сорцы надыбал
        аффтар все сорта говнокода собрал (он и спеллчекер то не осилил)
        Ответить

    • ava striker 18.01.2010 18:59 # +3

      зато наглядно, где завершаются все циклы. Очполезно!
      Ответить

      • ava xXx_totalwar 18.01.2010 19:04 # +6

        имелось в виду это:
        клиент с таким логином уже сужествует, введите другой
        Ответить

    • ava sbb 18.01.2010 23:49 # +12

      Во всём надо видеть плюсы. Никакой SQL-injection не стражен :)
      Ответить

      • ava Oleg_quadro 20.01.2010 02:01 # +3

        суровые челябинские программисты
        Ответить

        • ava guest 20.01.2010 08:57 # 0

          не правда, суровые челябинские программисты так не делают, для подобного выполняется штук 100 строк кода и создается куча объектов.
          Ответить

    • ava guest 19.01.2010 07:45 # −14

      показать все, что скрыто//в топку циклы, Вы конкретные гавно-кодеры, а если в базе 10к логинов?
      //опустим проверку $myLogin
      $query = "SELECT login FROM clients WHERE login = {$myLogin}";
      $result = mysql_query($query, $dbConn);
      if ($row = mysql_fetch_assoc($result))
      {
      echo "Fail >.<";
      }
      else
      {
      ...
      }
      Ответить

    • ava VanSanblch 19.01.2010 10:12 # +3

      Зато код комментирован :)
      Ответить

      • ava striker 19.01.2010 10:54 # +6

        да! идеально комментированный код сужествует!
        Ответить

    • ava guest 19.01.2010 20:52 # −6

      показать все, что скрытоНадо было использовать COUNT(), а то ресурсов на вас не напасешься.

      <?php
$isset = mysql_query("SELECT COUNT(*) AS count FROM clients WHERE login = '".$login."'";
echo mysql_result($isset,0,'count') > 0 ? 'Занято' : 'OK';
?>


      mrbig66
      Ответить

    • ava Pencioner 20.01.2010 01:16 # +6

      Ай малацца! :)
      И что самое интересное, эти люди пишут в резюме "знание МуSQL!Э :)

      ух, по сраке за такое... + всех афтаров выпускающих книжки "какая-то фигня за 21 день" линчевать
      Ответить

    • ava Suor 21.01.2010 15:13 # +2

      Особо радует внутренний цикл :)
      Ответить

    • ava guest 24.01.2010 18:57 # +4

      Япона мать. Внутренний цикл то сразу и не заметил. Думал "куда уж хуже.."
      Ответить

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