Discussion :

[Invité]

Bonjour,

Je voudrais optimisé mon code, quelqu'un peut m'aider et donné des suggestions.

$querychamp = mysql_query("SELECT * FROM gagnant",$db);
$num = mysql_num_rows($querychamp);
if ( $num != 0) {
while ( $gagnant = mysql_fetch_array($querychamp) ) {
$gagnantid = $gagnant['id'];
$cadeauxid = $gagnant['cadeaux'];
$membreid = $gagnant['gagnant'];
$date_mysql = $gagnant['date'];
$date = preg_replace('/(\d{4})-(\d{2})-(\d{2})/', '$3-$2-$1', $date_mysql);
$time = $gagnant['time'];
$querychamp2 = mysql_query("SELECT * from membre WHERE id_membre = $membreid",$db);
while ( $membre = mysql_fetch_array($querychamp2) ) {
$nom = $membre['nom'];
$prenom = $membre['prenom'];
echo strtoupper(substr($nom,0,1)).
strtolower(substr($nom,1)).' '.
strtoupper(substr($prenom,0,1)).'.';
}
$querychamp3 = mysql_query("SELECT * from jeuxcompteur WHERE id = $cadeauxid",$db);
while ( $cadeau = mysql_fetch_array($querychamp3) ) {
$titre = $cadeau['titre'];
echo " à gagné(e) au jeu *$titre* le $date à $time ";
}
echo "<br>";
}
} else {
echo "Il n'y a pas de cadeaux gagnés.";
}

merci beaucoup,

[titoon]

Si j'ai bien compris le schéma de ta base, tu peux utiliser comme requête :

Code :

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

1
2
3
4
5
 
SELECT * // attention aux champs "id" communs à gagnant et jeuxcompteur
FROM gagnant g, membre m, jeuxcompteur j
WHERE g.gagant = m.id_membre // jointure gagnant <=> membre
AND g.cadeau = j.jeucompteur // jointure cadeau <=> jeuxcompteur

[stephane eyskens]

Si j'ai bien compris le schéma de ta base, tu peux utiliser comme requête :

Code :

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

1
2
3
4
5
 
SELECT * // attention aux champs "id" communs à gagnant et jeuxcompteur
FROM gagnant g, membre m, jeuxcompteur j
WHERE g.gagant = m.id_membre // jointure gagnant <=> membre
AND g.cadeau = j.jeucompteur // jointure cadeau <=> jeuxcompteur

C'est indépendant du PHP et plutôt lié directement au forum "langage SQL". Ceci dit, je ne pense pas que cette requête ci-dessus soit si optimale, pour les raisons suivantes:

1. SELECT *, dans la mesure du possible, il faut toujours essayer d'éviter le "*" sauf si on a réellement besoin de rappatrier toutes les colonnes.
2. Il faut préférer les mots clés de jointure tels que INNER JOIN, LEFT JOIN, RIGHT JOIN, etc...plutôt que d'effectuer celles-ci dans la clause WHERE. Théoriquement, la clause WHERE est réservée aux critères uniquement.

Ceci dit, loin de moi de vouloir distribuer les bons points, donc, fais comme tu le sens

[Invité]

moi j'utilise toujours '*' mais quelles sont les risques d'utiliser celui-ci ?

[stephane eyskens]

moi j'utilise toujours '*' mais quelles sont les risques d'utiliser celui-ci ?

Toutes les colonnes de ta table sont rappatriées, donc plus de trafic sur le réseau, en RAM etc....Si par exemple tu as une table ayant cette structure

Table Clients

ID_Client
Nom
Prenom
Titre
DateNaissance
NumIdentite
ID_Addr
Commentaire
....

Et que tu as une portion de code qui sert à afficher uniquement les nom, prénom et titre

Tu devrais théoriquement faire

SELECT Nom,Prenom,Titre FROM Clients....

