Discussion :

[bndexsen]

bonjour je suis débutant sur SQL server 2012
Lorsque j'ai exécuté ma requête j'ai un message : Error message : divide by zero error encountered
C'est un problème au niveau de la formule "fnCP"

voici la requête :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
SELECT dbo.ViewDLG_Aggregate.DLG_BHD_BTCH_BatchID_value, dbo.ViewDLG_Aggregate.StartTime, dbo.ViewDLG_Aggregate.EndTime, dbo.ViewDLG_Aggregate.TotalLength, 
                  dbo.ViewDLG_Aggregate.MinDLG_CTRL_COLD_DiameterPVXY, dbo.ViewDLG_Aggregate.MaxDLG_CTRL_COLD_DiameterPVXY, 
                  dbo.ViewDLG_Aggregate.AvgDLG_CTRL_COLD_DiameterPVXY, dbo.ViewDLG_Aggregate.StdevDLG_CTRL_COLD_DiameterPVXY, 
                  dbo.ViewDLG_Aggregate.MinDLG_CTRL_COLD_OvalityPV, dbo.ViewDLG_Aggregate.MaxDLG_CTRL_COLD_OvalityPV, 
                  dbo.ViewDLG_Aggregate.AvgDLG_CTRL_COLD_OvalityPV, dbo.ViewDLG_Aggregate.StdevDLG_CTRL_COLD_OvalityPV, dbo.BHD.BHD_BHD_BTCH_BatchID_value, 
                  dbo.BHD.BHD_BHD_BTCH_PresetLength_value, dbo.BHD.BHD_BHD_BTCH_RecipeName_value, dbo.BHD.BHD_BHD_BTCH_StatisticsON_value, 
                  dbo.BHD.BHD_CTRL_COLD_DiameterLSL_value, dbo.BHD.BHD_CTRL_COLD_DiameterSP_value, dbo.BHD.BHD_CTRL_COLD_DiameterUSL_value, 
                  dbo.BHD.BHD_CTRL_COLD_OvalityUSL_value, dbo.fnCp(dbo.BHD.BHD_CTRL_COLD_DiameterUSL_value, dbo.BHD.BHD_CTRL_COLD_DiameterLSL_value, 
                  dbo.ViewDLG_Aggregate.StdevDLG_CTRL_COLD_DiameterPVXY) AS Cp_COLD_Diameter, dbo.fnCpK(dbo.BHD.BHD_CTRL_COLD_DiameterUSL_value, 
                  dbo.BHD.BHD_CTRL_COLD_DiameterLSL_value, dbo.ViewDLG_Aggregate.AvgDLG_CTRL_COLD_DiameterPVXY, 
                  dbo.ViewDLG_Aggregate.StdevDLG_CTRL_COLD_DiameterPVXY) AS CpK_COLD_Diameter
FROM     dbo.BHD INNER JOIN
                  dbo.ViewDLG_Aggregate ON dbo.BHD.BHD_BHD_BTCH_BatchID_value = dbo.ViewDLG_Aggregate.DLG_BHD_BTCH_BatchID_value

