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

    +149

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

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

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

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

    • ava anonimb84a2f6fd141 04.09.2013 16:40 # −1

      Вся суть php
      Ответить

    • ava xara 05.09.2013 14:38 # −1

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

      а по сути - если хуки не являются обьектами (а как понял они в виде функций), то spl_autoload их не подгрузит. Остаётся 2 варианта - склеивать их в один файл, или форычем подгружать по списку. Разумеется если не рассматривать "переделку на обьекты" и "запуск через обвёртку" (при таком варианте поломается код запускающий хуки).
      Ответить

      • ava fidelcomandante 05.09.2013 14:43 # 0

        Функциями? В том-то и дело, что хуки не являются даже функциями — просто файлы с кодом, которые вставляются в нужное место через require.
        Ответить

        • ava anonimb84a2f6fd141 05.09.2013 14:59 # 0

          В которых согласно негласному соглашению (во как! тройная тавтология :D) лежит то, что надо. Как уже написал - в этом вся суть php.
          Ответить

        • ava xara 05.09.2013 15:09 # 0

          наверное изначальный автор думал "раз запрещено использовать eval - положу ка я этот код в файл и каждый раз буду писать require". Интересно, а если бы один хук пришлось несколько раз запускать? было бы такое, как снизу?
          foreach($data as $table_row){
          $hook_input_data = $table_row;
          $table_html .= require($table_row_rendering_hook_file);
          }
          Ответить

      • ava bormand 05.09.2013 15:36 # +1

        > Мсье экономит на фигурных скобках? :)
        О, вот и человек, который имеет подходящее мнение ;) Я так и знал, что найду его...
        http://govnokod.ru/13730#comment194796

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

        Fight!
        Ответить

        • ava roman-kashitsyn 05.09.2013 16:12 # +1

          > Fight!

          Now KISS! http://fuuu.us/294
          Ответить

        • ava xara 05.09.2013 16:17 # −3

          а ты сам на чьей стороне?

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

          • ava roman-kashitsyn 05.09.2013 16:25 # +1

            > @bormand считающий, что стайлгайды - говно
            заблуждаетесь, ох заблуждаетесь... почитайте хоть, что по ссылке.
            Ответить

          • ava bormand 05.09.2013 17:30 # +1

            > а ты сам на чьей стороне?
            Я на стороне того способа, который юзали в проекте/конторе до меня. А если ограничений нет - пишу без лишних скобок.

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

    • ava nonamez 05.09.2013 16:54 # +1

      хук слева, хук справа - на тебе кодом по ебалу.
      Ответить

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