RSS - LES 5 DERNIERS POST DE PHPBB

Commentaire de Anthomicro le 27/12/2004 12:56:05

Salut ;-)

remplaces ton code par ça :

<?php

// **********************************************
// Variables à Définir
// **********************************************
$nombre_de_post_a_afficher=5;

// ********************************
// Connexion à la base Mysql
// ********************************
mysql_connect('server','login','password');
mysql_select_db('nom_de_la_base');

// **********************************************
// Récupération du nombre de post totale
// **********************************************
$id=mysql_query('select topic_id from z_forum_topics where forum_id!=11');
$nombre_de_post_total=mysql_num_rows($id);

// **********************************************
// Affichage de l'entête de la page Xml
// **********************************************
header('Content-Type: text/xml');
echo "<?xml version=\"1.0\" encoding=\"ISO-8859-1\"?>\n\n";
echo "<!DOCTYPE rss PUBLIC \"-//Netscape Communications//DTD RSS 0.91//EN\"\n";
echo " \"http://my.netscape.com/publish/formats/rss-0.91.dtd\">\n\n";
echo "<rss version=\"0.91\">\n\n";

// **********************************************
// Affichage de la catégorie
// **********************************************
echo "<channel>\n";
echo "<title>Le Forum du monde TCPIP avec FrameIP.com</title>\n";
echo "<link>http://www.frameip.com/</link>\n";
echo "<description>FrameIP, un forum pour les spécialistes IP - Les 5 derniers posts</description>\n";
echo "<language>fr</language>\n\n";
echo "<image>\n";
echo "<title>FrameIP</title>\n";
echo "<url>http://www.frameip.com/logos/logo_30.jpg</url>\n";
echo "<link>http://www.frameip.com</link>\n";
echo "<description>FrameIP, un forum pour les spécialistes IP. - TcpIP - Voip - Toip - Vpn - Ipv6</description>\n";
echo "<width>142</width>\n";
echo "<height>56</height>\n";
echo "</image>\n\n";

// **********************************************
// Récupération des Posts
// **********************************************
$id=mysql_query("select topic_title,topic_id from z_forum_topics where forum_id!=11 ORDER BY topic_id desc");

// **********************************************
// Affichage des Posts
// **********************************************
for ($i=1;$i<=$nombre_de_post_a_afficher;$i++)
    {
    $tempo=mysql_fetch_array($id);
    echo "<item>\n";
    echo "<title>$tempo[0]</title>\n";
    echo "<link>http://www.frameip.com/forum/viewtopic.php?t=$tempo[1]</link>\n";
    echo "</item>\n\n";
    }

// **********************************************
// Affichage de l'entête de la page Xml
// **********************************************
echo "</channel>\n\n";
echo "</rss>";

?>

Tu peux également optimiser les echos

a +

Commentaire de coucou747 le 27/12/2004 13:36:42 administrateur CS

short tags...
au moins :
<? ceci n'est pas portable, alors que <?php ceci l'est...

Commentaire de fg85 le 27/12/2004 15:59:16

Mouarf c'était pas le peine de faire une source pour ça ;-)
Malgré quelque fonction intérrésante que je m'enpresse de modifier/utiliser .
---------------------------
http://fg.logiciel.free.fr

Commentaire de Anthomicro le 02/01/2005 13:02:34

Tiens ça me fait penser que j'ai oublié de mentionner ça :

for ($i=1;$i<=$nombre_de_post_a_afficher;$i++)
    {
    $tempo=mysql_fetch_array($id);
    echo "<item>\n";
    echo "<title>$tempo[0]</title>\n";
    echo "<link>http://www.frameip.com/forum/viewtopic.php?t=$tempo[1]</link>\n";
    echo "</item>\n\n";
    }

à remplacer par ça :

for ($i=1;$i<=$nombre_de_post_a_afficher;$i++)
    {
    $tempo=mysql_fetch_row($id);
    echo "<item>\n";
    echo "<title>$tempo[0]</title>\n";
    echo "<link>http://www.frameip.com/forum/viewtopic.php?t=$tempo[1]</link>\n";
    echo "</item>\n\n";
    }

car dans ce cas le fetch_array ne sert à rien si ce n'est qu'à ralentir le script.

a +

Commentaire de Anthomicro le 02/01/2005 13:33:22

utilisé par beaucoup de sites, oui

vraiment bien au niveau optimisation, non.

Commentaire de Anthomicro le 02/01/2005 14:50:40

je rajoute :

$id=mysql_query("select topic_title,topic_id from z_forum_topics where forum_id!=11 ORDER BY topic_id desc");

à remplacer par

$id=mysql_query("SELECT topic_title,topic_id FROM z_forum_topics WHERE forum_id!=11 ORDER BY topic_id DESC LIMIT ".$nombre_de_post_a_afficher);

Ensuite
for ($i=1;$i<=$nombre_de_post_a_afficher;$i++)
    {
    $tempo=mysql_fetch_row($id);
    echo "<item>\n";
    echo "<title>$tempo[0]</title>\n";
    echo "<link>http://www.frameip.com/forum/viewtopic.php?t=$tempo[1]</link>\n";
    echo "</item>\n\n";
    }

à remplacer par

while($tempo=mysql_fetch_row($id))
{
    echo "<item>\n";
    echo "<title>".$tempo[0]."</title>\n";
    echo "<link>http://www.frameip.com/forum/viewtopic.php?t=".$tempo[1]."</link>\n";
    echo "</item>\n\n";
}

et encore...

a +

Commentaire de _sebf le 05/01/2005 21:42:21

Voilà j'ai tenu compte de vos remarques pour mettre à jour le code.

Merci à vous

_SebF

Commentaire de mickael9 le 22/08/2005 00:35:48

A la racine de ton forum.

Commentaire de spyro666 le 09/04/2006 12:35:47

Y a-t-il un quelconque danger si on ne fait pas mysql_close() apres un mysql_query() ?

Commentaire de coucou747 le 09/04/2006 13:15:42 administrateur CS

le mysql_close permet de libérer plus tot de la mémoire...

Bon, sinon, phpbb c'est pas super optimisé...