Discussion :

[kurul1]

Bonjour à tous

Je développe actuellement un logiciel en C++ qui attaque une base de données Access. Jusque là pas de problème

J'essaye dans une boite de dialogue d'afficher un DBGrid avec la liste des Groupes que j'ai créé ainsi que le nombre de Patient présent dans chaque groupe.

Voici la structure des deux tables de ma base de données utilisée dans cette requête :

Code :

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

1
2
3
4
 
Table Groupe : idGroupe, Groupe)
Table Patient : idPatient, Nom, Prénom, ...., GroupePatient, .......)
ou GroupePatient est une clé étrangère pointant sur la table Groupe (idGroupe)

Voici la requête que j'exécute

Code :

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

1
2
 
SELECT idGroupe, Groupe, (SELECT COUNT(*) FROM Patient, Groupe WHERE GroupePatient = idGroupe) AS NbPatient FROM Groupe

Sauf qu'elle me renvoit dans NbPatient pour chaque groupe non pas le nombre de Patient de chaque groupe mais le Nombre de Patient Total

Qu'est ce qui ne vas pas dans ma requête à votre avis

Merci d'avance pour votre aide

[al1_24]

L'erreur est là :

Code :

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

(SELECT COUNT(*) FROM Patient, Groupe WHERE GroupePatient = idGroupe)

[kurul1]

Merci pour ta réponse al1_24
Mais je ne comprend pas ce que tu me proposes car c'est déjà présent dans ma requête ???

[skuatamad]

Il faut corréler la sous-requête :

Code :

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

1
2
3
SELECT g.idGroupe, g.Groupe, 
       (SELECT COUNT(*) FROM Patient p WHERE p.GroupePatient = g.idGroupe) AS NbPatient 
  FROM Groupe g

Ou passer par une jointure :

Code :

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

1
2
3
4
select g.idGroupe, g.Groupe, count(*) as NbPatient
  from groupe g
  left join patient p on p.GroupePatient = g.idGroupe
 group by g.idGroupe, g.Groupe

[al1_24]

Ton problème démontre l'utilité de qualifier toutes les colonnes d'une requête parle nom (ou l'alias) de la table à laquelle elles se réfèrent.
Si tu l'avais fait, ton erreur t'aurait sauté aux yeux et tu ne te serais pas posé de question.
Réécrivons la requête en qualifiant les colonnes :

Code :

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

1
2
3
4
5
6
7
8
SELECT  grp.idGroupe
    ,   grp.Groupe
    ,   (   SELECT  COUNT(*) 
            FROM    Patient pat
                ,   Groupe  gp2
            WHERE   pat.GroupePatient = (grp ou gp2?).idGroupe
        )   AS NbPatient
FROM    Groupe  grp

Est-ce que l'utilisation de la table Groupe dans la sous-requête se justifie ?

[kurul1]

Merci à vous deux pour vos réponses

Pour skuatamad
Nickel, les deux requêtes me donne ce que je veux mais je t'avoue que pour la deuxième, je suis un peut rouillé pour arriver à la sortir de moi-même

J'ai d'ailleurs fait quelques test de mon côté avec l'aide d'une amie et voilà ce que nous avons obtenus et qui fonctionne aussi

Code :

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

1
2
3
4
5
6
 
SELECT idGroupe, Groupe, 
 COUNT(GroupePatient) AS NbPatient 
FROM Patient, Groupe
 WHERE GroupePatient = idGroupe
GROUP BY idGroupe, Groupe

Pour al1_24
Les deux ne fonctionne pas correctement
La première avec grp me double le nombre de Patient dans chaque groupe
La seconde avec grp2 me demande de saisir IdGroupe

[al1_24]

Pour al1_24
Les deux ne fonctionnent pas correctement
La première avec grp me double le nombre de Patient dans chaque groupe
La seconde avec grp2 me demande de saisir IdGroupe

Je n'ai pas cherché à te fournir une requête qui fonctionnait, j'ai juste essayé de mettre en évidence pourquoi ta requête ne fonctionnait pas afin que tu puisses comprendre comment la corriger.
PS : Pour la seconde, je n'ai pas écrit grp2 mais gp2, ce qui explique pourquoi Access te demande de saisir IdGroupe.

[skuatamad]

et voilà ce que nous avons obtenus et qui fonctionne aussi

Tu as fait une jointure interne, moi une jointure externe.

Tout dépend du besoin s'il est nécessaire d'afficher les groupes qui n'ont pas de patient avec le count(*) à 0 alors il faut utiliser une jointure externe, sinon une jointure interne. Comme tu étais parti sur une requête corrélée, la traduction en jointure est une jointure externe.

[kurul1]

J'ai bien noté, merci pour les explications skuatamad
Et effectivement, j'ai besoin que le Count(*) à 0 soit affiché

Pour al1_24 une faute de frappe de ma part
Mais j'ai bien compris d'où venait le problème