Code à optimiser

[Just-Soft]

Salut !

mon 1er code

Code :

1
2
3
4
5
6
7
8
9
10
11
FROM tb_process
   INNER JOIN tb_step ON (tb_process.code_step = tb_step.code_step)
   INNER JOIN tb_energie ON (tb_step.code_step = tb_energie.code_step)
   INNER JOIN tb_unite ON (tb_step.code_unite = tb_unite.code_unite)
   AND (tb_process.date_bilan = tb_energie.date_mois)
 
WHERE 
   (
      ((tb_process.date_bilan BETWEEN :date_debut AND :date_fin) AND
       (tb_energie.date_mois BETWEEN :date_debut AND :date_fin))
   )

mon 2ème code

Code :

1
2
3
4
5
6
7
8
9
10
FROM tb_process
   INNER JOIN tb_step ON (tb_process.code_step = tb_step.code_step)
   INNER JOIN tb_energie ON (tb_step.code_step = tb_energie.code_step)
   INNER JOIN tb_unite ON (tb_step.code_unite = tb_unite.code_unite)
 
WHERE 
   (
      (  (tb_process.date_bilan = tb_energie.date_mois) AND (tb_process.date_bilan BETWEEN :date_debut AND :date_fin) AND
       (tb_energie.date_mois BETWEEN :date_debut AND :date_fin))
   )

dans le 1er code la condition

tb_process.date_bilan = tb_energie.date_mois

est dans les jointures mais dans le second elle est dans la clause where.

Le résultat obtenu est le même mais d'après vous quel est le code le plus optimisé point de vue rapidité puisque ma base de données est distante.

Merci par avance.

[dehorter olivier]

Selon mes connaissances empiriques et la logiques, les 2 codes sont corrects et auront le même "niveau" d’exécution.

Penses à regarder les plans d'exécution généré, a priori ils devraient être identique.

par contre, je modifierais la requête sous cette forme

Code :

1
2
3
4
5
6
7
8
9
10
....
FROM tb_process
   INNER JOIN tb_step ON (tb_process.code_step = tb_step.code_step)
   INNER JOIN tb_energie ON (tb_step.code_step = tb_energie.code_step AND (tb_process.date_bilan = tb_energie.date_mois)
   INNER JOIN tb_unite ON (tb_step.code_unite = tb_unite.code_unite)
WHERE 
   (
      ((tb_process.date_bilan BETWEEN :date_debut AND :date_fin) AND
       (tb_energie.date_mois BETWEEN :date_debut AND :date_fin))
   )

et même :

Code :

1
2
3
4
5
....
FROM tb_process
   INNER JOIN tb_step ON (tb_process.code_step = tb_step.code_step AND tb_process.date_bilan BETWEEN :date_debut AND :date_fin)
   INNER JOIN tb_energie ON (tb_step.code_step = tb_energie.code_step AND tb_process.date_bilan = tb_energie.date_mois AND tb_energie.date_mois BETWEEN :date_debut AND :date_fin)
   INNER JOIN tb_unite ON (tb_step.code_unite = tb_unite.code_unite)

Là aussi la vitesse devrait être la même

[Just-Soft]

Ok mais je crois qu'avec des tables énormes c'est mieux d'utiliser la 1ière forme.

[dehorter olivier]

Je ne vois pas pourquoi, mais les super-experts de firebird devraient connaitre la réponse

As tu regardé les plans d’exécution, à mon humble avis, ils doivent être identiques

[Just-Soft]

Envoyé par dehorter olivier

Je ne vois pas pourquoi, mais les super-experts de firebird devraient connaitre la réponse

As tu regardé les plans d’exécution, à mon humble avis, ils doivent être identiques

Re,

je viens de voir les plans pour les deux requêtes, ils sont identiques.

[Barbibulle]

Peut aussi s'écrire :

Code :

1
2
3
4
5
 
....
FROM tb_process, tb_step, tb_energie, tb_unite
WHERE
tb_process.code_step = tb_step.code_step AND tb_process.date_bilan BETWEEN :date_debut AND :date_fin AND tb_step.code_step = tb_energie.code_step AND tb_process.date_bilan = tb_energie.date_mois AND tb_energie.date_mois BETWEEN :date_debut AND :date_fin AND tb_step.code_unite = tb_unite.code_unite

D'un point de vue de la norme SQL, et pour faciliter la lecture des requetes SQL il est préférable de laisser dans le WHERE les critères de sélection et dans les jointures apres le ON uniquement ce qui sert à lier les tables (le plus souvent correspond aux clés étrangères liées aux clés primaires).

Donc la forme :

Code :

1
2
3
4
5
6
7
8
9
....
FROM tb_process
   INNER JOIN tb_step ON (tb_process.code_step = tb_step.code_step)
   INNER JOIN tb_energie ON (tb_step.code_step = tb_energie.code_step AND (tb_process.date_bilan = tb_energie.date_mois)
   INNER JOIN tb_unite ON (tb_step.code_unite = tb_unite.code_unite)
WHERE 
   (
      ((tb_process.date_bilan BETWEEN :date_debut AND :date_fin) AND
       (tb_energie.date_mois BETWEEN :date_debut AND :date_fin))

Est plus adaptée.