Discussion :

[t-die]

Bonjour,

J'ai une question toute simple je fais une requête qui permet de voir qu'une personne a eu un entretien, étant donné la BdD mal conçue l'id de la personne est "harcodée":

Code :

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

1
2
 
client.cli_dossier_entretien AS e ON (e.dossier_id = d."id" AND e.collab_id = 236)

Dans ce cas, ça fonctionne parfaitement par contre, si je cherche d'autre personne en utilisant :

Code :

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

1
2
3
4
5
6
 
client.cli_dossier_entretien AS e ON (e.dossier_id = d."id" AND e.collab_id != 236)
 
client.cli_dossier_entretien AS e ON (e.dossier_id = d."id" AND e.collab_id <> 236)
 
client.cli_dossier_entretien AS e ON (e.dossier_id = d."id" AND e.collab_id < 236)

l'id 236 n'est jamais exclus

mais si je tape :

Code :

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

1
2
 
client.cli_dossier_entretien AS e ON (e.dossier_id = d."id" AND e.collab_id > 236)

J'ai aucune ligne qui ressort, ce qui est juste.

Est-ce que quelqu'un a une idée?

Merci d'avance.

[Waldar]

Il faudrait voir la requête complète, notamment le type de jointure.

[t-die]

Oui Effectivement :

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
 
SELECT
d."id" AS "did dossier client",
e."id" AS "eid dossier entretient",
"p"."id" AS "peid prestation",
"p".prestation_name,
r.collab_id
FROM
  client.cli_dossier AS d
INNER JOIN 
  client.cli_dossier_prestation_rel AS r ON (r.dossier_id = d."id")
INNER JOIN
  client.cli_prestation AS p ON (p."id" = r.prestation_id AND p."id"=5) 
INNER JOIN 
  -- client.cli_dossier_entretien AS e ON (e.dossier_id = d."id" AND e.mode_id = 1) 
  --client.cli_dossier_entretien AS e ON (e.dossier_id = d."id" AND e.mode_id = 1 AND r.collab_id != 236) 
WHERE
  e.entretien_date BETWEEN '2015-01-01' AND '2015-07-31' AND r.collab_id != 236

Si je fais la condition à la fin de la requête ça fonctionne est-ce ma jointure qui est fausse? Ou la conception de la BdD mauvaise...

[escartefigue]

Si vous cherchez à savoir si une personne a eu ou non un entretien, il faut alors faire une jointure de type "outer" vers la table des entretiens
Une jointure inner ne construit que l'intersection des tables jointes, donc les éléments présents à la fois dans les 2 tables

Voici une requete qui affiche les éléments pour l'individu 236, qu'il ait eu ou non un entretien :

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
SELECT dos.id AS "did dossier client",
       ent.id AS "eid dossier entretien",
       prs.id AS "peid prestation",
       prs.prestation_name,
       rel.collab_id
FROM  client.cli_dossier 
   AS dos
INNER JOIN client.cli_dossier_prestation_rel 
   AS rel
   ON rel.dossier_id = dos.id
INNER JOIN client.cli_prestation 
   AS prs
   ON prs.id = rel.prestation_id 
LEFT OUTER JOIN client.cli_dossier_entretien 
  AS ent 
  ON ent.dossier_id = dos.id 
WHERE rel.collab_id = 236
  AND prs.id = 5 
  AND (ent.mode_id = 1   OR  ent.mode_id is null)
  AND (    ent.entretien_date BETWEEN '2015-01-01' AND '2015-07-31' 
       OR  ent.entretien_date is null)

Modifs :

- j'ai remplacé l'inner join par un outer join pour la table entretien
- j'ai ajouté un test de nullité sur les colonnes de la table d'entretien, puisque l'absence d'entretien est possible
- j'ai déplacé les critères de filtrage dans le WHERE, pour ne conserver que ceux de jointure dans les ON (prs.id et ent.mode)
- j'ai supprimé les guillemets inutiles (je trouve que ca facilite la lecture)
- j'ai mis des alias un peu plus parlants
- j'ai supprimé le "T" à la fin d'entretien

Si la personne n'a pas eu d'entretien, alors la colonne dossier d'entretien affichera "null"

[t-die]

Bonsoir, bonne nuit,

Je vous remercie de la réponse, je ferais un test et je vous tiens au courant demain mais appartement, le métier c'est trompé depuis le début...environ 2 ans, l'id de l'employé ne dois pas être filtré tous les employés peuvent avoir ce type d'entretien selon un contrat que le métier a vérifier aujourd'hui....

Merci encore.