"PHP" / Говнокод #2628 Ссылка на оригинал

0

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

    ↑
    8
    ↓