Discussion :

[Gecko]

Bonjour!

J'ai un souci avec ma requête SQL:

Code SQL :

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

1
2
3
4
5
6
7
8
9
10
		WITH RECURSIVE children(nid,date,cid,parent,content,uid,level) AS (
			SELECT nc.news_id,nc.comment_pubdate,nc.comment_id,nc.comment_parent,nc.comment_content,nc.user_id, 1
			    FROM toine.news_comments nc
			    WHERE nc.news_id = :news_id
		    UNION ALL
			SELECT news_id,comment_pubdate,comment_id,comment_parent,comment_content,user_id, e.level+1
			    FROM toine.news_comments nc, children e
			    WHERE nc.comment_parent = e.cid
		)
		SELECT * FROM children ORDER BY cid

Les résultats ressortent plusieurs fois ce qui est embêtant. J'ai tenté d'ajouter GROUP BY mais ça me demande d'ajouter tous les arguments de children().

Du coup je sèche un peu, le but est d'éviter d'avoir plusieurs requêtes pour afficher une liste de commentaires.

Merci d'avance pour votre aide

[al1_24]

S'il s'agit d'un GROUP BY sur toutes les colonnes, autant utiliser un DISTINCT...

[Gecko]

S'il s'agit d'un GROUP BY sur toutes les colonnes, autant utiliser un DISTINCT...

Ca ne change rien

Je veut juste éliminer les doublons générés par la requête, je ne cherche pas forcément à traiter toutes les colonnes.

Je comprend pas pourquoi j'ai ce comportement

[al1_24]

Si DISTINCT ne regroupe pas les lignes, il ne s'agit donc pas de doublons complet.
La première chose à faire est de trouver la règle qui permettra d'identifier les lignes à conserver.

[estofilo]

Si comment_id est unique sur toute la table, la requête parait juste.
Tu es sûr que ce n'est pas un problème avec les données plutôt?

[Gecko]

J'ai cerné le problème, c'est level qui est pas au point.

En gros il répète le post jusqu'à atteindre le bon chiffre, voici un petit exemple en image.

http://www.toine.pro/theme/shiny_hill/etvepocrever.png

Et du coups beh c'est assez embêtant parce que je vois pas trop comment corriger ça

[estofilo]

OK. En 2eme lecture je pense que la sous- requête avec le level à 1 n'est pas juste car elle renvoie tous les commentaires de la discussion alors qu'elle devrait renvoyer uniquement ceux qui n'ont pas de parent (colonne comment_parent à NULL ou à 0 suivant ce que tu as choisi)

[Gecko]

Code :

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

1
2
3
4
5
6
7
8
9
10
		WITH RECURSIVE children(nid,date,cid,parent,content,uid,level) AS (
			SELECT nc.news_id,nc.comment_pubdate,nc.comment_id,nc.comment_parent,nc.comment_content,nc.user_id, 0
			    FROM toine.news_comments nc
			    WHERE nc.news_id = :news_id
		    UNION ALL
			SELECT news_id,comment_pubdate,comment_id,comment_parent,comment_content,user_id, e.level+1
			    FROM toine.news_comments nc, children e
			    WHERE nc.comment_parent = e.cid
		)
		SELECT * FROM children ORDER BY date, level

Voici la structure de ma table :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
                                      Table « toine.news_comments » 
     Colonne     |  Type   |                                Modificateurs 
-----------------+---------+------------------------------------------------------------------------------ 
 comment_id      | integer | non NULL Par défaut, nextval('toine.news_comments_comment_id_seq'::regclass) 
 comment_pubdate | bigint  | 
 comment_plus    | bigint  | non NULL Par défaut, 0 
 comment_minus   | bigint  | non NULL Par défaut, 0 
 comment_content | text    | 
 news_id         | bigint  | 
 user_id         | bigint  | 
 comment_parent  | bigint  | non NULL Par défaut, 0 
Index : 
    "news_comments_pkey" PRIMARY KEY, btree (comment_id)

J'ai toujours ce comportement, ou alors j'ai pas compris ce que tu m'as dis, ce qui se peut aussi ^^

[estofilo]

Ce que je voulais dire c'est remplacer

Code :

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

WHERE nc.news_id = :news_id

par

Code :

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

WHERE nc.news_id = :news_id AND comment_parent is null

[Gecko]

Merci beaucoup, ça fonctionne et ça m'a bien dégoûté en même temps

Je ne pensais pas que la solution serait aussi simple.

Et existe-t-il un moyen de couper la requête de manière propre ?
Par exemple si j'ai 140 commentaires et que je veux qu'il y en ait 10 par page.
C'est possible de stopper au dernier enfant précédant la limite si le commentaire suivant a trop d'enfants pour la respecter ?