Discussion :

[mitou]

Bonjour,

J'ai créé une requete sql, mais elle me semble un peu lourde à l'exécution, je me demandais si il en existait une forme "optimisée".
Cette requete sera exécutée en shell.

Code :

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

select a,b,(a/b*100) from (((select count(*) as a from drive where argument = 'OK') ra), ((select count(*) as b from drive) rb))

Merci d'avance

[Scorpi0]

Bonjour,

Commence par bien indenter a requête !!

[mitou]

OK je vais tenter d'indenter ma requete:

Code :

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

1
2
3
4
select a,b,(b/a*100) from(
     ((select count(*) as b FROM centreon.host WHERE host_register = "1") ra),
     ((select count(*) as a from centreon.host) rb)
)

[dgi77]

Ca n'est pas étonnant que ce soit lourd : tu n'as pas de jointure entre tes 2 sous-requêtes ra et rb, donc tu fais un produit cartésien.
Pour être lourd, ça doit être du très lourd, même pas besoin d'avoir des tables très volumineuses, et en plus fonctionnellement ça n'a certainement aucun sens !

[Scorpi0]

Le produit cartésien ne se fait que sur 2 tables d'une ligne, rien de bien méchant donc.

Pas moins lourd mais plus joli je trouve :

Code :

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

1
2
3
4
5
6
7
8
9
with 
t1 as (select count(*) as a from drive),
t2 as (
        select count(*) as b
        from drive 
        where argument = 'OK'
      ) 
select a, b, (a/b)*100
from t1, t2

Une question de gout ^^
Tu as combien de ligne dans ta table ? Quel proportion de argument = 'OK' ?


Une autre idée, si ça te dérange pas d'avoir ton résultat en lignes plutôt qu'en colonne :

Code :

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

1
2
3
select count(1), decode(argument, 'OK', 'OK', 'NOT OK')
from drive 
group by decode(argument, 'OK', 'OK', 'NOT OK')

[mitou]

Dans ma table je dois avoir dans les 20 lignes.

Et la proportion d'état "OK", un maximum j'espère, sauf gros soucis technique...

Ca n'est pas étonnant que ce soit lourd : tu n'as pas de jointure entre tes 2 sous-requêtes ra et rb, donc tu fais un produit cartésien.
Pour être lourd, ça doit être du très lourd, même pas besoin d'avoir des tables très volumineuses, et en plus fonctionnellement ça n'a certainement aucun sens !

Merci de l'information, mais si je post ici, c'est que j'ai bien conscience du problème. Je ne cherche pas des critiques, mais plutôt des conseils. Donc si suite à ta jolie démonstration de la nullité de ma requete tu avais également quelques solutions à proposer, je suis également preneur.

[Scorpi0]

Pour remettre en lignes la requête que je t'ai postée au dessus, tu peux faire un truc de ce genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
with myTable as
(
  select count(1) as n, decode(argument , 'OK', 'OK', 'NOT OK') as etat
  from drive 
  group by decode(argument , 'OK', 'OK', 'NOT OK')
)
select nOK, nNOTOK, nOK/nNOTOK*100
from
(
  select distinct (select n from myTable where etat = 'OK') as nOK, 
             (select n from myTable where etat = 'NOT OK') as nNOTOK
  from myTable
)

Y'a des fonctions pivots à partir d'Oracle 11g, mais ça je connais pas encore ^^

Au final, si ta table ne fait que 20 lignes et n'est pas destinée à en contenir plus, ta toute première requête devrait largement suffire. Tu te poses pas des questions pour optimiser des requête qui tape 20 lignes.
Si tu trouves ton temps d'exécution trop lent, le problème doit venir d'ailleurs.

[orafrance]

Dans ma table je dois avoir dans les 20 lignes.

Si en exécutant une requête sur 20 lignes tu as déjà des soucis de perfs, le problème vient surement d'ailleurs

Tu n'utiliserais pas un site web ou excel ou autre pour voir le résultat plutôt que SQL*Plus ?