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

    • ava guest 25.02.2010 17:53 # 0

      Это еще только снаружи, а что находится внутри, вообще ужос наверное...
      Ответить

      • ava nechin 25.02.2010 18:02 # 0

        Внутри ещё хлеще, можно практически весь клиентский код сюда выложить.
        Ответить

    • ava Death 25.02.2010 18:27 # 0

      На текущем примере это ещё не страшно, а если условий уже штук 20 :)
      Ответить

    • ava guest 27.02.2010 21:51 # 0

      Ну куда же без скотча. Будет условий штук 20, придется переделывать.
      Кстати это случайно не поп-ап? Есть подозрение...
      Ответить

      • ava nechin 28.02.2010 13:14 # 0

        Нет, не поп-ап. Это для окна с кнопкой ОК, в котором выбирается какое-то значение.
        Ответить

        • ava guest 28.02.2010 16:01 # +1

          То есть opener.opener это действительно ссылка на дедушку? Это жесть)))
          Ответить

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