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

    +161.8

    1. 1
    $tpl->assign("L_OPENER", ($arr[7] == 40 || $arr[7] == 41 || $arr[7] == 42) ? 'opener.opener' : 'opener');

    Вот так в один из шаблонов в зависимости от одного из параметров передаётся сами видите что. Почему так? Потому что бесчётное количество людей переделывали код. По мне, так это уже перебор.

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

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

    • Это еще только снаружи, а что находится внутри, вообще ужос наверное...
      Ответить
      • Внутри ещё хлеще, можно практически весь клиентский код сюда выложить.
        Ответить
    • На текущем примере это ещё не страшно, а если условий уже штук 20 :)
      Ответить
    • Ну куда же без скотча. Будет условий штук 20, придется переделывать.
      Кстати это случайно не поп-ап? Есть подозрение...
      Ответить
      • Нет, не поп-ап. Это для окна с кнопкой ОК, в котором выбирается какое-то значение.
        Ответить
        • То есть opener.opener это действительно ссылка на дедушку? Это жесть)))
          Ответить

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