Как правильно писать php вставки в tpl файлах?

Question

[Вячеслав Лебедев]

Смотрю уроки по php, но смущает некий код, вроде этого:
(Для наглядности распишу по подробнее)

<?php if (isset($hornav)) { ?>
  <div class="main">
    <?=$hornav?>
<?php } ?>
<?php if ($header) { ?><h1><?=$header?></h1><?php } ?>

Стоит ли объединить идущий подряд код на php, а так же заменить "<?=" на "<?php echo"?

Comments

[Вячеслав Лебедев]

Попытаюсь подвести итоги, следуя "правильным на мой взгляд" приведенным здесь практикам написания кода, с пояснениями в скобках :):

<div id="container">
<?php if ($count>0): ( ":" выглядит чисто) ?>
    <?php foreach($items as $item):
        <div class="item"><?php echo (лишние символы не страшны, буду придерживаться практике wordpress) $item?></div>
    <?php endforeach ?>
<?php endif (в отличии от "}" - сразу становится понятно к чему относится) ?>
</div>

В будущем, как наиграюсь с нативным php, буду углубляться в шаблонизатор Twing.
Всем спасибо за обсуждение данного вопроса, был интерес и как оказалось не просто так он возник. :)

[FanatPHP]

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

[Вячеслав Лебедев]

Вообще то уже нет. Написал вывод еще до убедительных доказательств в использовании "<?=" вместо "<?php echo" :)

Answer 1

[Алексей Павлов]

Код выше вполне нормальный. Последние две строчки можно объединить, на работе скрипта это не скажется. Но тут есть один момент. Шаблон - это обычно html-текст, и php-инструкции в нём чужеродные (с точки зрения html), и даже расширения таких файлов обычно делают не .php, а .phtml - чтобы подчеркнуть, что это не php-скрипт с кодом. Поэтому в таких случаях используют специальный формат записи php-инструкций, причём обычно пытаются одну команду расположить в одну строчку, и не смешивать php и html в одной строчке. С этой точки зрения лучше не объединять эти две строчки в одну инструкцию php.
Я обычно не пишу скобки, как в коде выше, я использую второй вариант открытия и закрытия блоков, без фигурных скобок:

<div id="container">
<?php if ($count>0): ?>
    <?php foreach($items as $item): ?>
        <div class="item"><?=$item?></div>
    <?php endforeach ?>
<?php endif ?>
</div>

Но лучше поиграйтесь с этим вариантом и переходите на внешний шаблонизатор, например Twig. (вот есть урок по его встраиванию в сайт без фреймворка)

Comments

[FanatPHP]

Единственный нормальный ответ в этой теме.

Answer 2

[zooks]

Короткую запись
<?=$hornav?>
желательно заменить на:
<?php echo $hornav; ?>

?>
<?php

Можно объединить.

Comments

[andreyqin]

Начиная с версии 5.4.0 оператор <?= доступен всегда, несмотря на значение опции short_open_tag в php.ini.

[zooks]

@andreyqin так рекомендуется писать не из-за директивы short_open_tag, которая включена на большинстве серверов, а из-за несовместимости с XML.

[Алексей Павлов]

Скажите, почему <?=$hornav?> желательно заменить на: <?php echo $hornav; ?>? чем это лучше? Интересны аргументы.

[zooks]

@lexxpavlov см. мой комментарий выше.

[FanatPHP]

Обезьяны хором повторяют заклинание "несовместимость с XML", но ни одна не в состоянии объяснить, что это означает. @lexxpavlov ничем. Тут написан бред.

[zooks]

@FanatPHP Обезьяны донимают всех вокруг, вместо того, чтобы открыть мануал. Сокращенные теги PHP несовместимы с заголовками XML: <?xml ?>.

[Алексей Павлов]

@zooks а как связаны php и xml? это разные файлы,и обрабатывать их должны разные программы. Укажите кейс, когда это может вызвать несовместимость.

[zooks]

@lexxpavlov да любой когда заголовок не вынесен в echo.

[Алексей Павлов]

@zooks я понял, это когда вы используете xml-шаблон и инлайните в него php. Да, в этом случае вместо

<?xml version="1.0"?>
<greeting>Hello, <?="qwerty"?>!</greeting>

выводить заголовок в виде строки:

<?="<?xml version=\"1.0\"?>\n"?>
<greeting>Hello, <?="qwerty"?>!</greeting>

Ни разу не сталкивался с таким подходом, я обычно генерирую xml библиотеками типа Simple_XML. Но если бы мне пришлось генерить xml именно так, я бы добавил вывод xml-заголовка в то же место, где стоит header("Content-Type: text/xml"); потому как этот код всё равно будет вызываться для каждого такого шаблона.
Не стоит отказываться от записи <?=$x?> в пользу <?php echo $x; ?> из-за одной строчки, слишком уж большая разница.
Кстати, Twig отлично справляется с xml-документами.

[Александр Борисович]

@lexxpavlov вопрос не в том чтобы отказываться от <? коим пользоваться абсолютно нормально, вопрос в том зачем вам не нужно отказываться от <?php . Как я уже говорил в вордпрессе это в стандартах кодинга исключительно в силу обратной совместимости. У них там это считается ужасным злом, ломать старое. Потому что при такой публике этого действительно делать нельзя. Сегодня для новых проектов это уже не так важно.

[Алексей Павлов]

А я и не отказываюсь от <?php и использую его для указания команд if и for/foreach - посмотрите мой ответ. Короткие теги <?=$x?> я использую только для инлайна переменных, иначе длинная форма разобьёт html-код и сильно ухудшит читаемость шаблона.
P.S. Точнее, не использую это, потому что перешёл на Twig. Ещё присматриваюсь к JadePhp, потому как пользуюсь именно Jade в Node.js.

Answer 3

[Александр Борисович]

<?php - делается как стандарт для решений предназначенных для всех. Потому что работает всегда и везде. Вордресс настаивает на этих тегах но меня они бесят)

Сокращенные <? ?> могут вызывать у некоторых серверов проблемы. Не включена поддержка в php.ini. К тому же они совпадают в некоторых случаях с тегами в xml.

C <?= тоже самое.

Comments

[FanatPHP]

И опять обезьянки хором повторяют услышанную где-то глупость.

[Александр Борисович]

@FanatPHP напишите разработчикам вордпресса в их код стайл, зачем они просят писать полные теги. Смысл не в тегах, а в в обратной совместимости при большой популярности продукта.

Answer 4

[FanatPHP]

Беда нашего многострадального языка в том, что на одного специалиста-практика приходится десяток мартышек-теоретиков, которые ни разу в жизни не сталкивались лично с обсуждаемой проблемой, но всегда рады с умным видом излить на окружающих потоки почерпнутой в интернетах мудрости.

Единственный нормальный ответ здесь - от lexxpavlov, с которым я могу только согласиться по всем пунктам:
- Oбъединять РНР операторы нельзя. Это нарушает сематическую корректность шаблона. Вместо простого и легко формализуемого синтаксиса мы получаем кашу.
- Заменять ?= на php echo не нужно - это абсолютно бессмысленная трата символов.
- поигравшись с native php следует перейти на twig, в котором одни только наследование и автоискейпинг заменят тонну говнокода на нативном пхп.

Остальные же ответы здесь - воспалённые фантазии, не имеющие никакого отношения к реальности. Никаких проблем <?= тег не создаёт.

Comments

[Александр Борисович]

Если вы не сталкивались в вашей не воспаленной фантазии с серверами, которые не поддерживают короткие теги, не значит что их нет.