Lier un comptage à une requête

**yazerty** · 18/01/2006, 09h01

Bonjour,

J'ai la requête suivante :

Code :

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

$sql = "SELECT RR.id, RR.pseudo_id, RR.pseudo, RR.email, RR.email_publication, RR.titre, RR.reve, RR.date, RRVR.pseudo as pseudo_inscrit FROM reves_reves RR, reves_reveurs RRVR WHERE RR.activity='on' AND RRVR.id = RR.pseudo_id ORDER BY RR.id DESC LIMIT $reves_bckid,$nb_reves_aff";

Celle-ci marche bien et fait ce que je veux (elle affiche les rêves sur une page grâce à une boucle while et sélectionne le pseudo du rêveur dans une table reves_reveur en fonction de son id; si il n'est pas membre inscrit son id est égal à 0 mais il y a un enregistrement "0" dans la table reves_reveurs, même si je ne me sert pas de celui-ci car il n'a pas de données intéressantes du coup).

Chaque rêve peut être commenté. J'affiche le nombre de commentaire de chaque rêve en dessous de celui-ci. Pour le moment ça se limite à intégrer ce bout de code dans la boucle d'affichage des rêves :

Code :

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

1
2
3
4
5
// Nombre de commentaires
$id_reve = $db_data_reves['id'];
$nb_comments_on_req = mysql_query("SELECT COUNT(*) AS nb_comments FROM reves_comments WHERE id_reve='$id_reve' AND activity='on'");
$nb_comments_on_data = mysql_fetch_assoc($nb_comments_on_req);
$nb_comments_on = $nb_comments_on_data['nb_comments'];

Mais je ne pense pas qu'intégrer une requête de ce type dans une boucle est très optimal... Je cherche donc une solution plus efficace. J'ai donc essayé d'intégré la requête précédente dans ma première requête, effectuée avant la boucle, puis affichée dans la boucle. La voici modifiée en :

Code :

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

$sql = "SELECT RR.id, RR.pseudo_id, RR.pseudo, RR.email, RR.email_publication, RR.titre, RR.reve, RR.date, RRVR.pseudo as pseudo_inscrit, RC.COUNT(*) FROM reves_reves RR, reves_reveurs RRVR, reves_comments RC WHERE RR.activity='on' AND RRVR.id = RR.pseudo_id AND RC.id_reve=RR.id ORDER BY RR.id DESC LIMIT $reves_bckid,$nb_reves_aff";

Mais ça me génère l'erreur SQL suivante :

Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource

