"PHP" / Говнокод #486 Ссылка на оригинал

0

  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)},  {/if} 
{?$i++}
{/}

No comments.

Запостил: guest guest, (Updated )

Комментарии (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}, {/if}<a href="/{_ LANG}/tv,translations?tag={$tag|escape:'url'}" >{$tag|escape}</a>{/}
    Ответить
  • gf:
    > Да наверное в том, что шаблонизаторы как бы придуманы для отделения кода от представления.
    Не кода от представления, а бизнес-логики от ЛОГИКИ представления. Просто эта логика должна быть не ублюдочная и не делать лишних операций как минимум =)
    Ответить

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

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?


    8