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

[Kauperwood]

нормально так будет?

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]

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

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


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

 

[Kauperwood]

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

И вот эта штука по крону будет вам генерить сайтмап?

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

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

т.е по запросу робота?

вместо вывода в файл выводите в стдаут(через echo типа)

Ссылка скрыта от гостей

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

 

[vital]

Не, не стоит в такие дебри лезть=) Оно не для того.
А я имел ввиду
header('Content-type: text/xml');
echo $head.$urls.$legs;

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

 

[Kauperwood]

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