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

    +159

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    if ((valid_username($_POST['username'])) && (valid_email($_POST['email'])))
    		{
    			$username = $_POST['username'];
    			$email = $_POST['email'];
    			
    			if ($user->check_email($email))
    			{	
    				if ($user->set_config($config, $username))
    				{
    	
    					$_SESSION['checking_flag'] = $_POST['checking'];
    				}
    				else
    				{
    					$html->add_bufer("Неправильное имя пользователя или email.");
    				}
    			}
    			else
    			{
    				$html->add_bufer("Неправильное имя пользователя или email.");
    			}
    		}
    		else
    		{
    			$html->add_bufer("Неправильное имя пользователя или email.");
    		}

    досталась мне в наследство цмска, разгребаю....
    это нечто!

    Запостил: litrumb, 04 Декабря 2009

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

    • чо тут говнокодистого? =)
      Ответить
      • Ну хотя бы то, что три случая когда данные будут отвергнуты и будет выведено сообщение о ошибке. Нельзя делать все проверки в одном месте?
        Ответить
        • понимаете - есть ГОВНОкод, а есть код ... ммм... неопытного программиста.
          Вы, ИМНО, перепутали второе с первым.
          Ответить
          • Возможно, Вы и правы. Поясните тогда что есть говнокод? Может быть я и не прав
            Ответить
            • Ну например предидущий пост :)
              http://govnokod.ru/2214
              Ответить
              • Ну помнится когда то давно, при такой же задачи я бы тоже такой код накакал )) Но это от незнания которое является следствие отсутствия опыта..
                Ответить
        • Я тоже так делал раньше. Потом забил. Относительно долго это делать, да и не стоит оно того, просто ошибка, мол, неправильный логин или пароль :))
          Ответить
      • код ацтой ни мешно
        Ответить
    • if($sql->Connect("localhost",$cfg['db_user'],$cfg['db_pwd'],$cfg['db_db']))
         {
            $res = $sql->Query("SELECT * FROM `accounts` WHERE `username`='$name' LIMIT 1") or die("cannot submit login request<br />".mysql_error());
            $account = $sql->GetResultArray();
            if($account['username'] == $name)
            {
               if($account['password'] == md5($id))
               {
                  session_start();
                  
                  echo "Logged in...!";
                  echo "<meta http-equiv='Refresh' content='1; url=cpl.php' />";
                  
                  $sql->Close();
               }
               else die("wrong password!");
            }
            else die("wrong username!");
         }
      }


      мой вариант 2 летней давности :D
      Ответить
      • да уж.. всё в одной куче: и валидация, и работа с базой, и интерфейс )
        Ответить

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