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

    +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
    $pass = 12345; 

if (!isset($_POST['pass'])) { 
$pass = $_GET['pass']; 
} 
else 
{ 
$pass = $_POST['pass']; 
} 
if ($pass !== $pass) { 

echo '<form method = "post" action = "pass.php?"> 
      Пароль:<br/> 
      <input name = "pass" type = "password"/><br/> 
      <input type = "submit" value = "OK"/></form>'; 
      } 
      echo 'Тебе сюда можна'; 
      ....блаблабла  
      } else { 
      echo 'Низя :)'; 
      break; 
      } 
      }

    >>Надо чтобы проверял если там дата[2] не пуста то показал форму инпут.
    жесть)))

    Запостил: Aligan, 23 Июня 2010

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

    • ava Denis Popov 23.06.2010 07:55 # +1

      Это не жесть, это код моего кореша Вебкилла.
      Ответить

    • ava mrbig66 23.06.2010 08:02 # +1

      Опять эта поебень — не POST, так GET.

      Человек в фигурных скобках запутался.
      Ответить

    • ava bugmenot 23.06.2010 09:05 # 0

      устаревший и неудобный способ копирования данных запроса в переменную
      Ответить

    • ava Мистер Хэнки 23.06.2010 10:08 # +7 

      if ($pass !== $pass) {  ... }

      в похапэ все так пишуть?
      Ответить

    • ava malleus 23.06.2010 11:57 # 0

      а это так принято, гнать html код напрямую? или таки правильные люди и в php используют шаблоны?
      Ответить

      • ava telnet 23.06.2010 12:03 # 0

        У новичков, обезьян, не желающих учиться, и в совсем маленьких приложениях так и принято. А так и шаблонизаторы есть, и сам PHP часто используется вместо шаблонизатора (просто весь HTML отделяется).
        Ответить

      • ava bugmenot 23.06.2010 12:22 # −1

        дОлжно быть:
        <?php if ($pass !== $pass) { ?>
<form> 
      <input name = "pass" type = "password"/><br/> 
      <input type = "submit" value = "OK"/>
</form>
<?php }  else { ?>
<!-- итд -->
        Ответить

        • ava malleus 23.06.2010 12:38 # 0

          это шаблонизатор или отделение html?
          Ответить

        • ava telnet 23.06.2010 12:42 # +4

          И что принципиально поменялось? Управляющая логика и вывод в браузер по-прежнему перемешаны. if ($pass !== $pass) вообще всегда false.
          malleus, ни то ни другое.
          Ответить

          • ava bugmenot 23.06.2010 20:22 # 0

            даже в совсем маленьких приложениях не принято распечатывать HTML
            Ответить

            • ava telnet 23.06.2010 20:33 # −1

              "Распечатывать"? Пост прогонялся через Google Translate туда-обратно? =) Ну, знаете, если мне надо "на один раз" сгенерировать sitemap.xml из карты сайта, забитой в БД, или составить таблицу со списком картинок из какой-нибудь папки, я уж точно не стану, лабая скрипты под эти задачи, страдать MVC головного мозга или использовать фреймворки.
              Ответить

              • ava bugmenot 24.06.2010 00:09 # −1

                Парирую: "вывод в браузер".
                Правильно, кому нужен DOM, когда есть такая мощная и гибкая возможность как echo Ж)
                Ответить

    • ava telnet 23.06.2010 12:15 # +3

      Код напомнил приснопамятную гостевуху от Webkill'а с её хардкодными паролями и "Здарова, %корешname%".
      Ответить

    • ava GoodTalkBot 23.06.2010 14:21 # +1

      то есть изначально переменная pass=12345, затем он её переписывает, присваивая ей get или post значение, а потом сравнивает саму с собой да есче и по типу, но сама изюминка это break; в конце :) кэп объясняет
      Ответить

      • ava Aligan 23.06.2010 15:17 # 0

        Да, про break это вообще верно заметил))))
        Ответить

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