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

    +159.3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    switch ($_SERVER['QUERY_STRING'])
{
case "comments":
                $content = $_POST['addcomment'] ? $cmt->process_addcomment() : $this->main_comments($_GET['id']);
            break;

    Нашел в очередной cms.

    Запостил: xynta, 17 Февраля 2010

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

    • ava guest 17.02.2010 21:59 # +2

      Спортивное нахождение говнокода в самопальных CMS следует признать олимпийским видом спорта и включить в программу игр 2014 года в Сочи :)
      Ответить

    • ava xynta 18.02.2010 03:24 # 0

      Эта кмс не очень то самопальная. EvoTopSites
      Ответить

    • ava guest 18.02.2010 11:31 # 0

      объясните в чем говнокод..
      Ответить

      • ava guest 18.02.2010 12:04 # 0

        Общая кривизна. Мутный switch по QUERY_STRING, ?: используется вместо if'а, нет проверок isset.
        Ответить

        • ava guest 18.02.2010 14:14 # 0

          Со всем согласен, а вот тернарник зря обидели.
          Ответить

          • ava guest 18.02.2010 17:14 # −1

            Тренарник я не обижал, я его очень люблю и уважаю. Но использовать его для выбора действия (если верить именам методов) - извращение
            Ответить

    • ava xynta 18.02.2010 17:39 # +1

      Оу нет!!! Вы ничего не поняли!
      Если $_SERVER['QUERY_STRING']=='comments', то $_GET['id'] в принципе существовать не может. Я это имел в виду.

      А тернарник - это не извращение, а один из наиболее "прямых" методов выбора действия
      Ответить

      • ava guest 18.02.2010 18:57 # 0

        Ну тогда можно вообще if устранить. Будет такой функциональный Пыхохаскель.
        Ответить

      • ava guest 24.02.2010 01:36 # 0

        Зачет.
        Ответить

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