1. Куча / Говнокод #3765

    +122

    1. 1
    Warning: file_get_contents(http://www.bbc.co.uk/radio1/programmes/schedules.xml) [function.file-get-contents]: failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in /home/atomzone/public_html/beta/at0m.rss.class.php on line 65

    Случайно найдено на: http://atomzone.co.uk/beta/audio/Blue%20Sky%20Black%20Death
    Я все понимаю, бета и все такое, но это же паблик бета...

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

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

    • ava Анонимус 23.07.2010 03:10 # 0

      php;)
      Ответить

    • ava bugmenot 23.07.2010 08:53 # −3

      паблик бета - тавтология;
      здесь beta == beta than nothin, т.е. универсальная отмазка для говённого релиза
      см. http://creatr.cc/creatr/

      Порадовал 12этажный стек трейс в консоли и резюме:
      TinyMCE detected. Various fixes applied. See browser.js for details
      Ответить

      • ava Lure Of Chaos 23.07.2010 10:20 # +1

        паблик бета - это недоделка, в доделывании до "релиза" где может учавствовать любой

        codemonkey detected. various healing methods applied. failed. please see coder.dumb for details
        Ответить

        • ava bugmenot 23.07.2010 12:21 # 0

          не совсем любой, указанный стек-трейс появляется при клике на [EDIT] и страница перегружается (последнее - грамотно)
          Ответить

          • ava Lure Of Chaos 23.07.2010 12:30 # 0

            ну главное, что это не релиз, который хотят оправдать, почему он говеный. НЕ релиз.
            а про [EDIT] - хорошая практика
            Ответить

      • ava Bjarne_Stroustrup 23.07.2010 15:41 # +2

        > паблик бета - тавтология;

        А что, бета может быть и закрытая.
        Ответить

    • ava Lure Of Chaos 23.07.2010 10:16 # +1

      рсс отвалился - бывает. где здесь говнокод?
      Ответить

    • ava wvxvw 23.07.2010 12:57 # +4

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

      • ava Lure Of Chaos 23.07.2010 13:41 # +2

        говнонастройка, а не говнокод. бить здесь не прогера, а админа
        Ответить

        • ava wvxvw 23.07.2010 14:18 # 0

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

          • ava Lure Of Chaos 23.07.2010 14:19 # +1

            и делать костыли?
            Ответить

            • ava wvxvw 23.07.2010 14:33 # 0

              Зачем, есть же например такая вещь: http://php.net/manual/en/function.simplexml-load-file.php можно и обработать ошибки. Ну или curl_init(), это же логично предположить, что какой-то URL может не существовать. Почему-то в других языках это считается нормально, и в API всегда есть возможность обработать ситуацию, когда файла нет / недоступен и т.п. а в PHP это типа норма жизни - ну, нет, так нет, пусть пользователь сам разбирается, что там случилось :)
              Ответить

              • ava Lure Of Chaos 23.07.2010 14:52 # +1

                вот именно, это пхп... тут не без костылей
                Ответить

              • ava istem 21.08.2010 02:31 # 0

                можно ещё и error_reporting'ом поманипулировать прежде чем выкладывать...
                Ответить

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