Seules les informations liées à ces colonnes te seront renvoyées par le SGBD. Tu réduiras du même coup le trafic réseau et ton traitement côté PHP sera lui aussi plus rapide car moins de données.

D'autre part, cette manière de travailler est beaucoup plus parlante dans le code car :

- Tu sais exactement ce dont tu as besoin
- Le fait de nommer explicitement les colonnes te renseigne déjà sur la structure de ta table.
- Si un jour, tu modifies ta table et que tu ajoutes un champ pour usage interne qui ne devrait jamais être affiché, en utilisant "*", tu rappatrieras à chaque fois la valeur de ce champ inutilement.

[Invité]

Si j'ai bien compris le schéma de ta base, tu peux utiliser comme requête :

Code :

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

1
2
3
4
5
 
SELECT * // attention aux champs "id" communs à gagnant et jeuxcompteur
FROM gagnant g, membre m, jeuxcompteur j
WHERE g.gagant = m.id_membre // jointure gagnant <=> membre
AND g.cadeau = j.jeucompteur // jointure cadeau <=> jeuxcompteur

j'ai essayer

$querychamp = mysql_query("SELECT *
FROM gagnant g, membre m, jeuxcompteur j
WHERE g.gagnant = m.id_membre
AND g.cadeaux = j.id",$db);
while ( $data = mysql_fetch_array($querychamp) ) {
print_r($data);
$gagnantid = $gagnant['id'];
$cadeauxid = $gagnant['cadeaux'];
$membreid = $gagnant['gagnant'];
$date_mysql = $gagnant['date'];
$date = preg_replace('/(\d{4})-(\d{2})-(\d{2})/', '$3-$2-$1', $date_mysql);
$time = $gagnant['time'];
$nom = $membre['nom'];
$prenom = $membre['prenom'];
$titre = $cadeau['titre'];
echo strtoupper(substr($nom,0,1)).
strtolower(substr($nom,1)).' '.
strtoupper(substr($prenom,0,1)).'.';
echo " à gagné(e) au jeu *$titre* le $date à $time <br>";
}

vous pouvez me dire comment l'affichez car la ça ne marche pas

[Invité]

$querychamp = mysql_query("SELECT date, time, nom, prenom, titre
FROM gagnant g, membre m, jeuxcompteur j
WHERE g.gagnant = m.id_membre
AND g.cadeaux = j.id",$db);
while ( $data = mysql_fetch_array($querychamp) ) {
$date_mysql = $data['date'];
$date = preg_replace('/(\d{4})-(\d{2})-(\d{2})/', '$3-$2-$1', $date_mysql);
$time = $data['time'];
$nom = $data['nom'];
$prenom = $data['prenom'];
$titre = $data['titre'];
echo strtoupper(substr($nom,0,1)).
strtolower(substr($nom,1)).' '.
strtoupper(substr($prenom,0,1)).'.';
echo " à gagné(e) au jeu *$titre* le $date à $time <br>";
}

c'est bon ça marche, vous pouvez me dire plus sur l'utilisation du INNER JOIN, LEFT JOIN, RIGHT JOIN, etc.. et montré une démonstration

merci beaucoup

PS: par rapport à ce que j'avais au début à maintenant j'ai perdu beaucoup de ligne en gardant le meme résultat

[titoon]

1. SELECT *, dans la mesure du possible, il faut toujours essayer d'éviter le "*" sauf si on a réellement besoin de rappatrier toutes les colonnes.
2. Il faut préférer les mots clés de jointure tels que INNER JOIN, LEFT JOIN, RIGHT JOIN, etc...plutôt que d'effectuer celles-ci dans la clause WHERE. Théoriquement, la clause WHERE est réservée aux critères uniquement.

OK pour le select *, perso je l'utilise jamais.

Par contre, pour le second point... t'as une doc là dessus ???

INNER JOIN et , (virgule) sont sémantiquement équivalents. Les deux opèrent une jointure totale sur les tables utilisées. Normalement, vous spécifiez les conditions de jointure dans la clause WHERE.

c'est bon ça marche, vous pouvez me dire plus sur l'utilisation du INNER JOIN, LEFT JOIN, RIGHT JOIN, etc.. et montré une démonstration

Tu trouveras ton bonheur dans la doc de mysql

[stephane eyskens]

OK pour le select *, perso je l'utilise jamais.
Par contre, pour le second point... t'as une doc là dessus ???

Tu peux par exemple aller voir sur sql.developpez.com

[gisele]

Salut,

je rajouterais que de nommer uniquement les champs dont on a besoin permet d'utilser mysql_fetch_row() (le plus économique de la famille des fetch) plus facilement car on n'a pas forcement en tete la position ordinale des champs de sa tables.

Quand au JOIN déja, il est issu de la norme SQL99 qui était venu remplacer la norme SQL92.
Il est donc plus au gout du jour, plus clair entant qu'il confère une syntaxe explicite et dédiée aux jointures en les séparant du reste de la clause WHERE.
En plus Il permet souvent l'economie des alias qui la surchargent.
Et surtout plus puissant que les jointures en clause where.
JE ne vais pas me répéter je vous renvoie a ce topic ou je donne quelques exemples simples.
http://www.developpez.net/forums/sho...d.php?t=191918 (message #8)

Sinon ben voila la version JOIN du présent problème (je part du principe qu'aucun nom de champ sité dans la requete n'est commun à 2 tables =>aucun d'alias)
J'ai changé l'ordre d'apparition des tables, afin de pouvoir enfiler les tables comme des perles dans ma requete (on commence toujours par une table a l'extrémité pour pouvoir enchainer les JOIN):

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
 
$querychamp = mysql_query("SELECT date, time, nom, prenom, titre 
FROM membre JOIN gagnant on(id_membre=gagnant)
JOIN jeuxcompteur ON (cadeaux=id)",$db);
while ( $data = mysql_fetch_array($querychamp) ) {
$date_mysql = $data['date'];
$date = preg_replace('/(\d{4})-(\d{2})-(\d{2})/', '$3-$2-$1', $date_mysql);
$time = $data['time'];
$nom = $data['nom'];
$prenom = $data['prenom'];
$titre = $data['titre'];
echo strtoupper(substr($nom,0,1)).
strtolower(substr($nom,1)).' '.
strtoupper(substr($prenom,0,1)).'.';
echo " à gagné(e) au jeu *$titre* le $date à $time <br>";
}

[titoon]

Erf, désolé, je suis en mysql 3.32.58, et cette version semble ne pas connaitre le JOIN seul... Je ne sais pas comment il se comporte (équivalent de LEFT JOIN ?)

Code :

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

1
2
3
4
5
6
 
mysql> SELECT date, time, nom, prenom, titre
    -> FROM membre JOIN gagnant ON (id_membre = gagnant)
    -> JOIN jeuxcompteur ON (cadeaux = id);
ERROR 1064: You have an error in your SQL syntax near 'ON (id_membre = gagnant)
JOIN jeuxcompteur ON (cadeaux = id)' at line 2

[gisele]

Salut,

effectivement essaye ca:

Code :

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

1
2
3
4
5
 
SELECT date, time, nom, prenom, titre 
FROM membre LEFT JOIN gagnant on(id_membre=gagnant)
AND gagnant IS NOT NULL
LEFT JOIN jeuxcompteur ON (cadeaux=id)

la condition is not null, permet a la requete de ne pas remonter également des lignes d'utilisateur qui n'ont pas gagné, (les champs correspondant a la table gagnant seraient comblés a null).

tu comprendras visuellement la finesse du LEFT JOIN en essayant d'abord sans le "AND gagnant IS NOT NULL " et ensuite avec.
la différence entre les retours te permettra de comprendre l'interet de jouer avec les LEFT JOIN et RIGHT JOIN.