FONCTION D'AFFICHAGE DE DONNÉES MYSQL

Commentaire de depression le 03/04/2008 23:35:55

Allez, je vais devoir endosser le rôle du grand méchant.

L'intention est louable, peut être qu'elle est pratique pour toi, mais cela fait une nouvelle "syntaxe" spécifique à apprendre.

En plus, je ne vois pas trop l'intérêt. Ca paraît trop fastidieux pour ce que tu fais. Tu devrais plutôt regarder du côté de PDO, ou t'essayer au Zend Framework.

Ensuite, je vais te repprocher ce que l'on repproche souvent ici (ou sur d'autres sites) :
On n'écrit plus <? mais <?php
Mets des simples quotes à la place des doubles
Faire de l'affichage dans une fonction, je ne suis pas pour du tout.
Les expressions régulières en ereg() sont obsolètes, renseigne toi plutôt autour de preg_match()


Enfin voila quoi.


J'espère ne pas t'avoir vèxé (ben si c'est le cas tant pis), mais bon, je pense que tu devrais repenser tout ça, et surtout simplifier, ou regarder d'autres solutions (Comme Zend Framework que j'ai déja cité).

Commentaire de yoman64 le 04/04/2008 06:30:14

Ah ouais et fait gaffe au injections SQL, informe toi sur mysql_real_escape_string.

Commentaire de Snakegun le 04/04/2008 12:34:15

En fait, sans vouloir abuser de votre temps et de votre générosité, j'aurais besoin de quelques éclaircissements avant de poster une nouvelle version...

Pourquoi utiliser les opérateurs de comparaison qui prennent en compte le type des variables? Je comprendrais si les données provenaient d'un formulaire mais là je vois pas bien... Idem pour mysql_real_escape_string.

Et pourquoi préférer les simples quotes, sachant que je suis de toute façon obligé d'utiliser des doubles lorsque je concatène du code HTML nécessitant des quotes avec des variables?

Et puis évidemment, nonobstant le fait que ca ne soit plus Noël, je ne déconsidérerais pas un petit exemple de simplification possible et un petit exemple de non respect des standards.

Enfin, je rougis, puis blanchis et re-rougis encore en comprenant que ce n'était pas ici qu'il me fallait poster cette source mais dans le forum... J'implore votre clémence!

Commentaire de malalam le 04/04/2008 19:59:56 administrateur CS

Hello,

@yoman => pour ta dernière remarque, tu es allé chercher en effet très, très loin...la différence est tellement infime qu'aucun applicatif web n'en profiterait vraiment. Fais le test, tu verras :-)

Concernant la source :
- une fonction qui utilise directement les GET a un problème : elle ne peut s'utiliser quand dans un contexte bien précis. A ta place, je passerais les GET en paramètre.

- ça :
#  if ($tri === $nvtri)
# {
# if($sens==='ASC')
# $sens = 'DESC';
# else
# $sens = 'ASC';
# }
# else
# {
# $sens = 'DESC';
# }
peut se condenser ainsi :
if ($tri === $nvtri && $sens = 'ASC') {
    $sens = 'DESC';
} else {
    $sens = 'ASC';
}
- ça :
#  $req.=" WHERE ";
# for($cpt=0;$cpt<$L;++$cpt)
# {
# if($cpt>0)
# $req.=" AND ";
# $temp = explode('%',$Links[$cpt]);
# $req.="`".$Tables[$temp[0]-1]."`.`".$temp[2]."`=`".$Tables[$temp[1]-1]."`.`".$temp[2]."`";
# }
peut se résumer largement avec un implode(). Ca évite une boucle. Je ne suis pas sûr que ce soit plus rapide néanmoins.
- on t'a dit d'utilier preg_* plutôt qu'ereg_*, certes. Néanmoins, on ne t'a pas dit d'utiliser strpos()quand tu n'avais pas besoin d'expression régulière. Ce qui est le cas.
- mysql_fetch_array() renvoie un tableau doublement indexé. Si tu n'utilises que les index numériques, autant passer la constante MYSQL_NUM  à cette fonction.

Dernière chose (je n'ai pas exploré le code à fond) : je ne suis pas d'accord avec Yoman sur le "framework". Utiliser PDO par exemple permettrait sans aucune modification d'utiliser ton code pour n'importe quelle bdd, ce qui n'est pas le cas actuellement (c'est mysql ou rien). D'autant plus que la connexion à la base se fait en dehors de ton code (seul moment ou le type de bdd utilisée est important avec PDO).

Commentaire de malalam le 04/04/2008 20:47:09 administrateur CS

@yoman => je n'avais pas vu Zend Framework, je m'étais arrêté à PDO ;-) Egalité! Donc je suis d'accord avec toi sur le framework.
Bref, je suis d'accord sur le zend Framework (excellent, hein...là n'est pas la question...on n'utilise pas un framework pour 3 fonctions génériques).
Une optimisation reste une optimisation : oui. C'est vrai. Si ça ne fait pas forcément du bien...ça ne peut pas faire du mal (en tous cas, pas dans ce cas: l'impact sur le code est nul). MAIS...je devais le préciser,parce que je ne voudrais pas que les futurs débutants se voient fustigés parce qu'il ont utilisé la post-incrémentation au lieu de la pré-incrémentation...si je sais que ce n'est pas ton but et que tu ne feras jamais de ce genre de remarques ton cheval de bataille, je me méfie de certains huluberlus se prenant pour des pros, et qui pourraient prendre ta remarque pour argent comptant, l'utilisant à chaque code pour descendre son auteur.
J'avais prévenu : je serai désormais plus intransigeant...donc, ne vois là aucun reproche : tu as raison :-) Et tu pourras le re-préciser au prochain code ! Je veux juste être sûr que ce ne soit pas utilisé à mauvais escient par d'autres plus tard.

Commentaire de depression le 04/04/2008 21:57:47

@malalam:

Oui pour le strpos, pour une fois, j'ai oublié de le rappeler ;-)

Quand à Zend Framework, je le cite car je travaille dessus en ce moment, et son modèle d'abstraction de bases de données est vraiment bien fait (Ca et le reste). Et bien sur, il utilise PDO (faut être logique, vu l'équipe des développeurs).
Bon, il demande d'assimiler pas mal de choses (bien plus que Jelix), mais on y gagne.

Commentaire de malalam le 04/04/2008 22:10:55 administrateur CS

@yoman => attention...quand j'ai parlé d'incidence, c'était contextuel. Dans ce cas là (celui d'une boucle for), ça n'a pas d'incidence. Et à vrai dire dans mon message, je ne parlais que d'incidence sur l'écriture du code (pas plus long, pas plus court), pas même sur ce que ces différntes notations font.
Parce que une incidence, il peut y en avoir une!! Et pas une petite...exactement comme en C si mes souvenirs sont bons, d'ailleurs, et très exactement comme en assembleur (parce que oui, j'en ai fait, j'ai même commencé par ça...enfin non, j'ai commencé par le basic de mon C64...puis j'ai enchaîne sur l'assembleur). Un exemple valant 1000 discours :
<?php
$a = 0;
$b = 0;
echo $a++;
echo ++$b;
echo $a;
echo $b;
?>
Le différence est énorme...donc les 2 écritures se justifient : tout dépend du contexte.