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

    +163

    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
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    <?php
include ("config.php");
include ("css/css.inc.php");
include ("inc/audioplayer.inc.php");

// Определяем текущий раздел.

$CurSec=empty($_GET['section'])?null:$_GET['section'];

    // Если после "?section=" ничего нет, то грузим файл по умолчанию,
    // а именно "index.inc.php" из папки "sections".

    if(empty($CurSec))
    include("./sections/index.inc.php");

    // Если "?section=MyHouse" и файл "MyHouse.inc.php" в папке "sections" существует,
    // то подгружаем его.

    elseif(file_exists("./sections/".basename($CurSec).".inc.php"))
    include("./sections/".basename($CurSec).".inc.php");

    // Если "?section=MyHouse" и файл "MyHouse.inc.php" в папке "sections" НЕ существует,
    // то подгружаем "error.inc.php" всё из той же папки "sections".

    else
    include("./sections/error.inc.php");

print "$body";

?>

    Запостил: sergylens, 07 Мая 2010

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

    • ava guest 07.05.2010 14:35 # −1

      чет не отследил здесь говнокод
      Ответить

      • ava sergylens 07.05.2010 14:51 # −2

        чего только просто inlcude стоит вместо использования православного require_once в php
        Ответить

        • ava Vasiliy 07.05.2010 15:01 # −2

          Если там только конфиги то не страшно о будет у тебя в коде определятся 20 раз переменная $mama_mbIJla_ramu="Мама мыла раму"; Хотя и не приятно.
          Ответить

          • ava sergylens 07.05.2010 16:11 # 0

            сказал ахинею какую ты =)

            к авторству данного кода я отношение не имею

            а require_once лучше использовать, т.к. если нет данного файла, чтобы не работал дальше и скрипт, а _once - от переопределения.

            include - костыль
            Ответить

            • ava Vasiliy 07.05.2010 16:18 # −1

              можно также использовать include_once без особых проблем. Я тебе про, что грю если даже этот скрипт будет загружатся инклудить include ("config.php");
              include ("css/css.inc.php");
              include ("inc/audioplayer.inc.php") эти файлы не один раз а скажем 20. то кроме тормазов. ничего страшного не будет. Код говно однозначно но не только из за include. Я даже больше скажу include это не самая большая беда этого кода.
              Ответить

              • ava Lure Of Chaos 13.05.2010 19:48 # −1

                require_once - для библиотек (все равно обвалится в коде, лучше обвалится раньше чем позже)
                require - редко имеет смысл ТРЕБОВАТЬ, причем не раз
                include - вставить НЕ код - мол, нету и не надо - хотя опасно так делать, вдруг случайно или злонамеренно будет код?
                include_once - не представляю
                Ответить

    • ava Vasiliy 07.05.2010 14:46 # 0

      // Если "?section=MyHouse" и файл "MyHouse.inc.php" в папке "sections" существует
      а если не MyHouse ?? где он проверяет что параметр = MyHouse. туда можно что угодно засунуть и если этот файл будет существовать он его за инклудит хорошо что не выше дирекории sections а то можно было многе узнать о сайте и я уверен много говно кода найти там :- )
      Ответить

    • ava mrbig66 07.05.2010 15:03 # +6

      А зачем там basename? Защита от Вебкила?
      Ответить

      • ava Vasiliy 07.05.2010 15:07 # −1

        Не Вебкил в армии дачи генералам строит. Интернет может спать спокойно.
        Ответить

        • ava mrbig66 07.05.2010 15:12 # 0

          Компрессию ведрами носит и давит на массу.
          Ответить

    • ava guest 08.05.2010 00:53 # 0

      >include ("css/css.inc.php");

      Уюиватьубиватьубивать
      Ответить

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