Discussion :

[keaton7]

Bonjour,

Je developpe un site Web en php5. J'ai cree tout un tas de classe, et je me soucis beaucoup de l'optimisation du code, plus au niveau de la lecture et de la logique pour le moment que au niveau du temps d'execution. En fait, c'est un bout de site CMSise qui va me resservir, donc plus je developpe des outils performants et plus j'irai vite pour la suite ...

Voila pour les presentations, passons a ma question.

Alors suivez moi :

Je veux afficher des catégories contenues dans une table SQL. Pour ce faire :
- J'initialise ma classe LesCategories avec des requete SQL d'une classe requeteBDD (les anciens etudiant IG reconnaitrons :-))
- qui elle meme ajoute des elements Categorie d'une classe categorie
- et les stocke dans une liste collection d'une classe collection
- qui elle meme est definie par le type listeDoublementChainee qui provient d'une classe liste

Ma tache la, c'est creer un select/option html avec la liste des categories.

Comme un cochon, je creerai ca dans ma classe LesCategories :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
function retournerCategories() {
  print "<select>";
  for($i=0;$i<$this->lesCategories->cardinal();$i++) {
    print "<option value='{$this->lesCategories->extraire("idCategorie")}'>$this->lesCategories->extraire("nomCategorie")</option>";
 }
  print "</select>";
}

Mais "il parait" qu'il faut eviter de mettre du code HTML dans les classes, et c'est vrai que quand je regarde ce qui se fait sur le net, on fini toujours par des "return", et quasi jamais avec du html dedans.

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
function retournerCategories() {
  return $this->lesCategories
}

Cette solution me parait un peu sterile ...

Une meilleure solution chers specialistes ?

D'avance merci.



Matt

[Rakken]

Mais "il parait" qu'il faut eviter de mettre du code HTML dans les classes,

Oui et non. Disons qu'il ne faut pas en mettre dans n'importe quelle classe.
Pour un cas comme ca, je retournerai dansl a fonction retournerCategories, un tableau "identifiant" "valeur".

Et a coté, je ferai une classe d'affichage, avec une méthode static genre

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

generateurHtml::creerSelect($tab)

Et c'est cette méthode là qui va lister les éléments de ton tableau et creer ton code html.

Comme ca, ton code html généré est a un seul endroit et tu pourra le réutiliser a chaque fois que tu veux faire un select, pour peu que tes classes qui vont chercher les données les sortent dans un format standard.

[keaton7]

La classe affichageHMTL, je n'y avait pas pense, pourquoi pas, je vais m'y pencher mais ca m'a l'air bien propre :-).

Mais j'ai une question stupide, dans la logique, puisque j'ai cree un systemem de listes chainees,ne devrais-je pas me restreinte a n'utiliser que ce systeme plutot qu'un tableau ? (non pas pour le plaisir de me restreindre, mais pour avoir un algo coherent)

[grunk]

Perso je vois pas bien l'intérêt des listes chainée dans ton affaire , c'est certes plus rapide qu'un tableaux dans un langage ou ces liste "existe" , mais en php ou les pointeurs ne sont pas utilisables je vois pas bien l'utilité face a des tableaux qu'on peut manipuler comme bon nous semble.

[keaton7]

J'ai cree ce systeme a cause de la gestion des index de tableau, c'etait ingerable de supprimer des entrees et de reafficher les contenus ... meme apres des tris ..., mais dans le cas present, le probleme ne se poserai pas.

Comme tu dis, les pointeurs n'existent pas, et il a fallu contourner ce maque par des astuces, mais le resultat est relativement rapide et tres fonctionnel puisque sur mesure.

Ce qui ne repond pas vraiment a ma derniere question, a savoir, dois-je utiliser les 2 types, listes ET tableaux ?

[keaton7]

J'ai une autre question alors que je m'attelle a realiser ta solution :

Je cree dans ma classe une fonction du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
	function retournerCategories() {
 
		$cardinal = $this->lesCategories->cardinal();
		$this->lesCategories->deplacerPointeur();
		$resultat = array();
 
		for($i=0;$i<$cardinal;$i++) {
 
			$resultat[$i][] = $this->lesCategories->extraire("idCategorie");	
			$resultat[$i][] = $this->lesCategories->extraire("nomCategorie");
 
		$this->lesCategories->deplacerPointeur('+');	
 
		}
 
		return $resultat;

Mais ca fait un peu redondance car j'ai deja une liste d'objet categories stockee dans $this->lesCategories.

Du coup en faisant ca, je transforme ni plus ni moins ma liste en tableau. C'est mieux ?


[J'ai edite mon post parce que j'avais ecris une enormite dans mon code ^^]