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

    +157.7

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    {?$i=0;}
    {foreach from=explode(',',$item.tags) item="tag"}
    <a href="/{_ LANG}/tv,translations?tag={$tag|escape}">{$tag|escape}</a>{if $i < (count(explode(',',$item.tags)) - 1)},&nbsp; {/if} 
    {?$i++}
    {/}

    No comments.

    Запостил: guest, 31 Января 2009

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

    • страйко:
      в чём соль?
      Ответить
    • pnx:
      Да наверное в том, что шаблонизаторы как бы придуманы для отделения кода от представления.
      Ответить
    • страйко:
      и как бы что? где тут логика приложения? explode? ну да, explode чуть не к месту, а другое?

      p.s. что это за шаблонизатор? похож на смарти
      Ответить
    • gf:
      Это Quicky (quicky-tpl.net). Соль в том что у человека растут руки не из того места, и он делает count+explode при каждой интерации цикла, да еще и $i гоняет. Хотя можно было обойтись проверкой $i > 0 перед тегом <a>.
      Хороший вариант:
      {foreach name='tags' from=explode(',',$item.tags) item="tag"}{if !first},&nbsp;{/if}<a href="/{_ LANG}/tv,translations?tag={$tag|escape:'url'}" >{$tag|escape}</a>{/}
      Ответить
    • gf:
      > Да наверное в том, что шаблонизаторы как бы придуманы для отделения кода от представления.
      Не кода от представления, а бизнес-логики от ЛОГИКИ представления. Просто эта логика должна быть не ублюдочная и не делать лишних операций как минимум =)
      Ответить

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