Amélioration performance requête

Version imprimable

Bonjour,

J'ai une requête mysql qui fonctionne très bien.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
$req = "SELECT maj_etat_civil.id_maj,id_user,maj_num_dos,maj_civilite_id, CONCAT(maj_nom,' ', maj_prenom,' ', maj_prenom_usage) as maj_nom,maj_etat_civil.id_trt,
			CONCAT(tiers_nom,' ',tiers_adr1,' ',tiers_adr2,' ',tiers_adr3,' ',cp,' ',libelle) as maj_prenom
			FROM ".$prefixe_table."t_users_dossiers
			LEFT OUTER JOIN ".$prefixe_table."maj_etat_civil 
			ON t_users_dossiers.id_maj=maj_etat_civil.id_maj
			LEFT OUTER JOIN ".$prefixe_table."maj_adresses
			ON t_users_dossiers.id_maj=maj_adresses.id_maj
			LEFT OUTER JOIN ".$prefixe_table."t_tiers
			ON t_tiers.id_tiers=maj_adresses.id_tiers
			LEFT OUTER JOIN ".$prefixe_table."t_ville
			ON t_ville.ville_id=t_tiers.tiers_ville_id
			WHERE '".$today."' BETWEEN (gest_date_debut AND gest_date_fin)
			AND t_users_dossiers.id_trt 
			BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt 
			BETWEEN 1 AND 4
			AND tiers_nom LIKE '%$argument2%'
			OR '".$today."'>=gest_date_debut 
			AND 0<=gest_date_fin
			AND t_users_dossiers.id_trt 
			BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt 
			BETWEEN 1 AND 4
			AND tiers_nom LIKE '%$argument2%'
			OR '".$today."' BETWEEN (gest_date_debut AND gest_date_fin)
			AND t_users_dossiers.id_trt 
			BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt 
			BETWEEN 1 AND 4
			AND tiers_adr1 LIKE '%$argument2%'
			OR '".$today."'>=gest_date_debut 
			AND 0<=gest_date_fin
			AND t_users_dossiers.id_trt 
			BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt 
			BETWEEN 1 AND 4
			AND tiers_adr1 LIKE '%$argument2%'
			OR '".$today."' BETWEEN (gest_date_debut AND gest_date_fin)
			AND t_users_dossiers.id_trt 
			BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt 
			BETWEEN 1 AND 4
			AND tiers_adr2 LIKE '%$argument2%'
			OR '".$today."'>=gest_date_debut 
			AND 0<=gest_date_fin
			AND t_users_dossiers.id_trt 
			BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt 
			BETWEEN 1 AND 4
			AND tiers_adr2 LIKE '%$argument2%'
			'".$today."' BETWEEN (gest_date_debut AND gest_date_fin)
			AND t_users_dossiers.id_trt 
			BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt 
			BETWEEN 1 AND 4
			AND tiers_adr3 LIKE '%$argument2%'
			OR '".$today."'>=gest_date_debut 
			AND 0<=gest_date_fin
			AND t_users_dossiers.id_trt 
			BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt 
			BETWEEN 1 AND 4
			AND tiers_adr3 LIKE '%$argument2%'
			ORDER BY maj_nom, maj_prenom
			limit $start,$nb_majeur";

Mon problème est juste au niveau de la performance. Existe-t-il une syntaxe plus légère de requête pour mysql au niveau de mes "OR" ?

Merci d'avance

09/01/2007, 13h12
xave

Salut,

Là franchement je ne sais pas mais va jeter un oeil là, c'est pas forcément du MySQL mais il y a de bons conseils pour optimiser les requêtes. ;)
09/01/2007, 16h06
lodan

Merci Xave, après relecture de ces tutos, je ne vois rien.

Je pensais plus à la syntaxe de ma requête.
09/01/2007, 16h55
xave

Concernant la syntaxe, je ne vois pas ce que tu pourrais modifier... :?

