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

    +161

    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
    if (empty($whatpage)) {  
             if (empty($catpage)) { $myFile='articles/main.dat';
             } else {
                      if (empty($subcatpage)) { $myFile="articles/$catpage/main.dat"; 
                      } else { $myFile="articles/$catpage/$subcatpage/main.dat";
                      }
             }
    } else { 
            if (empty($catpage)) { $myFile="articles/$whatpage.dat";
            } else {
                      if (empty($subcatpage)) { $myFile="articles/$catpage/$whatpage.dat"; 
                      } else { $myFile="articles/$catpage/$subcatpage/$whatpage.dat";
                      }
            }
    }
    $fh = @fopen($myFile, 'r'); 
    if (!$fh) { $myFile='articles/404.dat'; header('Status: 404'); header('HTTP/1.1 404 Not Found'); $fh = @fopen($myFile, 'r'); } 
    $data = fread($fh, filesize($myFile));

    кусок CMS с кучей бесполезной работы повторяющимся кодом и без знания об file_get_contents и file_exists

    Запостил: porese, 27 Октября 2010

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

    • Каждый говнокодер считает своим долгом написать говно CMS. да -да так и появилась Джумла.
      Ответить
    • насчет file_get_contents - это не совсем верно.
      функция file_get_contents появилась в PHP 4.3.
      возможно, код писался для предыдущей версии, а в них единственный способ получить весь контент файла - это:
      $data = fread($fh, filesize($myFile))

      насчет остального согласен - это ГК.
      Ответить
      • Не удивлюсь, если считанное из файла окажется php-кодом, в дальнейшем выводимым через eval. Как бы там ни было, не file_get_contents нужен был, а require или readfile.
        Ответить
      • Еще можно использовать file, вроде так читался файл целиком до появления file_get_contents
        Ответить
    • и, конечно же, багфикс заключается в волшебных (спекрумисты поймут, почему волшебных)) ) символах @
      Ответить
    • главный пиздец тут в собачках и каше из FS и HTTP.
      Ответить
    • Данный роутер присутствует в over9000 других "принципиальна новых, ривалюционых" CMS. Ему памятник пора ставить…
      Ответить
      • т.е. прийти всем в устновленное место и по какать ??? Чтоб горка такая получилась внушительная и табличку прикрепить памятник CMS
        Ответить

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