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

    • ava nolka4 04.12.2009 09:15 # +1

      чо тут говнокодистого? =)
      Ответить

      • ava Abductio 04.12.2009 10:05 # 0

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

        • ava guest 04.12.2009 11:07 # +3

          понимаете - есть ГОВНОкод, а есть код ... ммм... неопытного программиста.
          Вы, ИМНО, перепутали второе с первым.
          Ответить

          • ava Abductio 04.12.2009 11:18 # 0

            Возможно, Вы и правы. Поясните тогда что есть говнокод? Может быть я и не прав
            Ответить

            • ava guest 04.12.2009 11:45 # 0

              Ну например предидущий пост :)
              http://govnokod.ru/2214
              Ответить

              • ava Abductio 04.12.2009 12:15 # +1

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

        • ava nolka4 04.12.2009 11:59 # 0

          Я тоже так делал раньше. Потом забил. Относительно долго это делать, да и не стоит оно того, просто ошибка, мол, неправильный логин или пароль :))
          Ответить

      • ava guest 04.12.2009 19:01 # 0

        код ацтой ни мешно
        Ответить

    • ava nolka4 04.12.2009 12:07 # 0 

      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
      Ответить

      • ava cheef 04.12.2009 18:01 # +1

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

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