Essaie peut-être d'aller poser ta question dans le forum Base de données, c'est plus spécifiquement du SQL que du php, tu y trouveras certainement une meilleure réponse. ;)

Au besoin je peux déplacer ton message.
09/01/2007, 17h05
nuphius

D'après mon gros livre de PHP une requete préparé avec la fonction mysqli_prepare() puis exécuté avec les fonctions qui vont avec serait plus rapide qu'une fonction préparé puis exécuté par mysqli_query().

je sais pas trop si c'est vrais mais bon tu peux toujours tanté, mais fonctionne que depuis php 5
09/01/2007, 18h36
lodan

Merci, mais le serveur n'est pas encore prêt pour le PHP5.

Il faut utiliser des parenthèses pour factoriser tes conditions et éviter les répétitions.

Si je comprends bien, tu as trois conditions composées :

- $today est entre gest_date_debut et gest_date_fin, ou bien supérieur à gest_date_debut et gest_date_fin est positive (???)

- t_users_dossiers.id_trt et maj_etat_civil.id_trt sont entre 1 et 4

- tiers_nom, tiers_adr1, tiers_adr2 ou tiers_adr3 contient $argument2

Cela nous donne :
Code:

1 2 3 4 5 6 7 8 9 10 WHERE ('$today' >= gest_date_debut AND ('$today' <= gest_date_fin OR 0<=gest_date_fin) ) AND t_users_dossiers.id_trt BETWEEN 1 AND 4 AND maj_etat_civil.id_trt BETWEEN 1 AND 4 AND (tiers_nom LIKE '%$argument2%' OR tiers_adr1 LIKE '%$argument2%' OR tiers_adr2 LIKE '%$argument2%' OR tiers_adr3 LIKE '%$argument2%' )
PS : je ne comprends vraiment pas ce que tu fais avec 0<=gest_date_fin... Si tu veux gérer une période avec un début mais sans fin, ce devrait être '$today' >= gest_date_debut AND gest_date_fin IS NULL.

Merci pour la simplification de la requête.

Cela donnera plutôt quelque chose comme cela :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
WHERE '".$today."' BETWEEN (gest_date_debut AND gest_date_fin)
			AND t_users_dossiers.id_trt BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt BETWEEN 1 AND 4
			AND (tiers_nom LIKE '%$argument2%' 
			  OR tiers_adr1 LIKE '%$argument2%'
			  OR tiers_adr2 LIKE '%$argument2%'
			  OR tiers_adr3 LIKE '%$argument2%'
			)
			OR '".$today."'>=gest_date_debut AND gest_date_fin IS NULL
			AND t_users_dossiers.id_trt BETWEEN 1 AND 4
			AND maj_etat_civil.id_trt BETWEEN 1 AND 4
			AND (tiers_nom LIKE '%$argument2%' 
			  OR tiers_adr1 LIKE '%$argument2%'
			  OR tiers_adr2 LIKE '%$argument2%'
			  OR tiers_adr3 LIKE '%$argument2%'
			)
			ORDER BY maj_nom, maj_prenom
			limit $start,$nb_majeur";

Tu retombes dans le travers de développer à coup de copier-coller !
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 WHERE ( '$today' BETWEEN (gest_date_debut AND gest_date_fin) OR '$today'>=gest_date_debut AND gest_date_fin IS NULL ) AND t_users_dossiers.id_trt BETWEEN 1 AND 4 AND maj_etat_civil.id_trt BETWEEN 1 AND 4 AND (tiers_nom LIKE '%$argument2%' OR tiers_adr1 LIKE '%$argument2%' OR tiers_adr2 LIKE '%$argument2%' OR tiers_adr3 LIKE '%$argument2%' ) ORDER BY maj_nom, maj_prenom LIMIT $start,$nb_majeur";
Une simplification supplémentaire de ta condition today est celle-ci, mais je ne suis pas sûr du tout qu'elle apporte un gain de performance...
Code:

