- 1
- 2
- 3
- 4
- 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++}
{/}-
PHP / Говнокод #486
+157.7
No comments.
Запостил:
guest,
31 Января 2009
Follow us!
в чём соль?
Да наверное в том, что шаблонизаторы как бы придуманы для отделения кода от представления.
и как бы что? где тут логика приложения? explode? ну да, explode чуть не к месту, а другое?
p.s. что это за шаблонизатор? похож на смарти
Это 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>{/}
> Да наверное в том, что шаблонизаторы как бы придуманы для отделения кода от представления.
Не кода от представления, а бизнес-логики от ЛОГИКИ представления. Просто эта логика должна быть не ублюдочная и не делать лишних операций как минимум =)