Probleme de requête SQL trop longue

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:

---

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

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é.
@+

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:

---

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 :

Citation:

---

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.

Je pense que ça devrait aller mieux comme ça

Code:

---

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.

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