1 2 3 4 WHERE '$today' BETWEEN gest_date_debut AND COALESCE(gest_date_fin, '9999-12-01') AND t_users_dossiers.id_trt BETWEEN 1 AND 4 ...
On remplace ainsi une fin NULL par une date future qui ne sera jamais atteinte par ton système, classiquement appelée "la fin des temps".

Citation:
Envoyé par Antoun
Tu retombes dans le travers de développer à coup de copier-coller !
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 WHERE ( '$today' BETWEEN (gest_date_debut AND gest_date_fin) OR '$today'>=gest_date_debut AND gest_date_fin IS NULL ) AND t_users_dossiers.id_trt BETWEEN 1 AND 4 AND maj_etat_civil.id_trt BETWEEN 1 AND 4 AND (tiers_nom LIKE '%$argument2%' OR tiers_adr1 LIKE '%$argument2%' OR tiers_adr2 LIKE '%$argument2%' OR tiers_adr3 LIKE '%$argument2%' ) ORDER BY maj_nom, maj_prenom LIMIT $start,$nb_majeur";
Une simplification supplémentaire de ta condition today est celle-ci, mais je ne suis pas sûr du tout qu'elle apporte un gain de performance...
Code:

1 2 3 4 WHERE '$today' BETWEEN gest_date_debut AND COALESCE(gest_date_fin, '9999-12-01') AND t_users_dossiers.id_trt BETWEEN 1 AND 4 ...
On remplace ainsi une fin NULL par une date future qui ne sera jamais atteinte par ton système, classiquement appelée "la fin des temps".
Pardon, c'est vrai, je prends les chemins de traverses. :mouarf:

Bon à part cela, c'est syntaximent beaucoup mieux comme cela :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 SELECT maj_etat_civil.id_maj,id_user,maj_num_dos,maj_civilite_id, CONCAT(maj_nom,' ', maj_prenom,' ', maj_prenom_usage) as maj_nom,maj_etat_civil.id_trt, CONCAT(tiers_nom,' ',tiers_adr1,' ',tiers_adr2,' ',tiers_adr3,' ',cp,' ',libelle) as maj_prenom, maj_commentaire FROM ".$prefixe_table."t_users_dossiers LEFT OUTER JOIN ".$prefixe_table."maj_etat_civil ON t_users_dossiers.id_maj=maj_etat_civil.id_maj LEFT OUTER JOIN ".$prefixe_table."maj_adresses ON t_users_dossiers.id_maj=maj_adresses.id_maj LEFT OUTER JOIN ".$prefixe_table."t_tiers ON t_tiers.id_tiers=maj_adresses.id_tiers LEFT OUTER JOIN ".$prefixe_table."t_ville ON t_ville.ville_id=t_tiers.tiers_ville_id WHERE ('$today' BETWEEN gest_date_debut AND gest_date_fin OR '$today'>=gest_date_debut AND gest_date_fin IS NULL) AND t_users_dossiers.id_trt BETWEEN 1 AND 4 AND maj_etat_civil.id_trt BETWEEN 1 AND 4 AND (tiers_nom LIKE '%$argument2%' OR tiers_adr1 LIKE '%$argument2%' OR tiers_adr2 LIKE '%$argument2%' OR tiers_adr3 LIKE '%$argument2%' OR cp LIKE '%$argument2%' OR libelle LIKE '%$argument2%' ) ORDER BY maj_nom, maj_prenom
Par contre pour le "COALESCE" ce qui donnerai cela :
Code:

1 2 WHERE ('$today' BETWEEN gest_date_debut AND gest_date_fin OR '$today'>=gest_date_debut AND COALESCE(gest_date_fin, '9999-12-01'))
Heu le résultat est que je n'ai plus aucun résultat et j'avoue en lisant la doc

Citation:

COALESCE(list)
Retourne le premier élément non- NULL de la liste :

mysql> SELECT COALESCE(NULL,1);
-> 1
mysql> SELECT COALESCE(NULL,NULL,NULL);
-> NULL

