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

    +149

    1. 1
    2. 2
    foreach (cms::hooks('template_output') as $hook_file)
       require($hook_file);

    хуки содержатся в отдельных файлах и вставляются в код напрямую.

    Запостил: fidelcomandante, 04 Сентября 2013

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

    • Вся суть php
      Ответить
    • однострочные ифы видел, но чтобы однострочные foreach-ы писали... Мсье экономит на фигурных скобках? :)

      а по сути - если хуки не являются обьектами (а как понял они в виде функций), то spl_autoload их не подгрузит. Остаётся 2 варианта - склеивать их в один файл, или форычем подгружать по списку. Разумеется если не рассматривать "переделку на обьекты" и "запуск через обвёртку" (при таком варианте поломается код запускающий хуки).
      Ответить
      • Функциями? В том-то и дело, что хуки не являются даже функциями — просто файлы с кодом, которые вставляются в нужное место через require.
        Ответить
        • В которых согласно негласному соглашению (во как! тройная тавтология :D) лежит то, что надо. Как уже написал - в этом вся суть php.
          Ответить
        • наверное изначальный автор думал "раз запрещено использовать eval - положу ка я этот код в файл и каждый раз буду писать require". Интересно, а если бы один хук пришлось несколько раз запускать? было бы такое, как снизу?
          foreach($data as $table_row){
          $hook_input_data = $table_row;
          $table_html .= require($table_row_rendering_hook_file);
          }
          Ответить
      • > Мсье экономит на фигурных скобках? :)
        О, вот и человек, который имеет подходящее мнение ;) Я так и знал, что найду его...
        http://govnokod.ru/13730#comment194796

        В красном углу ринга @xara, защищающий стиль со скобками.
        В синем углу ринга @needle, считающий скобки вокруг одного стейтмента говном.

        Fight!
        Ответить
        • > Fight!

          Now KISS! http://fuuu.us/294
          Ответить
        • а ты сам на чьей стороне?

          В углах ринга @xara, @needle защищающие необходимость придерживатся стайлгайдов
          По центру ринга @bormand считающий, что стайлгайды - говно
          Ответить
          • > @bormand считающий, что стайлгайды - говно
            заблуждаетесь, ох заблуждаетесь... почитайте хоть, что по ссылке.
            Ответить
          • > а ты сам на чьей стороне?
            Я на стороне того способа, который юзали в проекте/конторе до меня. А если ограничений нет - пишу без лишних скобок.

            > По центру ринга @bormand считающий, что стайлгайды - говно
            Так толсто... В том треде я как раз поддерживаю стайлгайды :)
            Ответить
    • хук слева, хук справа - на тебе кодом по ебалу.
      Ответить

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