Как сделать вывод новостей через TPL?

Question

[Nokira Nokawa]

У меня вот в чем проблема:
Есть шаблон на TPL (main.tpl) главная страница
На ней отображаются новости, но когда я делаю их вывод, но выводится только одна последняя новость, а не все. Как это можно исправить? Вот код вывода новостей:

//Вывод новостей
$result = mysql_query ("SELECT * FROM news ORDER by id");
$myrow = mysql_fetch_array ($result) ;
 
do {
    $tpl->set_value('id',$myrow['id']);
    $tpl->set_value('N_TITLE',$myrow['n_name']);
    $tpl->set_value('SHORT',$myrow['descr']);
    $tpl->set_value('AUTHOR',$myrow['author']);
    echo $news;
} while ($myrow = mysql_fetch_array ($result));

Вот часть кода из main.tpl отвечающая за блок с новостями:

<div class="news_c">
	<div class="news_name">
		<h2>{N_TITLE}</h2>
	</div>
	<div class="news_short">
		<h4>{SHORT}... | Автор: {AUTHOR}</h4>
	</div>
</div>

Вот скриншот самого сайта:


Шаблонизатор писал по книге - самый простенький:

<?php
class template_class{
	var $values = array();
	var $html;
	
	function get_tpl($tpl_name){
		if(empty(tpl_name) || !file_exists($tpl_name)){
			return false;
		}else{
			$this->html = join('',file($tpl_name));
		}
	}
	
	function set_value($key,$var){
		$key = '{'.$key.'}';
		$this->values[$key] = $var;
	}
	
	function tpl_parse(){
		foreach($this->values as $find => $replace){
			$this->html = str_replace($find, $replace, $this->html);
		}
	}
}
$tpl = new template_class;
?>

Answer 1

[k2lhu]

Все правильно, используйте цикл для вывода записей к примеру for:

{% for user in users %}
{{ user.username|e }}
{% endfor %}

Просто так вам новости все сразу не выведутся, это же общание к массиву идут, еще есть смысл проверить массив, распечатайте его var_dump() либо print_r():
var_dump($a);
Посмотрите что в него попадает.
А то как вы хотите распечатать выведет только одну запись :)

Еще хотел бы добавить - используйте PDO или mysqli для работы с БД. Забудьте про mysql в срочном порядке.

Comments

[Nokira Nokawa]

Можно по подробнее?

[k2lhu]

Nokira Nokawa: поподробнее о чем именно? :)

[Nokira Nokawa]

k2lhu: почему забыть про MySQL?

[k2lhu]

Nokira Nokawa: не MySql, а mysql_query - уже не актуально и не поддерживается разработчиками, а при дампе памяти, сервер на него ругается, к тому же данное расширение небезопасно. В любых веб студиях и компаниях его уже давно не используют. Используйте mysqli_query либо PDO ^) На mysqli_query вам будет проще перейти. Чтобы точнее найти то, что вам надо, попробуйте в поиске искать "mysqli подключение к бд, подготовленный запросы", аналогично и с PDO.

Answer 2

[Stalker_RED]

Вы бы хоть написали, что за шаблонизатор у вас.
Похоже на смарти, если я угадал, то вот пример вывода циклом:
www.smarty.net/docsv2/ru/language.function.foreach.tpl

Comments

[Nokira Nokawa]

he he: самопис. Писал по учебнику

[Nokira Nokawa]

Stalker_RED, Шаблонизатор - самопис. Писал по учебнику. Прикрепил вверху

[Stalker_RED]

Nokira Nokawa: Ну, тогда есть два варианта:
1. Дописать шаблонизатор добавив туда обработку массивов и циклы.
2. Использовать готовый шаблонизатор, в котором помимо замены текста есть еще какие-то "фичи".

[Nokira Nokawa]

Stalker_RED: а не могли бы вы мне с этим помочь?Я новичок во всем этом :D