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

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
  27. 27
  28. 28
  29. 29
  30. 30
  31. 31
  32. 32
  33. 33
  34. 34
  35. 35
  36. 36
  37. 37
  38. 38
  39. 39
  40. 40
  41. 41
  42. 42
  43. 43
  44. 44
  45. 45
  46. 46
  47. 47
<?php

include "conf_db.php";
if(!empty($_REQUEST['doreg'])){
$error=array();
if(!$_REQUEST['name']) $error['name']="Имя";
if(!$_REQUEST['pass']) $error['pass']="Пароль";
if(!$_REQUEST['pass_to']) $error['pass_to']="Снова пароль";
if(!$_REQUEST['e_mail']) $error['e_mail']="E-mail";

if(!empty($error))
 {
 foreach ($error as $value)
  {
  echo "Не заполнено поле $value<br>";
  }
 include "form.php";
 }
 
 if(empty($error))
{
 $error_to=array();
if(trim($_REQUEST['pass']) !== trim($_REQUEST['pass_to'])) $error_to['pass']="Пароли не совпадают!";

if(empty($error_to)) 
{
$query="select * from users where name='".$_REQUEST['name']."'";
$id=mysql_query($query)or die(mysql_error());
$arr=mysql_fetch_array($id);
if(!empty($arr)) $error_to['db']="пользователь с таким именем уже зарегистрирован!"; 
}
if(!empty($error_to))
 {
  foreach($error_to as $value)
  {
  echo $value;
  }
 include "form.php";
 }
}
if(empty ($error_to)  && empty($error))
{

//Здесь проверка прошла.

}
?>

Скрипт регистрации

Запостил: desty desty, (Updated )

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

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

        воняет, но конкретные какашки трудно вылавливать из общей массы. тут и вывод хтмл, и инклуд, и двойные кавычки, и $_REQUEST вместо гет или пост, и непроэскейпенный скул...
        Даже уныло как-то. Хороший говнокод - как анекдот, он должен быть коротким и емким. А длинная история, местами смешная - утомляет.
        Ответить

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

          говнокод он и в африке говнокод, (хоть длинный хоть короткий)
          Ответить

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

          стандартный ПХПшный понос, с хардкодами ХТМЛя, SQL инъекцией, кашей из презентейшена, логики и DAO, инклудами посреди кода, зашитой локализацией, и нетестируемой впринципе логикой.
          Ответить

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

            Да что вы к пыхе цепляетесь Есть такая замечательная штука как MVC чтоб понять как его (паррент) использовать надо читать в обычных говонокнижках про это не пишут а пишут лишь бы было проще. Вот от этого и появляются таки образцы мысли.
            Ответить

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

              >паррент
              это типа транслитерация parrent =) ?
              лучше не используй термины, значение которых не понимаешь
              Ответить

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

              и ведь что характерно, теперь носятся со своим MVC как буддисты с писаной мантрой...
              Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

    ↑
    8
    ↓