UNE CLASS SITEMAP POUR AFFICHER L'ARBORESCENCE DE VOTRE SITE.

Commentaire de neigedhiver le 11/07/2008 17:22:02

Salut,

J'ai survolé rapidement (je le confesse, je n'ai pas regardé en détails) et deux petites choses me gênent...

Tout d'abord le fait que ta classe génère du code HTML. Certes, tu as des méthodes qui permettent de personnaliser un peu l'affichage en définissant des CSS.
J'ai tendance à penser, lors de mes développements, MVC : Modèle-Vue-Contrôleur. Une classe, selon moi, fait partie du Modèle. Le HTML et les CSS, ça fait partie de la vue. Je ne dis pas que ta classe est mauvaise, juste que ça me "semble" aller à l'encontre du modèle MVC (mais il se peut que je me trompe, je ne fais qu'émettre mon avis).
J'aurais plutôt vu un template pour le sitemap, avec des variables (ou équivalent) que le contrôleur se chargerait de remplacer par le contenu de la sitemap.
Ca permet une plus grande facilité d'intégration dans un site préexistant, puisque pour modifier l'affichage, on modifie la Vue, et pas le contrôleur comme c'est le cas dans ta source, grâce aux setters.
Ou alors, la classe pourrait générer du xhtml très basique (des simples listes non ordonnées), et le css étant à la charge de l'intégrateur.

Deuxièmement, la manière dont tu gères les exceptions... Ca me parait un peu inutile de mettre des exceptions si c'est pour les gérer au sein même de la classe, et même, dans la méthode qui lance l'exception.
L'inconvéinent que ça pose, c'est que l'utilisateur qui intègre ta classe dans son site ne peut absolument pas gérer les exceptions comme il l'entend, alors que c'est justement une fonctionnalité que les Exceptions PHP5 apportent.
A mon sens, les blocs try{} catch{} devraient se trouver dans le contrôleur, pas dans le modèle. Donc, pas dans la classe, mais plutôt dans les scripts appelés directement (qui sont, selon moi, le Contrôleur). A chacun d'utiliser (ou non) son gestionnaire d'exceptions. Dans l'état actuel, autant simplement afficher un message d'erreur, l'exception en elle-même n'apporte rien.

Sinon, ça m'a l'air propre, bien documenté (c'est constant chez toi) ce qui est bien. J'ai pas poussé dans le détail, notamment le fichier XML : est-ce qu'il ressemble au format des sitemaps pour Google ou est-ce que ça n'a rien à voir ?

Commentaire de nico1610 le 14/07/2008 11:27:56

Un template est un squelette de ton affichage qui te permet de générer l'affichage.

Par exemple on peut avoir un template img.tpl (exemple phplib) :
<!-- BEGIN img -->
<img src="{img.src}" alt="{img.alt}" />
<!-- END img -->

ainsi tu peux en php faire ceci :
$t = new Template(...); // je détaille pas de code ici !
...
$t->assign_block_vars("img",array("src"=>"mon_image.png"));
$t->assign_block_vars("img",array("src"=>"mon_image2.png"));
...
$t->pparse();

ce qui donnera :

<img src="mon_image.png" alt="" />
<img src="mon_image2.png" alt="" />

Ce n'est qu'un exemple bidon et simplifier des possibilité des templates, il en existe des tas, certain (comme phplib) ne gère que l'affichage, d'autre plus poussé sont capable de gérer une phase de traitement (boucle for, if ..) avant d'afficher.

Pour gérer les exeptions, c'est pas simple, faut imaginer tout ce qui pourrai faire buger ta classe, et avant que cella puisse générer une erreur, faire une vérification et stopper ton script en cas de besoin ; par exemple :
exit("ERROR : param $array need an array !");

Pour les sitemaps, Google microsoft et yahoo on fait ca : http://www.sitemaps.org/fr/ , regarde l'onglet "protocole".

Voila je pense que si tu veux de plus ample information tu peut faire appel a l'ami de tous : http://www.google.fr

Commentaire de nico1610 le 15/07/2008 17:21:42

Je n'ai donné qu'un exemple de template il en existe bien d'autre.
Bien que ce soit un site concurent, voila un très bon article sur les différents templates utilisable :
http://g-rossolini.developpez.com/comparatifs/php/templates/

Pour ma part je suis un la PhpLib car les graphistes qui me crée les pages templates et les chartes graphique ne vont pas s'ammusé a aprendre un code spécifique, avec phplib (utilisé par phpbb) tu n'asbesoin de connaitre que l'Html (ou Xhtml) et quelque regle d'usage.

Commentaire de waterw72 le 18/07/2008 09:41:17

Bête question,
pourquoi ma classe doit être une classe dans le "modèle" et non considérée comme un composant?
Un peu comme on utilise en ASP.Net avec des propriétés d'affichages, ect ...

Commentaire de waterw72 le 18/07/2008 10:21:16

Pour ma seconde question.
Apparemment, tu as considéré ma classe faisant partie du modèle dans un concept de développement MVC (Modèle Vue Contrôleur).

Moi, j'avais considéré ma classe comme un composant que l'on pourrait comparer avec un composant ASP.Net tel qu'un label.
Le composant label possède des méthodes qui permettent de traiter les données (ex : la propriété .Text) et des attributs qui permettent de modifier l'apparence (ex : .Forecolor pour la couleur du text).

rmq : Je ne pense pas que ASP.Net soi respecte la logique MVC.

Enfin, pour en venir à ma question, pourquoi ma classe ne pourrait-elle pas être considérée comme un composant avec des attributs (d'affichage) et des méthodes au même titre que le label d'ASP.Net?

___
Pour ne pas utiliser les templates, que je ne maîtrise pas encore, serait-il correct de faire une classe sitemap.class.php (sans HTML) et un controller sitemap.controler.php (avec HTML, CSS)qui aurait des propriétés tel que : image de séparation, class css, ... ect

le controler hériterait de la class?

je sais que dans ce cas, le php serait tout de même dans le html mais de cette manière :
$oSiteMap = new SiteMap_controler();
$oSiteMap->setImageUrl = "";
$oSiteMap->setCssClass = "";
$oSiteMap->setSecurity = "";
$oSIteMap->printSitemap();

Commentaire de waterw72 le 18/07/2008 12:37:08

Oui, juste une petite question la Vue, c'est l'interface donc du HTML CSS, ect mais une classe? si?