динамический Sitemap

  • Автор темы Kauperwood
  • Дата начала
K

Kauperwood

Гость
#1
нормально так будет?
PHP:
function sitemapgenerator() {
$urls = "";
$head = '<?xml version="1.0" encoding="UTF-8"?>
<urlset xmlns="http://www.google.com/schemas/sitemap/0.84">'."\n";
$legs = '</urlset>';
$sql = mysql_query('SELECT * FROM content');// выводим поля
while($record = mysql_fetch_array($sql)) {
$urls .= "<url>\n<loc>index.php?pageid=".$record['pageid']."</loc>\n<lastmod>".date("Y-m-d")."</lastmod>\n<changefreq>weekly</changefreq>\n<priority>0.5</priority>\n</url>\n";
}
$smaps = "sitemap.xml";
if(is_file($smaps)){
$fo = fopen($smaps, "w"); 
$fw=fputs($fo,$head.$urls.$legs);
fclose($fo);
}
else {
print "error"; 
}
}
 

vital

Больной Компом Детектед
29.01.2006
2 432
42
#2
Нет, не норм.
Первое бросается в глаза - SELECT * FROM content, когда все что юзаете $record['pageid']. Зачем тягать остальные поля?
Оставьте SELECT 'pageid' FROM content

if(is_file($smaps)) зачем? Типа если файла нету, то и не надо его создавать? Бред же.
да и вообще file_put_contents лучше юзайте.


Добавлено: И вот эта штука по крону будет вам генерить сайтмап? Мне кажется, лучше пусть скрипт выдает хмл по запросу. Ну т.е вместо вывода в файл выводите в стдаут(через echo типа).
 
K

Kauperwood

Гость
#3
по первым двум замечаниям,
спасибо поправлю.

И вот эта штука по крону будет вам генерить сайтмап?
у меня получается что генератор вызывается из начала дефолтной функции, т.е каждый раз это все сейчас обновляется.

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

вместо вывода в файл выводите в стдаут(через echo типа)
вроде чуток ознакомился, появились вопросы, пока что немного не поянтно как это работает, как это отдать роботу потом?
 

vital

Больной Компом Детектед
29.01.2006
2 432
42
#4
Не, не стоит в такие дебри лезть=) Оно не для того.
А я имел ввиду
header('Content-type: text/xml');
echo $head.$urls.$legs;

И не надо файлов никаких.
 
K

Kauperwood

Гость
#5
аа всмысле через header, теперь понял!