PETIT LECTEUR DE RSS SANS PRÉTENTION ...

gwadarepresent Messages postés 4 Date d'inscription samedi 16 septembre 2006 Statut Membre Dernière intervention 28 octobre 2007 - 5 juin 2007 à 12:29
opossum_farceur Messages postés 147 Date d'inscription lundi 16 août 2004 Statut Membre Dernière intervention 14 novembre 2009 - 7 juin 2007 à 19:19
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/42651-petit-lecteur-de-rss-sans-pretention

opossum_farceur Messages postés 147 Date d'inscription lundi 16 août 2004 Statut Membre Dernière intervention 14 novembre 2009
7 juin 2007 à 19:19
Bravo pour ce code très clean, très clair : un modèle de POO.
- Un petit bémol, cependant, certain fichiers ne sont pas correctement traités, exemple :
http://www.dicodunet.com/definitions.xml
- Un deuxième petit bémol, c'est l'aspect inachevé du script, une petite démo n'aurait pas fait de mal, et tu aurais eu davantage de retours, souvent utiles pour faire évoluer un script.
- Une question : l'emploi de la récursivité est-il vraiment nécessaire, pour parcourir un arbre dont on connait à l'avance la structure?
- A l'intention des novices en matière de DOM, un tuto utile à potasser :
Gestion du XML en PHP 5 : utiliser DOM (http://eusebius.developpez.com/php5dom/)

Amicalement.
LocalStone Messages postés 514 Date d'inscription mercredi 19 mars 2003 Statut Membre Dernière intervention 1 mars 2009
5 juin 2007 à 16:19
Je pense que tu t'es trompé de source, là ...
gwadarepresent Messages postés 4 Date d'inscription samedi 16 septembre 2006 Statut Membre Dernière intervention 28 octobre 2007
5 juin 2007 à 12:29
Slt, pas mal le générateur de CSS,
Mais il y a un pti beug, en fait, tu propose de renommer le préfixe des tables de ton éditeur RSS, mais ton script PHP ne les renommes pas, et les crè avec le préfixe par défaut "rsseditor_" en revanche, il est renommé dans ton script, donc biensure, le message d'erreur est retourné :
Table 'base.RSS_config' doesn't exist
Rejoignez-nous