De quoi cela provient-il (j'ai un doute sur mon RC.COUNT(*) mais ne sais pas comment tenter la chose autrement). Pourriez-vous m'aidez svp :- ?

[Sujet déplacé par Maximilian]
[ancien forum : MySQL]

**yazerty** · 21/01/2006, 16h30

Il n'y a rien de plus optimisé :o ?

**vg33** · 21/01/2006, 16h34

Ca veut dire que tu as une erreur de syntaxe dans ta requête. Rajoute ceci à ta requête pour connaître l'erreur :

Code :

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

$resultat=mysql_query($sql) or die(mysql_error());

et poste le message si tu ne comprends pas l'erreur.

**yazerty** · 21/01/2006, 21h37

Code :

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

1
2
3
$sql = "SELECT RR.id, RR.pseudo_id, RR.pseudo, RR.email, RR.email_publication, RR.titre, RR.reve, RR.date, RRVR.pseudo as pseudo_inscrit, RC.COUNT(*) FROM reves_reves RR, reves_reveurs RRVR, reves_comments RC WHERE RR.activity='on' AND RRVR.id = RR.pseudo_id AND RC.id_reve=RR.id ORDER BY RR.id DESC LIMIT $reves_bckid,$nb_reves_aff";
 
$req = mysql_query($sql) or die(mysql_error());

Donne cette erreur :

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '(*) FROM reves_reves RR, reves_reveurs RRVR, reves_comments RC

Mais je n'y comprends pas plus quelque chose :-/

**vg33** · 21/01/2006, 22h42

C'est un problème avec ton RC.COUNT(*).
Et si tu essayais avec un

Code :

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

COUNT(nom_d'une_des_colonnes_de_RC)

plutôt que

Code :

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

RC.COUNT(*)

?

**yazerty** · 22/01/2006, 17h06

si je fais RC.COUNT(id) ça me sort la même erreur :

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '(id) FROM reves_reves RR, reves_reveurs RRVR, reves_comments RC

Si je fais COUNT(RC.id) ça me donne :

Mixing of GROUP columns (MIN(),MAX(),COUNT()...) with no GROUP columns is illegal if there is no GROUP BY clause

**ePoX** · 22/01/2006, 17h16

C'est normal. Lorsque tu utilises une fonction d'agrégat mélangé à des lignes communes dans un select tu DOIS faire un group by dans la requete.

Donc essaies cela :

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
17
18
19
20
SELECT 
        RR.id, RR.pseudo_id, RR.pseudo, 
        RR.email, RR.email_publication, RR.titre, 
        RR.reve, RR.date, 
        RRVR.pseudo as pseudo_inscrit, RC.COUNT(*) 
FROM 
        reves_reves RR, 
        reves_reveurs RRVR, 
        reves_comments RC 
WHERE 
        RR.activity='on' 
        AND 
        RRVR.id = RR.pseudo_id 
        AND RC.id_reve=RR.id 
ORDER BY 
         RR.id DESC 
GROUP BY
         RC.id
LIMIT 
         $reves_bckid,$nb_reves_aff";

par contre je ne sais pas quel colonne est unique, soit RC.id soit RR.id, c'est à toi de voir.

Bye

PS : L'indentation n'est pas interdite dans les requetes SQL.

**yazerty** · 23/01/2006, 22h14

Merci de ton aide ePoX. Malheureusement cette requête me donne le message d'erreur suivant :

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'GROUP BY RC.id LIMIT 0,10' at line 17

J'ai essayé :
GROUP BY RR.id
GROUP BY RC.id_reve
GROUP BY RR.id,RC.id_reve

Mais ça ne marche toujours pas

.

RR.id et RC.id sont tous les 2 indépendants et uniques :
RR.id correspond à l'id du rêve enregistré dans la table reves_reves
RC.id correcpond à l'id du commentaire du reves enregistré dans la table reves_comments. D'où le WHERE (...) RC.id_reve=RR.id
RC.id_reve correpond en effet à l'id du rêve (enregistré dans la table reves_reves)

ps : qu'est-ce que l'indentation, en particulier dans les requête SQL :- ?

**ePoX** · 23/01/2006, 22h30

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
17
18
19
20
SELECT 
        RR.id, RR.pseudo_id, RR.pseudo, 
        RR.email, RR.email_publication, RR.titre, 
        RR.reve, RR.date, 
        RRVR.pseudo as pseudo_inscrit, RC.COUNT(*) 
FROM 
        reves_reves RR, 
        reves_reveurs RRVR, 
        reves_comments RC 
WHERE 
        RR.activity='on' 
        AND 
        RRVR.id = RR.pseudo_id 
        AND RC.id_reve=RR.id 
ORDER BY 
         RR.id DESC 
GROUP BY 
         RC.id 
LIMIT 
         $reves_bckid,$nb_reves_aff

J'ai inversé les clauses ORDER BY et GROUP BY, essaie ainsi :

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
17
18
19
20
SELECT 
        RR.id, RR.pseudo_id, RR.pseudo, 
        RR.email, RR.email_publication, RR.titre, 
        RR.reve, RR.date, 
        RRVR.pseudo as pseudo_inscrit, RC.COUNT(*) 
FROM 
        reves_reves RR, 
        reves_reveurs RRVR, 
        reves_comments RC 
WHERE 
        RR.activity='on' 
        AND 
        RRVR.id = RR.pseudo_id 
        AND RC.id_reve=RR.id 
GROUP BY 
         RC.id 
ORDER BY 
         RR.id DESC 
LIMIT 
         $reves_bckid,$nb_reves_aff

RR.id et RC.id sont tous les 2 indépendants et uniques :
RR.id correspond à l'id du rêve enregistré dans la table reves_reves
RC.id correcpond à l'id du commentaire du reves enregistré dans la table reves_comments. D'où le WHERE (...) RC.id_reve=RR.id
RC.id_reve correpond en effet à l'id du rêve (enregistré dans la table reves_reves)

La je suis un peu crevé j'ai du mal :/
Mais enfait comme on ajoute un GROUP BY, il faut que l'on se garde de faire passer des lignes à la trappe, donc l'idée et de groupe sur un colonne unique dans le contexte de cette requete.
Et sans schéma j'ai un peu du mal à le définir la.

ps : qu'est-ce que l'indentation, en particulier dans les requête SQL :- ?

C'est juste de mettre en forme ta requete demanière à la rendre plus lisible. La requete que tu as présenté n'était pas indenté. Celle que je t'ai proposée l'était.

bye

**yazerty** · 23/01/2006, 23h44

Mmm ça marche déjà mieux

. Merci

. Merci également pour ton explication sur l'indentation; je tacherai d'y faire attention à l'avenir car c'est vrai que c'est plus lisible !

Mais par contre maintenant ça ne m'affiche plus que les rêves qui ont des commentaires

.

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
17
18
19
20
$sql = "SELECT
        RR.id, RR.pseudo_id, RR.pseudo,
        RR.email, RR.email_publication, RR.titre,
        RR.reve, RR.date,
        RRVR.pseudo as pseudo_inscrit, COUNT(RC.id) as comments_nb (il préfère RC.id que COUNT(*) apparemment...)
FROM
        reves_reves RR,
        reves_reveurs RRVR,
        reves_comments RC
WHERE
        RR.activity='on'
        AND
        RRVR.id = RR.pseudo_id
        AND RC.id_reve=RR.id
GROUP BY
         RC.id
ORDER BY
         RR.id DESC
LIMIT
         $reves_bckid,$nb_reves_aff";

Le problème doit venir de mon AND RC.id_reve=RR.id non ? Mais si je le supprime ça fait bugger la page (même pas de message d'erreur, lag infini de la page...). Ce que je peux comprendre. Par contre je ne vois pas comment m'y prendre autrement sur ça :- ?

**ePoX** · 24/01/2006, 00h08

Okkkalors.euh

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
17
18
19
20
21
22
23
 
SELECT 
        RR.id, RR.pseudo_id, RR.pseudo, 
        RR.email, RR.email_publication, RR.titre, 
        RR.reve, RR.date, 
        RRVR.pseudo as pseudo_inscrit, COUNT(RC.id) as comments_nb (il préfère RC.id que COUNT(*) apparemment...) 
FROM 
        reves_reves RR
 
INNER JOIN reves_reveurs RRVR
ON RRVR.id = RR.pseudo_id
 
LEFT JOIN reves_comments RC 
ON RC.id_reve=RR.id
 
WHERE 
        RR.activity='on' 
GROUP BY 
         RR.id 
ORDER BY 
         RR.id DESC 
LIMIT 
         $reves_bckid,$nb_reves_aff

le left join vas tenter d'associer la table RC et RR,si il n'y arrive pas je suppose qu'il renvoie NULL.

cf http://www.w3schools.com/sql/sql_join.asp

Et en faits si sa laggue qd tu retires la contrainte dans l'ancienne requete c'est parce que mysql est entrain de process une grosse requete.

**yazerty** · 24/01/2006, 19h24

Ca m'affiche bien tous les rêves

!
Par contre il me reste un petit problème à résoudre : les commentaires peuvent être online ou offline. Cette caractéristique est déterminée par un champ activity dans la table reves_comments, soit égal à "on" soit égal à "off". Je cherche donc à compter uniquement les commentaires "on".
J'ai donc modifié la requête de cette façon :

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
17
18
19
20
21
22
23
$sql = "SELECT
        RR.id, RR.pseudo_id, RR.pseudo,
        RR.email, RR.email_publication, RR.titre,
        RR.reve, RR.date,
        RRVR.pseudo as pseudo_inscrit, COUNT(RC.id) as comments_nb
FROM
        reves_reves RR
 
INNER JOIN reves_reveurs RRVR
ON RRVR.id = RR.pseudo_id
 
LEFT JOIN reves_comments RC
ON RC.id_reve=RR.id
 
WHERE
        RR.activity='on'
        AND RC.activity='on'
GROUP BY
         RR.id
ORDER BY
         RR.id DESC
LIMIT
         $reves_bckid,$nb_reves_aff";

(j'ai ajouté le AND RC.activity='on')

Le problème c'est que du coup je me retrouve avec le même problème : seuls les rêves quiu ont des commentaires sont affichés

. J'ai lu cette page consacrée aux jointures, mais n'ai pas réussi à y trouver une solution à ce problème

.

**ePoX** · 24/01/2006, 20h32

La je ne voies pas trop comment faire sans utiliser une sous requete.

Mais voila ce que je peut te donner comme piste :

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
SELECT 
        RR.id, RR.pseudo_id, RR.pseudo, 
        RR.email, RR.email_publication, RR.titre, 
        RR.reve, RR.date, 
        RRVR.pseudo as pseudo_inscrit, 
		(
			SELECT COUNT(id) 
			FROM reves_comments 
			WHERE reves_comments.id_reve = RR.id 
			AND activity = 'on' 
		) 
		as comments_nb 
FROM 
        reves_reves RR 
 
INNER JOIN reves_reveurs RRVR 
ON RRVR.id = RR.pseudo_id 
 
LEFT JOIN reves_comments RC 
ON RC.id_reve=RR.id 
 
WHERE 
        RR.activity='on' 
        AND RC.activity='on' 
GROUP BY 
         RR.id 
ORDER BY 
         RR.id DESC 
LIMIT 
         $reves_bckid,$nb_reves_aff

Ceci dit j'emet quelques reserves sur la rapidité d'execution de cette requete.
C'est à toi de voir si il n'est pas plus interessant de faire une requete count par reves, ou une autre requete ensembliste ne catégoriant que les count, ou enfin si celle que je t'ai présenté au dessus reste tout à fait satisfaisante.

bye

**yazerty** · 24/01/2006, 21h25

Il m'informe malheureusement que :

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'SELECT COUNT(id) FROM reves_comments WHERE re

Mais par ailleurs :

Une sous-requête doit être placée à la suite d'une clause WHERE ou HAVING, et doit remplacer une constante ou un groupe de constantes qui permettraient en temps normal d'exprimer la qualification.

"une requete count par reves" : c'est ce que je faisais avant il me semble non (je ne sais pas si je comprends bien) ?
"ou une autre requete ensembliste ne catégoriant que les count" : uh :- ?

**ePoX** · 24/01/2006, 21h34

Mais

Citation:
Une sous-requête doit être placée à la suite d'une clause WHERE ou HAVING,

totalement faux. cf le site de mysql.
J'ai même refait le test par acquis. Après il y à peut être une question de verionnage de mysql....

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'SELECT COUNT(id) FROM reves_comments WHERE re

essaie d'executer la requete toute seule pour determiner l'erreur.

SELECT COUNT(id)
FROM reves_comments
WHERE reves_comments.id_reve = RR.id
AND activity = 'on'

"une requete count par reves" : c'est ce que je faisais avant il me semble non (je ne sais pas si je comprends bien) ?

ptet

"ou une autre requete ensembliste ne catégoriant que les count" : uh :- ?

L'idée serait de faire une requete complémentaire portant sur le même ensemble de résultat, mais qui vise à n'extraire que le dénombrement via count().C'est plus clair ?

**yazerty** · 24/01/2006, 22h01

Il avait un peu de mal avec les noms des tables et des champs (ambigus) et j'ai corrigé un peu la chose en fonction de la situation. Et donc en fait en faisant ça tout seul ça marche :

Code :

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

1
2
3
4
SELECT COUNT(reves_comments.id)
FROM reves_comments, reves_reves
WHERE reves_comments.id_reve = reves_reves.id
AND reves_comments.activity = 'on'

Enfin ça n'affiche rien (logique) mais il n'y a pas d'erreur.

Par contre dès que je l'intègre dans l'ensemble ça génère l'erreur

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'SELECT COUNT(reves_comments.id) FROM reves_comments, reves_reve

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
SELECT
        RR.id, RR.pseudo_id, RR.pseudo,
        RR.email, RR.email_publication, RR.titre,
        RR.reve, RR.date,
        RRVR.pseudo as pseudo_inscrit,
      (
SELECT COUNT(reves_comments.id)
FROM reves_comments, reves_reves
WHERE reves_comments.id_reve = reves_reves.id
AND reves_comments.activity = 'on'
      )
      as comments_nb
FROM
        reves_reves RR
 
INNER JOIN reves_reveurs RRVR
ON RRVR.id = RR.pseudo_id
 
LEFT JOIN reves_comments RC
ON RC.id_reve=RR.id
 
WHERE
        RR.activity='on'
        AND RC.activity='on'
GROUP BY
         RR.id
ORDER BY
         RR.id DESC
LIMIT
         $reves_bckid,$nb_reves_aff