Код выше вполне нормальный. Последние две строчки можно объединить, на работе скрипта это не скажется. Но тут есть один момент. Шаблон - это обычно 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. (
вот есть урок по его встраиванию в сайт без фреймворка)
