Tests performances et optmisation code php

[Roworll]

Une petite version objet (développement fait maison)

Code PHP :

<?php 

$db = new bdd();

$db->addTable('joueur');

$db->addField('nom');

$db->addField('xp');

$db->addWhere('joueur_name',$nom);

$db->addWhere('truc',$truc);

$db->addWhere('machin',$machin);

$db->setLimit($start,$end);

echo $db->getQuery(); // histoire de vérifier le code la la requête

$db->Execute(); 

[X-ZoD]

ca fai bocou de ligne et de caracteres non ? -_-

[Lys91]

sprintf("SELECT nom, xp FROM joueur WHERE joueur_name = '%s' AND truc='%s' AND machin='%s' LIMIT %d, %d", $name, $truc, $machin, $start, $end);

$query='';
$query.='SELECT nom, xp FROM joueur ';
$query.='WHERE joueur_name =\''.$name.'\' ';
$query.='AND truc=\''.$truc.'\' ';
$query.='AND machin=\''.$machin.'\' ';
$query.='LIMIT '.$start.', '.$end;

Apres c'est une question de point de vue sur la lisibilite du code j'en convient et pour la concatenation avec quelques regles de bonne pratique ca pose peu de soucis.

Je ne pense pas posseder la verite mais une solution alternative tout a fait viable.

(clavier qwerty encore une fois dsl)

Encore une fois que je pense que ta solution n'apporte - objectivement - que du mauvais. Si tu penses qu'elle a un quelconque intérêt, n'hésite pas à expliquer pourquoi.

Voilà ce que peut donner une requête un poil plus complexe avec sprintf() :

Code PHP :

<?php 

sprintf("

    SELECT nom, xp 

    FROM joueur 

    WHERE joueur_name = '%s' 

        AND truc='%s' 

        AND machin='%s' 

    LIMIT %d, %d", 

    $name, 

    $truc, 

    $machin, 

    $start, 

    $end

); 

[Sephi-Chan]

C'est vrai que la méthode que tu proposes (Roworll) est abusivement chargée. Je pense que l'objet c'est comme tout, il faut l'utiliser avec modération et dans le sens de nos besoins.

Aucun soucis, debattre me pose pas de soucis.
On peut aussi appelle une fonction dans la methode "concatenation"

[....]
$query.='AND machin=\''.ma_function($machin).'\' ';
[....]

Ou j'ai pas compris de quoi tu parlais ^^.

[...]

PS : quel variable intermediaire ?

Oui mais pourquoi faire une concaténation illisible alors que le sprintf() permet de faire la même chose proprement ? C'est ce que je veux comprendre, le sprinft() est clairement mieux - c'est objectif - alors pourquoi utiliser la concaténation ?

Pour reprendre le terme de Pascal, tous les caractères que tu utilises (le .=, les ', les \) ajoutent du bruit à ton script, ce qui en diminue la lisibilité. À combiner au fait que tu ne mets pas d'espaces blanc, ton script doit rapidement devenir illisible.

$query.='AND machin=\''.ma_function($machin).'\' ';

Les variables intermédiaires sont par exemples les variable de $_POST qui font l'objet d'une redéfinition, comme cela :

Code PHP :

<?php 

$message = mysql_real_escape_string($_POST['message']); 

[Lys91]

Les variable intermediaire sont les meme que pour la fonction sprintf, dans ce cas.

Avantage :
-pas d'appelle de fonction, donc plus leger
-c'est plus claire qu'un sprintf, je lis ma requete ligne par ligne je vois quelle variable j'y insere alors qu'avec sprintf je tombe sur un %s %d %f etc... je cherche la xieme variable et j'espere ne pas me planter sur le type.

Inconvenient :
-le bruit (question d'habitude)

Si on fait ca n'importe comment ca peut poser probleme c'est vrai par exemple ne pas oublier le $query=''; au depart pour eviter de concatener avec la requete precedente.

[pascal]

Le truc de Row est interessant aussi mais la pour le coup on perd un peu en lisibilite (imo).

( je continue dans le hs )
l'avantage de cette syntaxe objet se trouve dans les moteurs de recherche :
_ je fais une recherche multi critères
_ j'ajoute un nouveau champ à la db suite à une modif des besoins

=> je ne modifie pas la requête avec une expression de plus dans le WHERE, mais j'ajoute quelques lignes qui créent ce WHERE correctement ( _sous réserve que la classe db fonctionne correctement_ )

A+

Pascal

[Sephi-Chan]

Avantage :
-pas d'appelle de fonction, donc plus leger
-c'est plus claire qu'un sprintf, je lis ma requete ligne par ligne je vois quelle variable j'y insere alors qu'avec sprintf je tombe sur un %s %d %f etc... je cherche la xieme variable et j'espere ne pas me planter sur le type.

J'ai ri.


Sephi-Chan, * fatigué par ce combat, le jeune homme s'effondra, résigné à mourir, sacrifié sur l'autel de la mauvaise foi... *

[Plume]

Sephi, n'exagère pas
Lys, réfléchis.
Pourquoi concaténer abusivement ? Je ne suis pas utilisateur de la `méthode' sprintf(). Mais je trouve ta méthode tout de même abusive.

Code PHP :

<?php 

$query = '

            SELECT nom, xp 

            FROM joueur

            WHERE joueur_name = \'' . $name . '\' 

                        AND truc = \'' . $truc . '\' 

                        AND machin = \'' . $machin . '\'

            LIMIT ' . $start . ', ' . $end

; 

[Sephi-Chan]

Sephi, n'exagère pas
Lys, réfléchis.
Pourquoi concaténer abusivement ? Je ne suis pas utilisateur de la `méthode' sprintf(). Mais je trouve ta méthode tout de même abusive.

Code PHP :

<?php 

$query = '

            SELECT nom, xp 

            FROM joueur

            WHERE joueur_name = \'' . $name . '\' 

                        AND truc = \'' . $truc . '\' 

                        AND machin = \'' . $machin . '\'

            LIMIT ' . $start . ', ' . $end

; 

[Lys91]

(HS : j'insiste autant car je suis au boulot et j'ai pas grand chose à faire, donc jespere que tu n'as pas mal pris cette discussion)

Bon j'ai enfin compris que ce qui te posais problème cétait la suite de concaténation.

En conclusion la "méthode LexLxUs" remporte mon adhésion :p.

Sinon sur l'ouverture de fichier: http://www.phplibrairies.com/tutorial_op...on_fr.html

Pour ma part toujours utilisé le fopen.