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

    • ava y6uTbIu_CMEXOM 10.11.2009 17:42 # −1

      забыл подписать "форматирования в шаблонах"
      Ответить

      • ava paranoid 10.11.2009 17:58 # +4

        в шаблонах без шаблонизатора так и надо
        Ответить

        • ava guest 10.11.2009 21:06 # +1

          +1
          А вообще лучше смарти юзать.
          Ответить

          • ava Yozheg 13.11.2009 07:50 # 0

            тоже так раньше думал, а потом понял что нативный пхп - есть самый труЪ шаблонизатор
            Ответить

      • ava guest 11.11.2009 01:26 # 0

        Нормальный вариант для шаблонов.
        Ответить

    • ava guest 11.11.2009 01:36 # 0

      нормальное форматирование. я бы сказал,терпимое.
      Ответить

    • ava guest 11.11.2009 09:11 # 0

      Никакого говнокода тут не вижу. Вполне приемлемое форматирование. Или вы предпочитаете когда всё в одну строчку?
      Ответить

    • ava guest 11.11.2009 10:52 # −1

      Уродский говнокодище)
      помимо short_open_tags просто неудобно такое читать.
      if (count($arResult["ERRORS"])) {
    ShowError(implode("<br />", $arResult["ERRORS"]));
}

if (strlen($arResult["MESSAGE"]) > 0) {
    ShowNote($arResult["MESSAGE"]);
}

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

      • ava paranoid 11.11.2009 12:59 # +1

        в шаблоне говна нет.

        но оно наверняка есть там , где необходимые переменные для него создавались.
        count($arResult["ERRORS"]) вычислить стоило бы заранее, результат ф-й тоже,
        но шаблонный подход и короткие теги дляэтого случая - вполне гуд
        Ответить

        • ava guest 11.11.2009 23:58 # 0

          Если count вычисляется один раз, то ИМХО все равно в каком месте он будет вычисляться. Здесь даже понятнее - сразу видно с чем сравнивается
          Ответить

      • ava guest 11.11.2009 23:56 # 0

        Во, первых, ты там забыл написать echo. А во-вторых, зачем тогда вообще нужны html-вставки. Давайте все писать на PHP: echo '<html>...
        Ответить

      • ava guest 13.11.2009 11:27 # 0

        Ты не заметил что <?= это echo. И форматирование нативного пхп абсолютно валидное.
        Ответить

        • ava y6uTbIu_CMEXOM 13.11.2009 12:55 # 0

          валидное оно это верно
          но трудночитаемое
          тяжело воспринимается, когда таких вот конструкций 1500 строк
          в один шаблон запихано по несколько разных форм\страниц\и тд
          Ответить

          • ava ivmaroz 13.11.2009 13:31 # 0

            А как Вы хотели чтобы было? Покажите Ваш вариант
            Ответить

            • ava y6uTbIu_CMEXOM 13.11.2009 13:44 # 0

              я бы хотел что бы было по правилам
              всю логику выносить за пределелы шаблона
              и по возможности сокращать кодичество ">" "<"
              как было показано выше
              Ответить

              • ava ivmaroz 13.11.2009 14:21 # 0

                Что-то я не вижу в данном коде логики, чистый вывод с условием (в данном случае без этого никуда)
                Ответить

              • ava guest 23.11.2009 03:21 # 0

                >>>по правилам
                всю логику выносить за пределелы шаблона

                бред сивой кобылы. не путай логику приложения с логикой представления.
                Ответить

    • ava getrix 11.11.2009 14:49 # 0

      Ух ты! Какой новый, неожиданный говнокод! Svejaque
      Ответить

    • ava y6uTbIu_CMEXOM 12.11.2009 16:32 # 0

      извиняюсь перед всеми гуру, просто эти шаблоны меня убивают
      это всего лишь 6 строк и одного из сотни шаблонов для сайта.
      + ещё как обычно логика в шаблоне. Я против php в шаблонах
      на закуску смотрите говно номер 2137
      Ответить

    • ava guest 20.11.2009 22:53 # 0

      А что, я уже привык. Это из битрикса. Все шаблоны написаны так и если что-то писать через echo а что-то таким образом будет полная неразбериха. Стандартизация однако.
      Ответить

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