Discussion :

[piscou51000]

Bonjour,

Je viens faire appel aux talents de développez.com car je me casse les dents sur une requête sans réussir à la rendre satisfaisante.

Pour expliquer le problème:
1/ J'ai une table affaire qui possède un champs responsable.
2/ La table affaire possède un champs fk_departement. Par une jointure sur departement et sur secteur, on peux connaître le directeur du secteur.
3/ Je possède aussi une table affaire_user qui contient des couples fk_affaire et fk_user. Qui permet d'ouvrir des droits sur l'affaire à un user lambda.

Je veux créer pour un utilisateur donnée un écran de l'ensemble des affaires dont il a accès ( soit il en est le responsable, soit le directeur du secteur de l'affaire, soit cité sur l'affaire).

Voici ce que j'ai pondu comme requête mais elle met de 15 à 30 secondes avant de donner une réponse.

Code :

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

1
2
3
4
5
6
7
8
9
10
SELECT a.ref
FROM   ( llx_affaire as affaire
LEFT JOIN llx_affaire_user AS au ON affaire.rowid = au.fk_affaire ), llx_departement AS dep, llx_secteur AS secteur
WHERE affaire.fk_departement = dep.rowid
AND dep.secteur = secteur.rowid
AND (
au.fk_user =64
OR a.fk_responsable =64
OR secteur.fk_directeur =64
)

Pour info,
llx_affaire contient 6000 lignes.
llx_affaire_user contient 20 000 lignes.
llx_departement contient 20 lignes.
llx_secteur contient 6 lignes.


Merci d'avance de vos réponses et de votre aide.

Maximilien

[Maljuna Kris]

Saluton,
Je n'arrive pas bien à démêler ton code SQL qui mélange la syntaxe de jointure normalisée avec des jointures par clause WHERE.
Je vais essayer de reformuler cela en SQL normalisé.
@+

[CinePhil]

Tu as un alias 'a' qui ne correspond à aucune table. Je suppose qu'il s'agit de llx_affaire ?

Les alias sont faits pour simplifier et alléger le texte des requêtes. Mettre un alias 'secteur' à la place de llx_secteur n'apporte pas grand chose !

Tu mélanges la syntaxe obsolète depuis 1992 pour les jointures (FROM WHERE) avec la syntaxe normalisée (JOIN).

Choisis le bon camp camarade ! Adopte la jointure normalisée !

Code :

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

1
2
3
4
5
6
7
8
SELECT a.ref
FROM llx_affaire AS a
LEFT JOIN llx_affaire_user AS au ON a.rowid = au.fk_affaire 
INNER JOIN llx_departement AS dep ON a.fk_departement = dep.rowid
  INNER JOIN llx_secteur AS s ON dep.secteur = s.rowid
WHERE au.fk_user = 64
  OR a.fk_responsable = 64
  OR s.fk_directeur = 64

Ton besoin :

Je veux créer pour un utilisateur donné un écran de l'ensemble des affaires dont il a accès ( soit il en est le responsable, soit le directeur du secteur de l'affaire, soit cité sur l'affaire).

Ta requête me semble bonne.

Les tables sont-elles bien indexées ?
Il faut indexer les clés étrangères.

[Maljuna Kris]

Je pense que ça devrait aller mieux comme ça

Code sql :

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

1
2
3
4
5
6
7
8
9
10
11
SELECT aff.ref
FROM  llx_affaire AS aff
LEFT JOIN llx_affaire_user AS au 
    ON aff.rowid = au.fk_affaire
INNER JOIN llx_departement AS dep
    ON aff.fk_departement = dep.rowid
INNER JOIN llx_secteur AS sect
    ON dep.secteur = sect.rowid
WHERE  au.fk_user = 64
    OR aff.fk_responsable = 64
    OR secteur.fk_directeur = 64

Mais bien sûr les conseils d'indexation de Cinephil ne sont pas superflus.

[piscou51000]

Merci beaucoup de vos réponses.

Vous avez bien identifiés les problèmes.
L'indexation des champs + les jointures normalisées = résultat immédiat de la requête.

Encore une fois merci, les qualités des réponses de developpez.com sont réellement de bonnes qualités.

Bon WE à tous