Сообщение #2 korshunov » 02.01.2017, 08:04
http://demookay.com/sitemap.xmlДобавлено спустя 30 минут 39 секунд:Замечания для разработчиков:
1. sitemap.php и sitemap_generate.php вроде бы должны создавать одинаковую XML-структуру. А в файлах в значительной степени дублируется код, в первом пишется понятно и прозрачно
print "\t<url>"."\n";
print "\t\t<loc>$url</loc>"."\n";
а во-втором - монстрообразный код
file_put_contents('sitemap'.$l.'_'.$sitemap_index.'.xml', "\t<url>"."\n", FILE_APPEND);
file_put_contents('sitemap'.$l.'_'.$sitemap_index.'.xml', "\t\t<loc>$url</loc>"."\n", FILE_APPEND);
Это ж явно нерационально. По-хорошему надо иметь ОДНУ основу для создания XML, и использовать ее в двух местах...
2. Простому пользователю тяжело догадаться, что для языка надо использовать
http://demookay.com/sitemap_generate.php?lang_id=3и уж совсем невозможно догадаться до явно усложненного адреса
http://demookay.com/sitemap.xml?lang_label=en&lang_id=23. А потом еще разбираться с тем, как использовать НЕСКОЛЬКО сгенерированных файлов...
4. Надо бы дать для пользователей хоть простенькое описание того, как с этим работать...
http://demookay.com/sitemap.xml
Замечания для разработчиков:
1. sitemap.php и sitemap_generate.php вроде бы должны создавать одинаковую XML-структуру. А в файлах в значительной степени дублируется код, в первом пишется понятно и прозрачно
print "\t<url>"."\n";
print "\t\t<loc>$url</loc>"."\n";
а во-втором - монстрообразный код
file_put_contents('sitemap'.$l.'_'.$sitemap_index.'.xml', "\t<url>"."\n", FILE_APPEND);
file_put_contents('sitemap'.$l.'_'.$sitemap_index.'.xml', "\t\t<loc>$url</loc>"."\n", FILE_APPEND);
Это ж явно нерационально. По-хорошему надо иметь ОДНУ основу для создания XML, и использовать ее в двух местах...
2. Простому пользователю тяжело догадаться, что для языка надо использовать
http://demookay.com/sitemap_generate.php?lang_id=3
и уж совсем невозможно догадаться до явно усложненного адреса
http://demookay.com/sitemap.xml?lang_label=en&lang_id=2
3. А потом еще разбираться с тем, как использовать НЕСКОЛЬКО сгенерированных файлов...
4. Надо бы дать для пользователей хоть простенькое описание того, как с этим работать...