Discussion :

[serge0934]

Bonjour,

dans un ETL, j'ai souvent ce bloc dans les select

Code :

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

CONVERT(NUMERIC(30,2),ROUND(SUM(ISNULL(REVENUE,0)*CONVERT(DECIMAL(15,10),(1/CASE WHEN R.CURRENCY_CODE =''' + @PIVOTCURRENCY + ''' THEN 1 ELSE A.RATE END ))*(CASE WHEN ''' + @CURRENCY + '''=''' + @PIVOTCURRENCY + ''' THEN 1 ELSE B.RATE END)),2)) REVENUE

ma question:
faut-il faire une fonction du type

Code :

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

create function dbo.calc_revenue (@revenue float,@PIVOTCURRENCY varchar(50),@CURRENCY varchar(50)) return NUMERIC(30,2)

ou laisser le bloc ?

aura t on un gain de faire une function ?

merci

[Babyneedle]

Le seul avantage dans ce cas à utiliser une fonction est pour la clarté du code.

Une fonction est parfois un choix judicieux quand il faut sortir du contexte T-SQL comme pour faire des opérations répétitives ou avec des choix multiples en grandes quantités.

[elsuket]

Bonjour,

Effectivement, on pourrait en faire une fonction de type TABLE en ligne, qui ne retourne qu'une seule ligne. Veillez à l'utiliser avec l'opérateur APPLY
Attention à ne pas en faire une fonction scalaire qui est appelée pour chaque ligne du résultat (on peut le voir à l'aide de l'outil Profiler).

@++

[serge0934]

merci Elsuket,

donc un truc que j'utiliserai du type

Code :

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

1
2
3
select * 
from toto
       cross apply dbo.calc_revenue (@xxx)

?

[elsuket]

Exactement

@++