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