voici la formule :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
USE [Dexline_10]
GO
/****** Object:  UserDefinedFunction [dbo].[fnCp]    Script Date: 12/08/2016 13:58:38 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
ALTER function [dbo].[fnCp] (@USL real,@LSL real, @StdPV real)
returns real
begin
	declare @CP real
		if @StdPV=0
		begin
			set @CP = 999999
		end
		set @CP = (@USL - @LSL)/(6*@StdPV)
	return @CP
end

Si quelqu'un peut m'aider ....
Merci bien

[al1_24]

L'erreur est dans la fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
begin
	declare @CP real
		if @StdPV=0
		begin
			set @CP = 999999
		end
		else
		begin
			set @CP = (@USL - @LSL)/(6*@StdPV)
		end
	return @CP
end

[bndexsen]

Merci je vais regardé dans la formule , je vois pas ou elle se situe .... , de plus c'est une fonction a déjà été utilisé pour un mise en service sans problème ...

[al1_24]

Je t'ai donné la solution. Il manque juste un else

[bndexsen]

Je t'ai donné la solution. Il manque juste un else

Juste ! merci bien ...

[elsuket]

Bonjour,

Le problème avec les fonctions scalaires, c'est qu'elles sont exécutées une fois par ligne de résultat, avec des effets néfastes sur le comportement de l'optimiseur.
Si l'on transforme la fonction scalaire en fonction de table en ligne, on élimine souvent ces deux effets :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
CREATE FUNCTION fnCp
	(
		@USL real
		,@LSL real
		, @StdPV real
	)
RETURNS TABLE
WITH SCHEMABINDING
AS
RETURN
(
	SELECT CASE @StdPV
			WHEN 0 THEN 999999
			ELSE (@USL - @LSL)/(6*@StdPV)
		END AS Cp
)

Ici la clause WITH SCHEMABINDING indique au moteur qu'il doit faire des vérifications supplémentaires, notamment déterminer si la fonction accède à des données, système ou non, si la fonction est déterministe, ... Ces propriétés influent sur l'optimiseur : si la fonction est déterministe, précise et liée au schéma (sous-entendu : la fonction ne peut pas changer de définition au cours de l'exécution d'une requête), alors le moteur utilise des règles qui ne sont pas les mêmes que lorsque ce n'est pas le cas, ce qui peut le conduire, par exemple, à ne pas produire de spools, ...

La requête devient alors :

Code :

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

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
SELECT		AG.DLG_BHD_BTCH_BatchID_value
		, AG.StartTime
		, AG.EndTime
		, AG.TotalLength
		, AG.MinDLG_CTRL_COLD_DiameterPVXY
		, AG.MaxDLG_CTRL_COLD_DiameterPVXY
		, AG.AvgDLG_CTRL_COLD_DiameterPVXY
		, AG.StdevDLG_CTRL_COLD_DiameterPVXY
		, AG.MinDLG_CTRL_COLD_OvalityPV
		, AG.MaxDLG_CTRL_COLD_OvalityPV
		, AG.AvgDLG_CTRL_COLD_OvalityPV
		, AG.StdevDLG_CTRL_COLD_OvalityPV
		, BHD.BHD_BHD_BTCH_BatchID_value
		, BHD.BHD_BHD_BTCH_PresetLength_value
		, BHD.BHD_BHD_BTCH_RecipeName_value
		, BHD.BHD_BHD_BTCH_StatisticsON_value
		, BHD.BHD_CTRL_COLD_DiameterLSL_value
		, BHD.BHD_CTRL_COLD_DiameterSP_value
		, BHD.BHD_CTRL_COLD_DiameterUSL_value
		, BHD.BHD_CTRL_COLD_OvalityUSL_value
		, CP.Cp AS Cp_COLD_Diameter
		, dbo.fnCpK(BHD.BHD_CTRL_COLD_DiameterUSL_value, BHD.BHD_CTRL_COLD_DiameterLSL_value, AG.AvgDLG_CTRL_COLD_DiameterPVXY, AG.StdevDLG_CTRL_COLD_DiameterPVXY) AS CpK_COLD_Diameter
FROM     	dbo.BHD AS BHD
INNER JOIN	dbo.ViewDLG_Aggregate AS AG
			ON BHD.BHD_BHD_BTCH_BatchID_value = AG.DLG_BHD_BTCH_BatchID_value
CROSS APPLY	dbo.fnCp(BHD.BHD_CTRL_COLD_DiameterUSL_value, BHD.BHD_CTRL_COLD_DiameterLSL_value, AG.StdevDLG_CTRL_COLD_DiameterPVXY) AS CP

Avec un peu de change, on peut faire de même avec la fonction dbo.fnCpK().

@++

[aieeeuuuuu]

Bonjour,

Vous testez bien si le dénominateur vaut zéro (et attribuez alors une valeur arbitraire à la variable de retour), mais le calcul n'est pas mis dans un ELSE et est donc effectué dans tous les cas, même lorsque le dénominateur vaut zéro.

Ajoutez un ELSE, ou faites directement un RETURN dans le IF afin de sortir de la fonction.

[EDIT]grillé par al1_24[/EDIT]