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

    • ололо. вот это классический говнокод. классический не в смысле боян, а в смысле вот для таких шедевров и предназначен сей ресурс!
      Ответить
      • ага)) отличные сорцы надыбал
        аффтар все сорта говнокода собрал (он и спеллчекер то не осилил)
        Ответить
    • зато наглядно, где завершаются все циклы. Очполезно!
      Ответить
      • имелось в виду это:
        клиент с таким логином уже сужествует, введите другой
        Ответить
    • Во всём надо видеть плюсы. Никакой SQL-injection не стражен :)
      Ответить
      • суровые челябинские программисты
        Ответить
        • не правда, суровые челябинские программисты так не делают, для подобного выполняется штук 100 строк кода и создается куча объектов.
          Ответить
    • показать все, что скрыто//в топку циклы, Вы конкретные гавно-кодеры, а если в базе 10к логинов?
      //опустим проверку $myLogin
      $query = "SELECT login FROM clients WHERE login = {$myLogin}";
      $result = mysql_query($query, $dbConn);
      if ($row = mysql_fetch_assoc($result))
      {
      echo "Fail >.<";
      }
      else
      {
      ...
      }
      Ответить
    • Зато код комментирован :)
      Ответить
      • да! идеально комментированный код сужествует!
        Ответить
    • показать все, что скрытоНадо было использовать COUNT(), а то ресурсов на вас не напасешься.

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


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

      ух, по сраке за такое... + всех афтаров выпускающих книжки "какая-то фигня за 21 день" линчевать
      Ответить
    • Особо радует внутренний цикл :)
      Ответить
    • Япона мать. Внутренний цикл то сразу и не заметил. Думал "куда уж хуже.."
      Ответить

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