Куча говна / Говнокод #3765 Ссылка на оригинал

0

  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 wvxvw, (Updated )

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

  • ava bugmenot # Ссылка на оригинал 0

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

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

    • ava Lure Of Chaos # Ссылка на оригинал 0 ⇈

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

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

      • ava bugmenot # Ссылка на оригинал 0 ⇈

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

        • ava Lure Of Chaos # Ссылка на оригинал 0 ⇈

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

  • ava wvxvw # Ссылка на оригинал 0

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

      • ava wvxvw # Ссылка на оригинал 0 ⇈

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

          • ava wvxvw # Ссылка на оригинал 0 ⇈

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

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