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

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
  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 litrumb, (Updated )

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

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

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

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

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

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

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

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

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

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

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

  • ava nolka4 # Ссылка на оригинал 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 летней давности 😀
    Ответить

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

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

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

Помни, guest, за тобой могут следить!

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

    ↑
    8
    ↓