Как по другому вывести цикл, чтобы данные вносились в таблицу, а не дублировали её?

Question

[Максим Спиридонов]

Есть вот такой цикл

while ($row = mysql_fetch_assoc($query)) {
    $tpl->set("{number}", $number++);
    $tpl->set("{id}", $row['id']);
    $tpl->set("{site_name}", $row['site_url']);
    
    if ($row['status'] == '1') {
        $tpl->set("{status}", 'Сайт работает');
        $tpl->set("{icon}", 'Сайт работает');
    }
    if ($row['status'] == '2') {
        $tpl->set("{status}", 'Сайт на модерации');
        $tpl->set("{icon}", 'fa fa-eye');
    }
    if ($row['status'] == '3') {
        $tpl->set("{status}", 'Сайт отключен');
        $tpl->set("{icon}", 'Сайт работает');
    }
    $tpl->load_template('main_user.tpl');
    $tpl->compile('content');
    $tpl->clear();
}

Затем я в шаблон main_user.tpl вывожу данные в таблицу:

<div class="panel panel-default">
    <!-- Default panel contents -->
    <div class="panel-heading">Мои сайты</div>

    <!-- Table -->
    <table class="table">
        <thead>
            <tr>
                <th>#</th>
                <th>Адрес сайта</th>
                <th>Статус</th>
                <th>Настройки</th>
            </tr>
        </thead>
        <tbody>
            <tr>
                <td>{number}</td>
                <td>{site_name}</td>
                <td><i class="{icon}"  title="{status}" data-placement="top" data-toggle="tooltip"></i></td>
                <td><a href="/cabinet/edit/{id}">Редактировать</a></td>
            </tr>

        </tbody>
    </table>
</div>

В данном случае у меня получается следующее:



Хотя должно быть так:

Comments

[Максим Спиридонов]

Radiocity: :D

Answer 1

[link_irk]

Вам нужно данные по строкам таблицы накапливать в массиве. И потом передавать уже этот массив в шаблон, где в цикле массив будет разбираться на строки таблицы. Или же выносить тело таблицы в отдельный подшаблон. И на каждую итерацию mysql_fetch_assoc создавать объект с этим шаблоном, парсить их, потом соединить в одну строковую переменную и передать основному шаблону уже готовый HTML в переменную вроде:

...
<tbody>
{sites_data}
</tbody>
...

Я вообще с шаблонизаторами не работаю. Но, полагаю, что у него должен быть механизм работы с подшаблонами. Изучите этот момент по документации к Вашему шаблонизатору

Comments

[Максим Спиридонов]

Шаблонизатор взят от cms dle, вот собственно он pastebin.com/6wJ5fGzx .

Answer 2

[FanatPHP]

Не зачеркивать домен красными чернилами?

Ну или например можно выкинуть этот помоечный шаблонизатор в соответствующее место и использовать хотя бы native PHP

$statuses = [1 => "Сайт работает", "Сайт на модерации", "Сайт отключен"];
$data = [];
$number = 1;
while ($row = mysql_fetch_assoc($query)) {
    $row['stattext'] = $statuses[$row['status']];
    $row['number'] = $neumber++;
    $data[] = $row;
}
$tpl->set("data", $data);
$tpl->load_template('main_user.tpl');

Ну и шаблон

<div class="panel panel-default">
    <!-- Default panel contents -->
    <div class="panel-heading">Мои сайты</div>
    <!-- Table -->
    <table class="table">
        <thead>
            <tr>
                <th>#</th>
                <th>Адрес сайта</th>
                <th>Статус</th>
                <th>Настройки</th>
            </tr>
        </thead>
        <tbody>
<? foreach ($data as $row): ?>
            <tr>
                <td><?=$row['number']?></td>
                <td><?=$row['site_name']?></td>
                <td>
                    <i class="icon-<?=$row['status']?>"  
                     title="<?=$row['stattext']?>" 
                     data-placement="top" data-toggle="tooltip"></i>
                </td>
                <td><a href="/cabinet/edit/<?=$row['id']?>">Редактировать</a></td>
            </tr>
<?php endforeach ?>
        </tbody>
    </table>
</div>

К вопросу о старой байке про "дизайнера". Уже не первый десяток лет дремучие пользователи похапе рассказывают друг другу страшилки про Неизвестного Дизайнера, которого никто никогда не видел, но который до судорог боится РНР. И ради него изобретают самые уродливые шаблонизаторы, которыми в итоге неудобно пользоваться ни тому "дизайнеру", ни самому программисту.