que je n'ai pas compris comment cela fonctionnait.

Merci encore.

13/01/2007, 15h39
Antoun
COALESCE(gest_date_fin, '9999-12-01') veut dire "si la gest_date_fin est renseignée, on la prend, si elle NULL on prend '9999-12-01'".

Donc
Code:

1 2 '$today' BETWEEN gest_date_debut AND COALESCE(gest_date_fin, '9999-12-01')
remplace les deux conditions :
Code:

1 2 3 '$today' BETWEEN (gest_date_debut AND gest_date_fin) OR '$today'>=gest_date_debut AND gest_date_fin IS NULL )
13/01/2007, 16h38
lodan

Merci, c'est bien ce que j'avais fait, mais dans ce cas

Citation:

'$today' BETWEEN gest_date_debut AND COALESCE(gest_date_fin, '9999-12-01')

mon résultat est null alors qu'avec

Citation:

'$today' BETWEEN (gest_date_debut AND gest_date_fin)
OR '$today'>=gest_date_debut AND gest_date_fin IS NULL )

j'ai 30 enregistrements et c'est bon.

A l'exécution dans le 1er cas, j'ai

Code:

SELECT maj_etat_civil.id_maj, maj_num_dos, id_user,maj_civilite_id, maj_nom, maj_prenom, maj_prenom_usage, maj_commentaire,maj_etat_civil.id_trt FROM t_users_dossiers LEFT OUTER JOIN maj_etat_civil ON t_users_dossiers.id_maj=maj_etat_civil.id_maj WHERE ('2007-01-13' BETWEEN gest_date_debut AND COALESCE(gest_date_fin, '9999-12-01')) AND t_users_dossiers.id_trt BETWEEN 1 AND 4 AND maj_etat_civil.id_trt BETWEEN 1 AND 4 and maj_nom like "%ab%" ORDER BY maj_nom, maj_prenom

Dans le 2ème :

Code:

SELECT maj_etat_civil.id_maj, maj_num_dos, id_user,maj_civilite_id, maj_nom, maj_prenom, maj_prenom_usage, maj_commentaire,maj_etat_civil.id_trt FROM t_users_dossiers LEFT OUTER JOIN maj_etat_civil ON t_users_dossiers.id_maj=maj_etat_civil.id_maj WHERE ('2007-01-13' BETWEEN gest_date_debut AND gest_date_fin OR '2007-01-13'>=gest_date_debut AND gest_date_fin IS NULL) AND t_users_dossiers.id_trt BETWEEN 1 AND 4 AND maj_etat_civil.id_trt BETWEEN 1 AND 4 and maj_nom like "%ab%" ORDER BY maj_nom, maj_prenom
13/01/2007, 23h35
Antoun

c'est bizarre... à quoi ressemblent les dates de début et fin des 30 lignes en question ?
14/01/2007, 16h18
lodan

Sur un extrait j'ai ceci comme date

Citation:

+ date_debut + date_fin +
+ 2006-10-15 + 0000-00-00 +
+ 2006-10-02 + 0000-00-00 +
+ 2007-01-06 + 0000-00-00 +
+ 2007-01-06 + 0000-00-00 +
+ 2007-01-10 + 0000-00-00 +
+ 2006-11-01 + 0000-00-00 +
+ 2006-11-08 + 0000-00-00 +
+ 2006-11-21 + 0000-00-00 +
14/01/2007, 19h57
Antoun

donc tes dates de fin ne sont pas NULL mais 0. C'est pour cela que le COALESCE ne marche pas. Il est plus étonnant que le IS NULL fonctionne, mais je suppose que cela s'explique par ta version de MySQL et/ou ton @@sql_mode (qui n'est sans doute pas très strict.
15/01/2007, 09h06
lodan

mysql 4.1.9.

Pour ce qui est de @@sql_mode, je ne m'en suis jamais soucié, j'ai laissé les valeurs par défaut lors de l'installation de phpmyadmin.