1. Куча / Говнокод #4240

    +144

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 13 Сентября 2010

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

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

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

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

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

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

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