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

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

      Порадовал 12этажный стек трейс в консоли и резюме:
      TinyMCE detected. Various fixes applied. See browser.js for details
      Ответить
      • паблик бета - это недоделка, в доделывании до "релиза" где может учавствовать любой

        codemonkey detected. various healing methods applied. failed. please see coder.dumb for details
        Ответить
        • не совсем любой, указанный стек-трейс появляется при клике на [EDIT] и страница перегружается (последнее - грамотно)
          Ответить
          • ну главное, что это не релиз, который хотят оправдать, почему он говеный. НЕ релиз.
            а про [EDIT] - хорошая практика
            Ответить
      • > паблик бета - тавтология;

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

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