Куча говна / Говнокод #4240 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
<!-- Редактировать/удалить -->
<ul class="action">
	<!-- говнокод --><li style="display: none;"></li><!-- /говнокод -->
	{if $oUserCurrent and ($oUserCurrent->getId()==$oTopic->getUserId() or $oUserCurrent->isAdministrator() or $oBlog->getUserIsAdministrator() or $oBlog->getUserIsModerator() or $oBlog->getOwnerId()==$oUserCurrent->getId())}
		<li class="edit"><a href="{cfg name='path.root.web'}/{$oTopic->getType()}/edit/{$oTopic->getId()}/" title="{$aLang.topic_edit}">{$aLang.topic_edit}</a></li>
	{/if}
	{if $oUserCurrent and ($oUserCurrent->isAdministrator() or $oBlog->getUserIsAdministrator() or $oBlog->getOwnerId()==$oUserCurrent->getId())}
		<li class="delete"><a href="{router page='topic'}delete/{$oTopic->getId()}/?security_ls_key={$LIVESTREET_SECURITY_KEY}" title="{$aLang.topic_delete}" onclick="return confirm('{$aLang.topic_delete_confirm}');">{$aLang.topic_delete}</a></li>
	{/if}
</ul>

3-я строчка.
/me выдал (: Видимо лень было думать и оборачивать IF'ом, а валидацию не проходило без <li> внутри <ul>. Понадобилось время, чтобы понять свой же говнокод.

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

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

  • <!-- говнокод --> - это в продакшене? ))

    вонючий Смарти детектед
    Ответить
    • в девелопмент пока что =)
      с вонючим смарти что поделать-то мне? в лайвстрите оно юзается. не заниматься же вырезанием..
      Ответить
      • > с вонючим смарти что поделать-то мне?
        мучайтесь учитесь. Хороший программист, в отличие от плохого, не только хорошо программит, но еще и умеет поддерживать говно.
        так сказать, per asperas ad astrum
        Ответить
  • а зачем нужен смарти?
    Если ты всё равно пишешь на пхп -- используй PHP как шаблонизатор. Зачем же еще смарти городить?

    Чем {if лучше чем <?if
    Ответить
        • Интересный Вы человек. Что сказали - то и использую. Может расскажите про общедоступные альтернативы LiveStreet?
          Ответить
          • >>Что сказали - то и использую
            понятно)

            мне просто интересно -- почему было принято именно такое решение.

            >> Может расскажите про общедоступные альтернативы LiveStreet?
            для начала озвучьте бизнес-задачу)
            Ответить
            • Задача - социальная сеть. Некоторый функционал LS даже избыточен для неё.
              Решение принял не я. Т.к. сначала задача состояла в натяжке дизайна - ничего подозрительного я не увидел. Потом написание своего функционала, переписывание JS, стопиццот переделок. В процессе выяснилось, что лайвстрит не так уж поворотлив.
              Я знаю одну альтернативу ему - взять какой-то Django и сделать. Затраты по времени были бы одинаковы, если сразу так сделал бы.
              Ответить
                • Особенно порадовало Bigstreet CMS. Вы серьёзно предлагаете заняться некрофилией? Оно же сдохло.
                  Нужно уметь думать, а не искать. Вы показали альтернативы, но они ни чем не лучше.
                  Ответить

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

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

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


    8