Discussion :

[Lomic]

Bonjour à tous,

Je migre un forum d'une plateforme à une autre (Connectix Boards vers PhpBB3) et je suis à la phase import des sujets.

Cependant en développant mon script qui va piocher d'un côté pour insérer de l'autre, je me suis rendu compte que pas mal de sujets ont 0 messages, du coup je voudrais les supprimer dans la table `cb_topics` avant de faire tourner mon script.

Y'a + de 50.000 sujets dans cette table mais seuls 47.000 d'entre eux ont des messages.

Du coup je tente ça :

Code :

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

SELECT topic_id from cb_topics WHERE topic_id NOT IN (SELECT msg_topicid AS topic_id FROM cb_messages GROUP BY msg_topicid HAVING COUNT( msg_id ) >0) LIMIT 0,10

Mais mysql n'a pas aimé

La sous-requête passe au poil par contre (c'est là que j'obtiens 47000 résultats par rapport aux 50000 topics existants).

Donc d'un côté une table `cb_messages` qui comporte msg_id et msg_topicid (l'id du topic auquel le message appartient) et de l'autre une table `cb_topics`qui comporte topic_id

Je veux trouver, puis éliminer les topics dont l'id n'est référencé dans aucun enregistrement de la table messages.

Des conseils pour le faire proprement et en douceur, je vais me faire gronder si je continue à faire tomber mysql

Merci

[al1_24]

Tout simplement comme ça

Code :

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

1
2
3
4
5
6
7
SELECT  topic_id
from    cb_topics   tpc
WHERE   not exists
        (   SELECT  null
            FROM    cb_messages msg
            where   msg.msg_topicid = tpc.topic_id
        )

[Lomic]

Merci mais je t'avoue que ça n'est pas limpide pour moi :

Code :

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

1
2
3
4
5
6
7
SELECT  topic_id
from    cb_topics   tpc
WHERE   not exists
        (   SELECT  null
            FROM    cb_messages msg
            where   msg.msg_topicid = tpc.topic_id
        )

si je traduis en français, c'est : trouver les topic_id, n'existant pas dans la sous-requête, qui retourne "rien" à chaque fois qu'elle trouve une correspondance entre msg_topicid et topic_id (correspondance qui se produit à chaque fois qu'il se trouve un message dont le champ msg_topicid a la valeur de topic_id)

point de vue charge de la base, c'est plutôt soft ou pas ? pas envie de refaire mal à mysql

merci pour ton retour et désolé de ne pas avoir répondu plus tôt, j'étais sur la route tout l'après-midi

[escartefigue]

Une requête (NOT)EXISTS a pour résultat un booléen vrai/faux et ce quelque soit le contenu de la clause SELECT.

par exemple WHERE EXISTS (SELECT 1 FROM MA_TABLE WHERE MA_COLONNE = 1) renvoie exactement la même chose que WHERE EXISTS (SELECT * FROM MA_TABLE WHERE MA_COLONNE = 1).

On utilise souvent une constante (SELECT 1) voire une division par zéro (SELECT1/0) peu importe.
Attention toutefois SELECT NULL n'est pas supporté par tous les SGBD

[Artemus24]

Salut à tous.

Bonne année, bonne santé, mes meilleurs voeux à toutes et à tous pour l'année 2020.

Mais mysql n'a pas aimé

C'est normal car le "in" ne sait gérer qu'un nombre limité de valeurs.
Vous avez le choix entre augmenter cette capacité ou utiliser le "exists" à la place.
La bonne façon de faire est la solution proposée par al1_24.

Mais en regardant de plus près, je constate que votre requête n'est pas conforme à ce que vous cherchez à faire.
Il est totalement inutile d'utiliser LIMIT.
Pourquoi passer par une sous requête et ne pas faire une jointure ? Voici ce que je vous propose :

Code :

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

1
2
3
4
5
6
7
8
9
select           t1.topics_id,
                 count(msg_id) as nbre
           from  cb_topics as t1
 
left outer join  cb_messages as t2
             on  t2.msg_topicid = t1.topics_id
 
       group by  t1.topics_id
         having  nbre < 1

Pour des raisons de performances, vérifiez :
a) la colonne msg_topicid doit être une clef étrangère.
b) mettre un index sur la colonne msg_id.

@+

[Lomic]

C'est normal car le "in" ne sait gérer qu'un nombre limité de valeurs.

bon à savoir

Il est totalement inutile d'utiliser LIMIT.

juste un exemple de mes compétences limitées en SQL je me suis dit que ça limiterait le nombre de résultats que la requête me sortirait, des fois que ça charge un peu trop le serveur, au moins il ne me sortirait que les 30 premiers résultats

Pourquoi passer par une sous requête et ne pas faire une jointure ?

parce que j'ai du mal avec les jointures

Code :

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

1
2
3
4
5
6
7
8
9
select           t1.topics_id,
                 count(msg_id) as nbre
           from  cb_topics as t1
 
left outer join  cb_messages as t2
             on  t2.msg_topicid = t1.topics_id
 
       group by  t1.topics_id
         having  nbre < 1

Je comprend ce code merci pour ces explications intéressantes et bonne année également

[tatayo]

Bonjour,
On peut simplifier la requête d'Artemus24:

Code sql :

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

1
2
3
4
5
6
 
select t1.topics_id,
from cb_topics as t1
left outer join cb_messages as t2
on  t2.msg_topicid = t1.topics_id
where t2.msg_topicid is null

Tatayo.

[Artemus24]

Salut à tous.

parce que j'ai du mal avec les jointures

A votre âge, avoir des problèmes de jointures ou de genoux (je - nous).
C'est pas bon du tout pour le relationnel !

On peut simplifier la requête d'Artemus24:

Merci Tatayo, je n'avais pas pensé au NULL.

C'est excellent, ça mérite '+1'.

@+

[SQLpro]

Bonjour,
On peut simplifier la requête d'Artemus24:

Code sql :

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

1
2
3
4
5
6
 
select t1.topics_id,
from cb_topics as t1
left outer join cb_messages as t2
on  t2.msg_topicid = t1.topics_id
where t2.msg_topicid is null

Tatayo.

C'est ce que l'on appelle une semi anti-jointure.

• SEMI : parce que l'on ne veut que les informations d'une des tables de la jointure et pas de l'autre.
• ANTI : parce que l'on filtre sur le fait que la jointure ne se fait pas entre les deux tables.

D'où l'OUTER JOIN accompagné par le filtre WHERE …. IS NULL

A +