Bonjour
 
J'ai pris l'habitude coder comme cela pour séparer le php du html

 
Ca me convient mais j'ai l'impression que c'est pas très optimisé surtout au niveau de la mémoire
quand la table contient beaucoup d'entrées
 
Avez un conseil à me donner hormis le fait de virer le * de ma requête ?
 
Merci d'avance

Je vois au moins trois points négatifs dans cette façon de coder :
 
1] Le count doit être en dehors de la boucle :
 
$max = count(...);
for ($i = 0; $i < $max; ++$i) {
...
}
 
2] N'oublies pas de faire un mysql_free_result($result) ou un truc du genre, sinon c'est comme si tu avais 2 fois ton tableau en mémoire ($result et $actif). Je t'invite par ailleurs à utiliser PDO, qui te permetra d'avoir des requêtes paramétrées et donc plus rapides en seconde exécution.
 
3] Faire N <?php instancie N fois le parseur PHP. Il vaut mieux faire un seul <?php, ecrire et concaténer ton html dans une chaine, et faire un echo à la fin.

Avis personnel: je ne vois pas vraiment ce qui pourrait "optimiser" dans le sens revoir l'algorithme derrière, c'est une simple requète avec construction d'une table HTML. Par contre, il me semble que depuis un moment mysql_connect est déprécié, on utilise PDO ou mysqli.

Edit ::

Effectivement vu de la facon de Cyberdenix, le count en amont du for permet de ne pas avoir à réévaluer à chaque passage de boucle. Par contre pour le point n°3 je découvre, mais j'aimerai bien avoir plus d'infos sur l'impact en perfs car j'ai toujours fait ainsi dans mes vues

Merci beaucoup pour vos conseils !
 
Comme je charge tout dans un tableau, je sature la mémoire
Pourtant le moteur de template smarty fait bien comme ça non?
 
Je vais faire des tests avec vos conseils

On me demande d'optimiser... j'optimise !
 
Après c'est un deal entre la performance pure (mesurable), et la lisibilité personnelle que l'on attribue au code (pas mesurable). On penche dans un sens ou dans l'autre.
 
Par exemple, j'adore faire des foreach.
Si je m'écoutais, je les transformerais tous en while :B
 

Ok merci pour le petit éclaircissement

Bonjour,
 
Une optimisation pourrait être le fait que finalement tu parses deux fois tes données et les stocks dans un tableau...  
Ce qui effectivement mange ta memory PHP.
 
Si tu avais un objet qui implémente la SPL (Iterator et Countable) et qui utilise les api mysql pour fetcher le resultat de mysql sur la methode "next()" de Iterator et qui renverrai un "num_rows" sur la methode "count()" alors tu n'aurais plus a utiliser la memory PHP ni a fetcher deux fois ton jeu de resultat.
Tu enverrai juste cet objet à ta vue et basta. Encore mieux : la methode "current()" renverrai un objet disposant lui même de méthode propre à un model donné.
 
A mon sens c'est là qu'il y a optimisation car ton appli fonctionnera quelque soit le nombre de ligne de ta table alors qu'actuellement ton memory limit arrivera un jour au max si les lignes de la requetes deviennent plus nombreuses ...