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

    +156

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    <?php

if(true) {
...
}
else $blacklist_numbers_error="неверный тип файла";

?>

    разбираю на работе код ><

    Запостил: DmitryDick, 22 Июля 2010

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

    • ava Lure Of Chaos 22.07.2010 13:12 # +2

      заглушка
      Ответить

      • ava 3.14159265 22.07.2010 13:19 # 0

        а умные компилеры и иде скажут unreachable code.
        Ответить

        • ava Kornew 22.07.2010 13:36 # 0

          в пхп разве есть компилер?(
          Ответить

        • ava Lure Of Chaos 22.07.2010 13:55 # 0

          > умные компилеры
          дык умные, не пхп
          > в пхп разве есть компилер?(
          а по моему там интерпретатор. Все таки транслятор не значит компилятор
          Ответить

          • ava 3.14159265 22.07.2010 13:58 # −1

            >>>дык умные, не пхп
            отож
            Ответить

            • ava Lure Of Chaos 22.07.2010 14:09 # 0

              умные, к примеру, если файл не удалось открыть на чтение, прерывают нормальный ход исполнения, а пхп всего лишь кидает ворнинг (причем в браузер, если не настроено иначе) и радостно исполняет код, где производится чтение, причем, мало того, раз файл не окрыт и дескриптор файла не получен, то и читать нечего, функция fgetc (как правило юзают именно эту) возвращает false, опять кидает ворнинг, и продолжает скрипт как ни в чем ни бывало и в зависимости от говенности скрипта мы можем повесить систему тыщами этих ворнингов...а в итоге говноскрипт, скажем,посимвольного копирования при отсутствии файла таки произведет новый файл с записью false.
              Это градус дебильности пхп, просто акуеть.

              кстати, братцы, напомните, если аналогичный код написать на с, мы че поломаем?
              Ответить

              • ava 3.14159265 22.07.2010 15:38 # 0

                >>>причем в браузер, если не настроено иначе
                я не в теме, а че для пхп нет ниче вроде log4?
                Ответить

                • ava Анонимус 22.07.2010 16:02 # 0

                  http://php.net/manual/en/function.set-error-handler.php
                  Ответить

                  • ava 3.14159265 22.07.2010 16:54 # 0

                    дак это использование стандартных средств
                    и доработка напильником - function myErrorHandler - чего тебе надо
                    а я о нормальном логгере с встроенными аппендерами, уровнями логирования и пр.
                    а тут все надо руками...

                    хотя с другой стороны хороший пример, того на пхп можно писать годный код, и если руки не из жопы можно написать нормальный хендлер а не пугать юзеров ворнингами вверху страницы или не затыкать их.
                    Ответить

                • ava Lure Of Chaos 22.07.2010 17:43 # 0

                  мля, ну я ж для примера говорю, какой логгер у того, у кого скрипты ворнингами кидаются?
                  я вообще про дизайн пхп
                  Ответить

              • ava da4ever 22.07.2010 15:42 # 0

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

                • ava Lure Of Chaos 22.07.2010 17:45 # 0

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

                  • ava da4ever 22.07.2010 21:14 # 0

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

            • ava Lure Of Chaos 22.07.2010 14:11 # 0

              вот честно, лучше бы разрабы пхп вместо всех этих нотисов сразу кидали fatal error, убивая скрипт
              Ответить

        • ava Анонимус 22.07.2010 14:00 # 0

          интерпретатор php разбирает код по ходу
          Ответить

    • ava ReallyBugMeNot 23.07.2010 08:32 # 0

      Есть мнение http://cssing.org.ua/2010/04/15/rit-2010/
      что язык так и должен себя вести - не заваливаться в фатальную ошибку, а выдавать ворнинг.
      Ответить

      • ava Lure Of Chaos 23.07.2010 10:29 # 0

        окей, не фатальную. но хотя бы что-то в виде исключения, которое можно программно обработать. Скрипт НЕ должен делать вид, что ничего не произошло, если что-то произошло и втихую вести хронику проблем.
        жестче надо с программерами - глядишь, и людьми станут
        Ответить

      • ava 3.14159265 23.07.2010 11:52 # +1

        кстати линк доставляет - смотрите наш браузер на специально созданном нами тесте и полюбому заточенный по этот тест рвет/наравне со всеми остальными )))
        Ответить

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