При этом все замены сводяфтся к ОБФУСКАЦИИ.

Есть у нас, к примеру, блок кода, который надо повторять, пока есть данные.
В нормальном шаблонизаторе мы пишем что-нибудь вроде FOREACH, LOOP или подобное.
В кастрированном шаблонизаторе мы СКРЫВАЕМ назначение данного блока - а то не дай бог оператор цикла увидит "дизайнер" и у него тут же слвчится заворот мозгов! Поэтому мы пишем какую-нибудь ерунду типа <!-- section -->
При этому суть блока НЕ ИЗМЕНИЛАСЬ! Это остался тот же цикл. И для корректной верстки нужно понимать, что это цикл. Но узнать об этом неоткуда... кроме РНР кода контроллера! За что боролись - на то и напоролись.

По части суеверий похапе программисты будут почище средневековых крестьян.

Comments

[Максим Спиридонов]

Нужно разделить логику сайта на код и шаблон. Так как помимо меня, в шаблон будет заглядывать дизайнер и разобраться в php коде будет сложно.

[FanatPHP]

Не надо писать ерунды. В ЭТОМ шаблоне ты и сам разобраться не можешь. потому что это не шаблон, а уродливый обрубок шаблона. В нормальном шаблонизаторе всегда видно - где цикл, а где включение подшаблона. А здесь из самого шаблона ВООБЩЕ НЕ ПОНЯТЬ, как он компилируется. И верстальщику таки ПРИДЕТСЯ лезть уже в настоящий пхп код, чтобы разобраться с шаблоном.

[shurph]

Максим Спиридонов: странно, что дизайнер не сможет разобраться в php коде, но сможет разобраться в шаблоне, в котором не может разобраться программист (т.е. вы).

Поддерживаю FanatPHP в этом вопросе: возможно, стоит выбрать другой шаблонизатор?

Answer 3

[OlegLazarenko]

Как-то вот так, хот не совсем понятно что за 'content' передается в $tpl->compile()

Цикл меняем вот так:

$tpl->load_template('main_user_thead.tpl');
$tpl->compile('content');
$tpl->clear();

while ($row = mysql_fetch_assoc($query)) {
    $tpl->set("{number}", $number++);
    $tpl->set("{id}", $row['id']);
    $tpl->set("{site_name}", $row['site_url']);
    
    if ($row['status'] == '1') {
        $tpl->set("{status}", 'Сайт работает');
        $tpl->set("{icon}", 'Сайт работает');
    }
    if ($row['status'] == '2') {
        $tpl->set("{status}", 'Сайт на модерации');
        $tpl->set("{icon}", 'fa fa-eye');
    }
    if ($row['status'] == '3') {
        $tpl->set("{status}", 'Сайт отключен');
        $tpl->set("{icon}", 'Сайт работает');
    }
    $tpl->load_template('main_user.tpl');
    $tpl->compile('content');
    $tpl->clear();
}

$tpl->load_template('main_user_tfoot.tpl');
$tpl->compile('content');
$tpl->clear();

Создаем файл шаблона main_user_thead.tpl с таким содержимым:

<div class="panel panel-default">
    <!-- Default panel contents -->
    <div class="panel-heading">Мои сайты</div>

    <!-- Table -->
    <table class="table">
        <thead>
            <tr>
                <th>#</th>
                <th>Адрес сайта</th>
                <th>Статус</th>
                <th>Настройки</th>
            </tr>
        </thead>
        <tbody>

Содержимое main_user.tpl меняем на такое:

<tr>
    <td>{number}</td>
    <td>{site_name}</td>
    <td><i class="{icon}"  title="{status}" data-placement="top" data-toggle="tooltip"></i></td>
    <td><a href="/cabinet/edit/{id}">Редактировать</a></td>
</tr>

И сооздаем файл шаблона main_user_tfoot.tpl с таким содержимым:

</tbody>
    </table>
</div>

Если бы была информация по классу, экземпларом которого является $tpl, можно было бы сказать больше. Возможно как-то можно передать объекту повторяющиеся данные, и не пришлось бы разбивать шаблон на 3 части.

Comments

[Максим Спиридонов]

Я уже разбивал таблицу подобным образом, но мне показалось это чрезмерным извращением. Использую template.Class от cms dle, вот, поглядите - pastebin.com/6wJ5fGzx

[Максим Спиридонов]

Шаблонизатор умеет делать вот такие блоки, возможно через них как-то можно.
$tpl->set_block ( '#\[block\](.+?)\[\/block\]#is', '\\1' );