Discussion :

[truemistress]

Bonsoir à tous,

Voici mon soucis : j'ai une table 'clans', qui contient les clans participant à un tournoi, avec un champ 'nb_pts', qui équivaut à la somme des points de chaque membre qui compose le clan.
Les points de chaque membre sont stockés dans la table 'clans_pts', avec un champ 'clanid' qui contient l'id du clan auquel appartient le membre. (Bien entendu, il y a plusieurs membres par clan, et chaque membre peut avoir plusieurs scores.)
Je veux donc, pour chaque clan, enregistrer dans la champ nb_pts la somme des points des membres de ce clan, qui sont dans la table clans_pts.

Voici donc la dernière version de mon code (car j'ai fait des tonnes de tests différents) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
$res_scores_clan = $db->dbQuery("SELECT SUM(nb_pts) as total_pts, clanid 
                                 FROM clans_pts 
                                 GROUP BY clanid");
$k = 0;
while ($dat_scores_clan = $db->dbFetch($res_scores_clan))
{
    $scores[$k] = $dat_scores_clan;
    $update_pts_clan = $db->dbQuery("UPDATE clans 
                                     SET nb_pts = '".$scores[$k]['total_pts']."' 
                                     WHERE id = '".$scores[$k]['clanid']."'");
     $k++;
}

Malheureusement, sans que je comprenne d'ou ça vient, ben ça ne fonctionne pas... La requete me renvoie bien 'true' autant de fois qu'il y a de résultat, mais le champ nb_pts de la table clans n'est pas mis à jour....

Quelqu'un pourrait-il m'éclairer..? Je n'ai peut etre pas choisi la bonne méthode pour obtenir le réusltat que je souahite....

Merci d'avance

[Antoun]

Quelqu'un pourrait-il m'éclairer..? Je n'ai peut etre pas choisi la bonne méthode pour obtenir le résultat que je souahite....

non, et sur plusieurs points :

- à partir du moment où ta première requête te permet de reconstituer le total des points de chaque clan, tu n'as pas besoin d'enregistrer cette info qq part ; ce serait introduire une redondance, et risquer d'interroger une info qui n'est plus à jour

- quand bien même tu devrais le faire, il est inutilement lent et compliqué de le faire à travers une boucle ; une seule requête suffit :

Code sql :

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

1
2
3
4
5
6
UPDATE clans c 
SET c.nb_pts = (
  SELECT SUM(p.nb_pts) 
  FROM clans_pts p 
  WHERE p.clanid = c.clanid
)

- si ton dbQuery renvoie à mysql_query, le résultat "vrai" veut juste dire qu'il n'y a pas eu d'erreur, pas qu'une modification a vraiment été effectuée (ce que tu verrais avec mysql_affected_rows)

[truemistress]

merci Antoun. effectivement, la solution que tu donnes est la 1ère à laquelle j'ai pensé, mais après maints tests, je n'arrivais pas à ce que ça fonctionne... Du coup, j'ai testé pleins d'autres solutions, et finallement j'me suis éloignée de la solution correcte....

Cependant, j'ai testé la syntaxe que tu m'indiques, mais ça ne fonctionne toujours pas... Je n'ai pas d'erreur de requete, mais mon champ 'clans.nb_pts' n'est pas mis à jour (mysql_affected_rows renvoie 0) Pourtant j'ai bien vérifié scrupuleusement les noms des champs dans ma BDD, et tout semble ok.... donc là, je bloque...

Vous auriez une idée sur la raison pour laquelle ça ne fonctionne pas ?

[Antoun]

Cependant, j'ai testé la syntaxe que tu m'indiques, mais ça ne fonctionne toujours pas... Je n'ai pas d'erreur de requete, mais mon champ 'clans.nb_pts' n'est pas mis à jour (mysql_affected_rows renvoie 0) Pourtant j'ai bien vérifié scrupuleusement les noms des champs dans ma BDD, et tout semble ok.... donc là, je bloque...

Vous auriez une idée sur la raison pour laquelle ça ne fonctionne pas ?

Oui : ça a fonctionné une fois, depuis les chiffres sont corrects et il n'y a donc plus de mise à jour.

[truemistress]

J'ai trouvé la raison pour laquelle ça ne fonctionnait pas : et c'est entièrement ma faute.... En effeet, cela vient de la base de données sur laquelle je travaille (qui a été créée par qqn d'autre car j'intègre un projet déjà existant). Le mec a mal mis en place la table et les propriétés des champs, et c'est la raison pour laquelle les nouvelles valeurs ne s'enregistraient pas.... (grrrrrrrrrrr je vais le tuer..... et me tuer après de ne pas avoir diagnostiqué ça + tot....! )

J'ai donc corrigé ça, et maintenant la requete que tu m'as indiquée marche effectivement parfaitement.

Donc encore merci, et désolée pour la perte de temps que j'ai entrainée, en mettant autant de temps à trouver l'erreur...
SHAME ON ME ! BIG UP ON YOU !

[truemistress]

par contre, j'aurais juste une petite question de syntaxe, avec ou sans le AS :
"UPDATE clans c" est-il équivalent à "UPDATE clans AS c",
ou
"FROM clans_pts p" est-il équivalent à "FROM clans_pts AS p" ?

Ces 2 syntaxes sont-elles correctes et totalement équivalentes, dans tous les cas ? Ou bien l'une d'entre elles est-elle + conseillée que l'autre ?

Merci

[Antoun]

AS est conseillé pour de pures raisons de lisibilité. Techniquement, c'est exactement la même chose.

[truemistress]

merci Antoun !

[truemistress]

A l'usage, je me suis apperçue qu'il y avait un soucis avec le calcul total, lorsque la table concernée contient des champs avec valeur=0.
Je m'explique : chaque clan a des membres. Le nombre de points de chaque clan correspond donc au total des points de chacun de ses membres.
Pour connaitre le nombre de points des clans, je dis donc faire une requete qui calcule ce total, dans la table clans_pts, qui contient une entrée par membre, et qui précise de quel clan le membre fait partie. Voici donc la requete, toute simple :

Code :

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

1
2
3
4
SELECT SUM(nb_pts) AS total 
FROM clans_pts 
GROUP BY clanid 
ORDER BY total DESC

Tout seul comme ça, ça marche bien. Mais le soucis c'est dès que j'y applique mes conditions :

Code :

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

1
2
3
4
5
6
7
8
SELECT SUM(p.nb_pts) AS total 
FROM (clans_pts AS p, clans_att AS a, clans AS c) 
WHERE p.clanid = c.id 
    AND c.valid = '1' 
    AND c.id = a.clanid 
    AND a.status IN ('0','1') 
GROUP BY p.clanid 
ORDER BY total

Et là, je m'apperçois que lorsque j'ai des membres qui ont 0 points, cela crée une erreur dans le total. Il semblerait qu'au lieu d'ajouter 0 dans l'addition, le total des points est ajouté à lui-même chaque fois qu'il rencontre un score = à 0.
Ex : le clan 01 a 3 membres : membre01 qui a 300 pts, membre02 qui a 0 points, et membre03 qui a 0 points. Avec ma requete actuelle, il me renverra comme total des points du clan01 : 900 (300X3membres....).
J'ai donc essayé pleins de tests différents, pour lui mettre comme condition de ne prendre en compte que les scores != 0 comme par exemple ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT SUM(p.nb_pts) AS total 
FROM (clans_pts AS p, clans_att AS a, clans AS c) 
WHERE p.clanid = c.id 
    AND c.valid = '1' 
    AND c.id = a.clanid 
    AND a.status IN ('0','1') 
    AND p.nb_pts != '0'
GROUP BY p.clanid 
ORDER BY total

Mais rien ne semble fonctionner, j'ai toujours le meme résultat incorrect... (alors que sans les conditions, je n'avais pas ce problème)

[truemistress]

C'est bon, j'ai trouvé la cause de mon erreur, c'est simplement qu'il manquait une condition nécessaire à la bonne compréhension d'une autre condition. En la rajoutant, j'obtiens bien le résultat correct.
Ce qui donne donc :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT SUM(p.nb_pts) AS total 
FROM (clans_pts AS p, clans_att AS a, clans AS c) 
WHERE p.clanid = c.id 
    AND c.valid = '1' 
    AND c.id = a.clanid 
    AND p.userid = a.userid
    AND a.STATUS IN ('0','1') 
GROUP BY p.clanid 
ORDER BY total