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

    +157

    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
    <?
@$ok=$HTTP_POST_VARS["ok"];
@$user=$HTTP_POST_VARS["user"];
@$pwd=$HTTP_POST_VARS["pwd];
if(!isset($ok))
echo "<form action=.$2.php. method=POST>"
."Name<input type=text name user><br>"
."Password<input type=password name=pwd><br>"
."<input type=submit name=ok value=Войти>"
."</form>";
else
{
if(($user="Demo")&&($pwd=="Demo"))
echo "Wellcome";
else
echo "Access Blocked";
}
?>

    Говорят, что это не говнокод.

    Запостил: kal1sha, 14 Июля 2010

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

    • ava Vasiliy 14.07.2010 10:26 # 0

      это не говнокод это проба пера какого то паскалиста if(($user="Demo")&& ..... не Гк а оплошность

      о а вот это echo "<form action=.$2.php. method=POST>" просто гон
      Ответить

      • ava kal1sha 14.07.2010 10:29 # −1

        те кто знает паскаль понимают, что есть разные типы данных, а этот мало что понимает.
        Ответить

    • ava Lure Of Chaos 14.07.2010 10:54 # 0

      за одно HTTP_POST_VARS бить пока не исправит

      и при этом @$ok=$HTTP_POST_VARS["ok"]; почему то, а не $ok=@$HTTP_POST_VARS["ok"]; если уж на то пошло
      Ответить

      • ava mindfreakthemon 14.07.2010 11:04 # 0

        @func($undef_var), func(@$undef_var) и @func(func2(func3(....($undef_var)))) спрячут ошибку одинаково.

        если все функции заданы, а вот переменная - нет.
        Ответить

        • ava Lure Of Chaos 14.07.2010 11:12 # +2

          вот потому и ругают пхп, что позволяет ворнинговать прямо в стдоут (в данном случае бровсер) и при этом затыкать ошибку. Так и получаем говно, когда вдруг у кодообезьянки на продакшене начинают сыпаться нотисы,
          он: не обращайте внимания!
          ему: нехорошо, исправь!
          и он затыкает этим оператором "пробка" и радуется

          лично я не в курсе, потому что как правило не затыкаю функции, при этом еррор_репортинг на макс. - в лог.

          Золотые слова друга:
          лучше видеть откуда говно потекло. Лучше пусть громко перднет, чем предательски набздит. (ц)
          Ответить

          • ava mindfreakthemon 14.07.2010 11:19 # 0

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

            • ava Lure Of Chaos 14.07.2010 11:24 # 0

              на самом деле, это не так уж сложно
              особенно с помощью сквозной функциональности (механизм исключений, паттерн-матчинг)
              Ответить

              • ava WebKill_HochetVBan 14.07.2010 13:59 # 0

                >с помощью сквозной функциональности (механизм исключений, паттерн-матчинг)
                Где? На PHP? О_о Это же не наш метод! :D
                Ответить

                • ava Lure Of Chaos 14.07.2010 14:12 # −2

                  исключения появились славабогу.
                  Ответить

                  • ava Анонимус 14.07.2010 14:57 # +2

                    ага, только ими никто не пользуется, потому что 80% API возвращает ошибку через something_error(), еще 10 через обычные php ошибки
                    Ответить

    • ava Анонимус 14.07.2010 14:16 # +3

      За HTTP_POST_VARS -- пол года (этот подход деприкейтед).
      За каждую собчку по 5 лет.
      За пропущенную кавычку -- еще год.
      За echo HTMLя еще 10.
      Итого -- пожизненное исключение из ПТУ.
      Ответить

    • ava aerowave 14.07.2010 16:12 # 0

      На 13-ой строке присвоение? хD
      Ответить

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