Optimisation d'une subquery

[elwiss]

Bonjour!
Je suis présentement stagiaire dans une boite de T.I et aujourd'hui on m'a demandé de faire une requête Oracle qui fais affaire à 3 table pour insérer dans une 4ième. J'ai fait le premier truc qui m'est venu à l'esprit soit un 'switch case' de subquery. Je sais pertinemment que ce n'est pas la manière la plus efficace mais c'est la seule que je connais. J'ai souvent lu que les JOIN sont plus efficace que les subquery.

J'aimerai savoir de quel manière je pourrais ré-écrire mon énoncé de manière efficace.

Code :

1
2
3
4
5
6
7
 
SELECT A.*, Case when (SELECT length_dist_unit FROM scenario WHERE A.sce_scenario_name = scenario_name) = 'FT'
									then Cast((SELECT wheelbase FROM ac_data WHERE A.ad_ac_code = ac_code ) AS decimal(10,1))
								when (SELECT length_dist_unit FROM scenario WHERE A.sce_scenario_name = scenario_name) = 'M'
									then Cast(((SELECT wheelbase FROM ac_data WHERE A.ad_ac_code = ac_code ) * 0.3048) AS decimal(10,1))
								End AS WHEELBASE          
FROM scn_aircraft A;

Ma question est une simple curiosité et par souci de bien faire les choses parce que mon algo ne va servir qu'une fois.

Merci de votre aide

[skuatamad]

Ca devrait être quelque chose comme ça, si tu n'as pas besoin de la jointure externe (LEFT JOIN) supprime la (en supprimant le mot clé LEFT)

Code :

1
2
3
4
5
6
7
8
9
SELECT A.*,
       case when s.length_dist_unit = 'FT'
            then Cast(ad.wheelbase AS decimal(10,1))
            when s.length_dist_unit = 'M'
            then Cast(ad.wheelbase * 0.3048) AS decimal(10,1))
        end AS WHEELBASE
  FROM scn_aircraft A
  LEFT JOIN scenario s ON s.scenario_name = A.sce_scenario_name
  LEFT JOIN ac_data ad ON ad.ac_code = A.ad_ac_code

[elwiss]

Merci beaucoup pour cette réponse rapide! C'est beaucoup plus simple comme ça!

[elwiss]

Je viens d'essayer la réponse que skuatamad m'a proposé pour comparer les vitesse d'exécution j’obtiens une erreur: SQL statement not properly ended.

Je n'arrive pas à voir pourquoi...

[skuatamad]

Je viens juste de voir ton message, j'ai fait une erreur de syntaxe, il faut évidemment changer la ligne :

Code :

then Cast(ad.wheelbase * 0.3048) AS decimal(10,1))

par :

Code :

then Cast(ad.wheelbase * 0.3048 AS decimal(10,1))

Mais je pense que tu l'avais trouvé par toi même.