<?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(); 

Sephi-Chan a écrit :

Lys91 a écrit :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

); 

Elle combine les avantages (lisibilité, possibilité de test internes, etc.) de toutes les techniques sans défauts. Que demande le peuple ?

Je note que tu parlais des tests internes (avec l'opérateur ternaire, je suppose), c'est clairement un ennemi de la lisibilité ! On peut les faire avec sprintf(), mais c'est à mon avis - dans la plupart des cas une bêtise : si on veut de la clarté (en utilisant sprintf()), ce n'est pas pour la ruiner à coup d'opérateurs ternaires.

L'utilisation de sprintf() peut paraître plus coûteuse en ressources, mais je pense que c'est en fait économique puisqu'il évite de recourir à des variables intermédiaires puisque l'on peut utiliser des fonctions directement dans les arguments.

Je précise quand même que je n'ai rien contre toi, je poste dans le seul but de fournir des clés pour un code plus propre et lisible aux lecteurs.



Sephi-Chan, qui songe à se faire une userbar "Sprintf() user"...


@ X-ZoD > Je suggère de mettre les liens sous forme de liste, de manière à avoir quelque chose de clair. Ces URLs mises les unes à la suite des autre c'est... assez rebutant.

Lys91 a écrit :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 ?

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

<?php 

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

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

Lys91 a écrit :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.

<?php 

$query = '

            SELECT nom, xp 

            FROM joueur

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

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

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

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

; 

LexLxUs a écrit :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

; 

~L~

Guide de style pour bien coder a écrit :Merci de ne pas utiliser la virgule et d'éviter de mettre un espace après le point. Cela ne fait qu'augmenter la taille de la ligne et ce n'est vraiment pas indispensable pour la bonne compréhension du code.