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

    +146.3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    <?if (count($arResult["ERRORS"])):?>
    	<?=ShowError(implode("<br />", $arResult["ERRORS"]))?>
    <?endif?>
    
    <?if (strlen($arResult["MESSAGE"]) > 0):?>
    	<?=ShowNote($arResult["MESSAGE"])?>
    <?endif?>

    уже глаза разрывает от этого классного форматирования

    Запостил: y6uTbIu_CMEXOM, 10 Ноября 2009

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

    • забыл подписать "форматирования в шаблонах"
      Ответить
      • в шаблонах без шаблонизатора так и надо
        Ответить
        • +1
          А вообще лучше смарти юзать.
          Ответить
          • тоже так раньше думал, а потом понял что нативный пхп - есть самый труЪ шаблонизатор
            Ответить
      • Нормальный вариант для шаблонов.
        Ответить
    • нормальное форматирование. я бы сказал,терпимое.
      Ответить
    • Никакого говнокода тут не вижу. Вполне приемлемое форматирование. Или вы предпочитаете когда всё в одну строчку?
      Ответить
    • Уродский говнокодище)
      помимо short_open_tags просто неудобно такое читать.
      if (count($arResult["ERRORS"])) {
          ShowError(implode("<br />", $arResult["ERRORS"]));
      }
      
      if (strlen($arResult["MESSAGE"]) > 0) {
          ShowNote($arResult["MESSAGE"]);
      }

      вот неужели так хуже было бы?
      Ответить
      • в шаблоне говна нет.

        но оно наверняка есть там , где необходимые переменные для него создавались.
        count($arResult["ERRORS"]) вычислить стоило бы заранее, результат ф-й тоже,
        но шаблонный подход и короткие теги дляэтого случая - вполне гуд
        Ответить
        • Если count вычисляется один раз, то ИМХО все равно в каком месте он будет вычисляться. Здесь даже понятнее - сразу видно с чем сравнивается
          Ответить
      • Во, первых, ты там забыл написать echo. А во-вторых, зачем тогда вообще нужны html-вставки. Давайте все писать на PHP: echo '<html>...
        Ответить
      • Ты не заметил что <?= это echo. И форматирование нативного пхп абсолютно валидное.
        Ответить
        • валидное оно это верно
          но трудночитаемое
          тяжело воспринимается, когда таких вот конструкций 1500 строк
          в один шаблон запихано по несколько разных форм\страниц\и тд
          Ответить
          • А как Вы хотели чтобы было? Покажите Ваш вариант
            Ответить
            • я бы хотел что бы было по правилам
              всю логику выносить за пределелы шаблона
              и по возможности сокращать кодичество ">" "<"
              как было показано выше
              Ответить
              • Что-то я не вижу в данном коде логики, чистый вывод с условием (в данном случае без этого никуда)
                Ответить
              • >>>по правилам
                всю логику выносить за пределелы шаблона

                бред сивой кобылы. не путай логику приложения с логикой представления.
                Ответить
    • Ух ты! Какой новый, неожиданный говнокод! Svejaque
      Ответить
    • извиняюсь перед всеми гуру, просто эти шаблоны меня убивают
      это всего лишь 6 строк и одного из сотни шаблонов для сайта.
      + ещё как обычно логика в шаблоне. Я против php в шаблонах
      на закуску смотрите говно номер 2137
      Ответить
    • А что, я уже привык. Это из битрикса. Все шаблоны написаны так и если что-то писать через echo а что-то таким образом будет полная неразбериха. Стандартизация однако.
      Ответить

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