[2008 R2] Arithmetic overflow

**Kropernic** · 18/01/2013, 11h34

Bonjour,

Je rencontre un souci lors de l'exécution d'un job et je ne parviens pas à situer le problème.

Voici le code du step qui pose problème :

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
/*----------------------------------------------------------------------------------*
*SCRIPT D'IMPORTATION DE L'ANCIENNE DB GIFTCARD DANS LA NOUVELLE DB                    *
*  _                                                                          _        *
* /!\ PENSER A DESACTIVER LES TRIGGER AFTER UPDATE SUR LES TABLES STS ET SFS /!\    *    
* ¯¯¯                                                                         ¯¯¯    *
*-----------------------------------------------------------------------------------*/
SET ANSI_WARNINGS OFF;
 
DECLARE @SEQNRHIST    INT;
DECLARE @BARCODE    VARCHAR(13);
DECLARE @DATE        DATETIME;
DECLARE @ACTION        VARCHAR(1);
DECLARE    @ORIGIN        INT;
DECLARE @DESTTYPE    INT;
DECLARE    @DEST        INT;
DECLARE @VALUE        INT;
DECLARE    @SALETYPE    INT;
DECLARE    @USERTYPE    INT;    
 
DECLARE @GFT_ID        INT;
DECLARE @STR_ID        INT;
DECLARE @DOS_ID        INT;
DECLARE @DECIMAL_VALUE    DECIMAL(6,2);
DECLARE @TMP        VARCHAR(100);
DECLARE @TRN_TILL    SMALLINT;
DECLARE @TRN_TICKET    SMALLINT;
DECLARE @USERNAME    VARCHAR(50);
 
DECLARE C CURSOR FOR
 
SELECT
        SEQNRHIST,
        BARCODE,
        DATTIM,
        ACTION,
        ORIGIN,
        DESTTYPE,
        DEST,
        VALUE,
        SALETYPE,
        USERTYPE
FROM
        DBO.T_CHEQUE_ACHAT_HISTORY
WHERE
        ACTION = 'T'
ORDER BY
        DATTIM,
        SEQNRHIST
 
OPEN C;
 
FETCH NEXT FROM C INTO @SEQNRHIST, @BARCODE, @DATE, @ACTION, @ORIGIN, @DESTTYPE, @DEST, @VALUE, @SALETYPE, @USERTYPE;
 
WHILE @@FETCH_STATUS = 0
BEGIN
    --ON RECUPERE L'ID DU GIFT
    EXEC TMP_UP_CREATE_GIFT_IF_NOT_EXISTS @BARCODE, @GFT_ID OUTPUT, @VALUE;
 
    --ON NOTE LA LIGNE EN COURS DE TRAITEMENT POUR IDENTIFIER SI PLANTAGE
    UPDATE DBO.T_CURRENT_LINE_PROCESSING_CLP
    SET
            SEQNRHIST = @SEQNRHIST,
            BARCODE = @BARCODE,
            ACTION = @ACTION
 
    --ON DEFINIT SI L'UTILISATEUR EST NORMAL OU ADMIN
    IF @USERTYPE <> 0
        SET @USERNAME = 'ADMIN';
    ELSE
        SET @USERNAME = 'NORMAL';
 
    --VENDU A LA CAISSE (TTR_ID = 0)
    IF @ACTION = 'T'
    BEGIN
        --ON RECUPERE L'ID DU MAGASIN
        SET @ORIGIN = @ORIGIN * 10000
        IF @ORIGIN = 0
            SET @STR_ID = 17
        ELSE
            EXEC TMP_UP_GET_STORE_ID @ORIGIN, @STR_ID OUTPUT;
 
        --ON RECUPERE LA CAISSE ET LE TICKET
        SET @TMP =  RIGHT('000000'+CAST(@DEST AS VARCHAR(6)),6);
        SET @TRN_TILL = CAST(LEFT(@TMP,2) AS SMALLINT);
        SET @TRN_TICKET = CAST(RIGHT(@TMP,4) AS SMALLINT);
        IF @USERTYPE <> 0 --SI L'UTILISATION EST SIMULEE PAR UN ADMIN --> CAISSE ET TICKET = 0
        BEGIN
            SET @TRN_TICKET = 0;
            SET @TRN_TILL = 0;
        END
 
        --ON INSERE LA TRANSACTION
        SET @DECIMAL_VALUE = CAST(@VALUE AS DECIMAL(10,2)) / 100;
        INSERT INTO DBO.T_TRANSACTION_TRN(    TTR_ID, GFT_ID, STR_ID, TRN_DATE, TRN_TILL, TRN_TICKET, TRN_VALUE, TRN_CREATED_BY)
        VALUES(0, @GFT_ID, @STR_ID, @DATE, @TRN_TILL, @TRN_TICKET, @DECIMAL_VALUE, @USERNAME);
 
        --ON LE RETIRE DU STOCK DU MAGASIN
        UPDATE dbo.V_STORE_STOCK_VSS
        SET
                SSS_DATE_OUT = @DATE
        WHERE
                GFT_ID = @GFT_ID
            AND SSS_DATE_OUT IS NULL
            AND STR_ID = @STR_ID
    END
 
 
    FETCH NEXT FROM C INTO @SEQNRHIST, @BARCODE, @DATE, @ACTION, @ORIGIN, @DESTTYPE, @DEST, @VALUE, @SALETYPE, @USERTYPE;
END
 
CLOSE C;
DEALLOCATE C;

Le message d'erreur étant celui-ci :

Executed as user: XXXXXXXX. Arithmetic overflow occurred. [SQLSTATE 01000] (Message 3606) Cannot insert the value NULL into column 'CHQ_AMOUNT', table 'GIFT_MANAGEMENT.dbo.T_CHEQUE_CHQ'; column does not allow nulls. INSERT fails. [SQLSTATE 23000] (Error 515) The statement has been terminated. [SQLSTATE 01000] (Error 3621). The step failed.

J'ai donc été voir dans le seul endroit du code où une insertion dans la table T_CHEQUE_CHQ est susceptible d'avoir lieu. Il s'agit de la procédure stockée TMP_UP_CREATE_GIFT_IF_NOT_EXISTS dont voici le code :

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
USE [GIFT_MANAGEMENT]
GO
 
/****** Object:  StoredProcedure [dbo].[TMP_UP_CREATE_GIFT_IF_NOT_EXISTS]    Script Date: 01/18/2013 10:28:14 ******/
SET ANSI_NULLS ON
GO
 
SET QUOTED_IDENTIFIER ON
GO
 
 
CREATE PROCEDURE [dbo].[TMP_UP_CREATE_GIFT_IF_NOT_EXISTS]
    @BARCODE    CHAR(13),
    @GFT_ID        INT OUTPUT,
    @VALUE        INT = NULL
AS
BEGIN
    SET NOCOUNT ON;
    SET ANSI_WARNINGS OFF;
 
    DECLARE @TCA_ID        SMALLINT = 0;
    DECLARE @TCA_CODE    SMALLINT = CAST(SUBSTRING(@BARCODE,3,3) AS SMALLINT);
    DECLARE @CRD_SERIAL    INT    = CAST(SUBSTRING(@BARCODE,6,7) AS INT);
 
    DECLARE @CHQ_TYPE    TINYINT = CAST(SUBSTRING(@BARCODE,3,2) AS TINYINT);
    DECLARE @CHQ_YEAR    TINYINT = CAST(SUBSTRING(@BARCODE,5,1) AS TINYINT);
    DECLARE @CHQ_SERIAL INT = CAST(SUBSTRING(@BARCODE,6,7) AS INT);
 
    IF LEFT(@BARCODE,2) = '21' --CHEQUE
    BEGIN
        --CREATION DU CHEQUE S'IL N'EXISTE PAS
        IF NOT EXISTS    (
                        SELECT
                                *
                        FROM
                                DBO.T_CHEQUE_CHQ
                        WHERE
                                CHQ_PREFIX = 21
                            AND    CHQ_TYPE = @CHQ_TYPE
                            AND    CHQ_YEAR = @CHQ_YEAR
                            AND    CHQ_SERIAL = @CHQ_SERIAL
                        )
        BEGIN
            INSERT INTO    DBO.T_GIFT_GFT DEFAULT VALUES;
            SET @GFT_ID = SCOPE_IDENTITY();
 
            INSERT INTO    DBO.T_CHEQUE_CHQ(GFT_ID, CHQ_PREFIX, CHQ_TYPE, CHQ_YEAR, CHQ_SERIAL, CHQ_AMOUNT)
            VALUES(@GFT_ID, 21, @CHQ_TYPE, @CHQ_YEAR, @CHQ_SERIAL, CAST(@CHQ_TYPE AS DECIMAL(6,2)));
        END
        ELSE
            SELECT
                    @GFT_ID = GFT_ID
            FROM
                    DBO.T_CHEQUE_CHQ
            WHERE
                    CHQ_TYPE = @CHQ_TYPE
                AND    CHQ_YEAR = @CHQ_YEAR
                AND    CHQ_SERIAL = @CHQ_SERIAL
    END
    ELSE
        IF LEFT(@BARCODE,2) = '24' --CARTE
        BEGIN
            --CREATION DU TYPE DE CARTE S'IL N'EXISTE PAS
            IF NOT EXISTS    (
                            SELECT
                                    *
                            FROM
                                    DBO.T_TYPE_CARD_TCA
                            WHERE
                                    TCA_CODE = @TCA_CODE
                            )
            BEGIN
                INSERT INTO    DBO.T_TYPE_CARD_TCA(TCA_STANDARD, TCA_CODE)
                VALUES(0, @TCA_CODE);
                SET @TCA_ID = SCOPE_IDENTITY();
            END
            ELSE
                SELECT    
                        @TCA_ID = TCA_ID
                FROM
                        DBO.T_TYPE_CARD_TCA
                WHERE
                        TCA_CODE = @TCA_CODE
 
            --CREATION DE LA CARTE SI ELLE N'EXISTE PAS
            IF NOT EXISTS    (
                            SELECT
                                    *
                            FROM
                                    DBO.T_CARD_CRD
                            WHERE
                                    TCA_ID = @TCA_ID
                                AND    CRD_SERIAL = @CRD_SERIAL
                            )
            BEGIN
                INSERT INTO DBO.T_GIFT_GFT DEFAULT VALUES;
                SET @GFT_ID = SCOPE_IDENTITY();
 
                INSERT INTO DBO.T_CARD_CRD(GFT_ID, CRD_PREFIX, TCA_ID, CRD_SERIAL, CRD_RECHARGEABLE)
                VALUES(@GFT_ID, 24, @TCA_ID, @CRD_SERIAL, 0) 
            END
            ELSE
                SELECT
                        @GFT_ID = GFT_ID
                FROM
                        DBO.T_CARD_CRD
                WHERE
                        TCA_ID = @TCA_ID
                    AND    CRD_SERIAL = @CRD_SERIAL
        END
        ELSE
            IF LEFT(@BARCODE,2) = '22' --CHEQUE VARIABLE
            BEGIN
                --CREATION DU CHEQUE S'IL N'EXISTE PAS
                IF NOT EXISTS    (
                                SELECT
                                        *
                                FROM
                                        DBO.T_CHEQUE_CHQ
                                WHERE
                                        CHQ_PREFIX = 22
                                    AND    CHQ_TYPE = @CHQ_TYPE
                                    AND    CHQ_YEAR = @CHQ_YEAR
                                    AND    CHQ_SERIAL = @CHQ_SERIAL
                                )
                BEGIN
                    INSERT INTO    DBO.T_GIFT_GFT DEFAULT VALUES;
                    SET @GFT_ID = SCOPE_IDENTITY();
 
                    DECLARE @CHQ_AMOUNT DECIMAL(6,2);
                    SET @CHQ_AMOUNT = CAST(@VALUE AS DECIMAL(6,2)) / 100;
                    INSERT INTO    DBO.T_CHEQUE_CHQ(GFT_ID, CHQ_PREFIX, CHQ_TYPE, CHQ_YEAR, CHQ_SERIAL, CHQ_AMOUNT)
                    VALUES(@GFT_ID, 22, @CHQ_TYPE, @CHQ_YEAR, @CHQ_SERIAL, @CHQ_AMOUNT);
                END
                ELSE
                    SELECT
                            @GFT_ID = GFT_ID
                    FROM
                            DBO.T_CHEQUE_CHQ
                    WHERE
                            CHQ_TYPE = @CHQ_TYPE
                        AND    CHQ_YEAR = @CHQ_YEAR
                        AND    CHQ_SERIAL = @CHQ_SERIAL
            END
END
 
GO

Comme je m'attendais à ce que ce truc ne marche pas du premier coup, je note la ligne en cours de traitement dans la table T_CURRENT_PROCCESSING_LINE_CLP.

Je sais donc exactement quelle ligne plante et j'ai voulu reproduire cette erreur en reprenant le même code que le step (avec le curseur en moins) et en initialisant les variables "manuellement" en faisant un select sur la ligne concernnée. Et là, pas de souci. Du coup, je ne comprends pas.

J'ai cherché un peu partout où cette erreur pouvait se produire mais je ne trouve pas.

D'autant plus que la ligne traitée juste avant celle défectueuse est exactement la même à l'exection de la valeur de la colonne BARCODE.

Bref, je suis totalement perdu...
Auriez-vous une idée vers quoi me diriger pour corriger cela ?

**Kropernic** · 18/01/2013, 12h36

Pour ma part, l'erreur se produit forcément dans la procédure stockée dont j'ai fourni le code car la table contenant la ligne en cours de traitement n'est pas mise à jour. Or, cette mise à jour se trouve juste après la procédure stockée (elle aurait en fait du être avant mais ce mauvais placement est finalement le bienvenu car il permet de localiser où se situe le problème).

Cependant, j'ai exécuté la procédure stockée en passant en paramètre les valeurs de la ligne qui pose problème et elle fonctionne parfaitement.

Quelle pourrait être la cause d'un comportement comme celui-ci ?

**aieeeuuuuu** · 18/01/2013, 12h46

Envoyé par Kropernic

Comme je m'attendais à ce que ce truc ne marche pas du premier coup, je note la ligne en cours de traitement dans la table T_CURRENT_PROCCESSING_LINE_CLP.

Je sais donc exactement quelle ligne plante et j'ai voulu reproduire cette erreur en reprenant le même code que le step (avec le curseur en moins) et en initialisant les variables "manuellement" en faisant un select sur la ligne concernnée. Et là, pas de souci. Du coup, je ne comprends pas.

Est-ce que vous gérez une transaction ? dans ce cas l'insertion de la ligne "qui plante" peut être annulée lors de l'erreur, et la "dernière" ligne que vous voyez est en fait la dernière qui ait fonctionné.

Une question :
est-il normal que vous insériez CAST(@CHQ_TYPE AS DECIMAL(6,2)) dans la colonne CHQ_AMOUNT de votre table T_CHEQUE_CHQ ???

pouvez vous poster la DDL de cette table ?

**Kropernic** · 18/01/2013, 12h53

Niveau transaction, je suis un noob... J'ai bien essayé mais j'obtenais des messages d'erreur que je ne comprenais pas. Du coup, pas de transaction. Enfin, pas de transaction explicite. Il n'est pas impossible qu'il y en ait qui soit implicite.

Pour la question sur l'insertion, oui c'est normal.

Dans le cas d'un gift dont le barcode commence par '21', je sais alors qu'il s'agit d'un gift cheque dont la valeur est égale au type. C.à.d. que les gift dont les barcodes commence par '2105' ont une valeur de 5€, ceux qui commence par '2110' ont une valeur de 10€, etc. où les 3e et 4e digit correspondent au type.

Pour info, la ligne sur lequel s'est planté le step du job à comme valeur pour les colonnes BARCODE et VALUE respectivement '228880124526' et 1000.

Avec cela, on arrive donc dans la dernière partie de procédure. Le préfixe 22 sur un barcode définissant un gift-cheque à valeur variable, j'ai donc le montant de cette valeur dans une variable. Ce qui n'est pas le cas pour les gift-cheque à valeur fixe (préfixe = '21') car elle est égale au type.

C'est mal foutu... Mais c'est pour ça que j'importe tout dans une nouvelle DB ^^

**Kropernic** · 18/01/2013, 12h56

J'oubliais !

J'ai donc testé la procédure en créant sa petite soeur nommée UP_TEST dans laquelle j'ai remplacée les ordres d'insertion par des ordres de sélection sur les variables de la clause VALUES afin de voir si j'obtenais une valeur null pour la colonne CHQ_AMOUNT.

Et ce ne fut pas le cas...

Je sais bien que l'erreur arithmetic overflow se produit lorqu'on veut mettre un nombre dans une variable qui est trop grand pour le type de la variable.

Mais y aurait-il d'autres cas ???

**Kropernic** · 18/01/2013, 13h13

Sorry pour le triple poste mais je me rends compte que j'ai oublié de poster le DDL de la table T_CHEQUE_CHQ. Le voici :

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
USE [GIFT_MANAGEMENT]
GO
 
/****** Object:  Table [dbo].[T_CHEQUE_CHQ]    Script Date: 01/18/2013 12:12:29 ******/
SET ANSI_NULLS ON
GO
 
SET QUOTED_IDENTIFIER ON
GO
 
SET ANSI_PADDING ON
GO
 
CREATE TABLE [dbo].[T_CHEQUE_CHQ](
    [GFT_ID] [int] NOT NULL,
    [CHQ_PREFIX] [tinyint] NOT NULL,
    [CHQ_TYPE] [tinyint] NOT NULL,
    [CHQ_YEAR] [tinyint] NOT NULL,
    [CHQ_SERIAL] [int] NOT NULL,
    [CHQ_AMOUNT] [decimal](6, 2) NOT NULL,
    [CHQ_CREATED_ON] [datetime] NOT NULL,
    [CHQ_CREATED_BY] [varchar](100) NOT NULL,
    [CHQ_MODIFIED_ON] [datetime] NULL,
    [CHQ_MODIFIED_BY] [varchar](100) NULL,
 CONSTRAINT [PK_T_CHEQUE_CHQ] PRIMARY KEY CLUSTERED 
(
    [GFT_ID] ASC
)WITH (PAD_INDEX  = OFF, STATISTICS_NORECOMPUTE  = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS  = ON, ALLOW_PAGE_LOCKS  = ON) ON [PRIMARY]
) ON [PRIMARY]
 
GO
 
SET ANSI_PADDING OFF
GO
 
ALTER TABLE [dbo].[T_CHEQUE_CHQ]  WITH CHECK ADD  CONSTRAINT [FK_T_CHEQUE_CHQ_T_GIFT_GFT] FOREIGN KEY([GFT_ID])
REFERENCES [dbo].[T_GIFT_GFT] ([GFT_ID])
GO
 
ALTER TABLE [dbo].[T_CHEQUE_CHQ] CHECK CONSTRAINT [FK_T_CHEQUE_CHQ_T_GIFT_GFT]
GO
 
ALTER TABLE [dbo].[T_CHEQUE_CHQ] ADD  DEFAULT (getdate()) FOR [CHQ_CREATED_ON]
GO
 
ALTER TABLE [dbo].[T_CHEQUE_CHQ] ADD  DEFAULT (suser_sname()) FOR [CHQ_CREATED_BY]
GO

EDIT :

Pire ! Si je modifie le code du step pour qu'il reprenne à la ligne qui a planté, il continue le traitement avec succès pour quelques milliers de lignes et replante avec la même erreur et toujours apparemment sans raison... Du moins, je ne vois pas...

**Kropernic** · 21/01/2013, 10h42

Personne n'aurait une petite idée de où et cette procédure pourrait générer un arithmetic overflow ?

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
USE [GIFT_MANAGEMENT]
GO
 
/****** Object:  StoredProcedure [dbo].[TMP_UP_CREATE_GIFT_IF_NOT_EXISTS]    Script Date: 01/18/2013 10:28:14 ******/
SET ANSI_NULLS ON
GO
 
SET QUOTED_IDENTIFIER ON
GO
 
 
CREATE PROCEDURE [dbo].[TMP_UP_CREATE_GIFT_IF_NOT_EXISTS]
    @BARCODE    CHAR(13),
    @GFT_ID        INT OUTPUT,
    @VALUE        INT = NULL
AS
BEGIN
    SET NOCOUNT ON;
    SET ANSI_WARNINGS OFF;
 
    DECLARE @TCA_ID        SMALLINT = 0;
    DECLARE @TCA_CODE    SMALLINT = CAST(SUBSTRING(@BARCODE,3,3) AS SMALLINT);
    DECLARE @CRD_SERIAL    INT    = CAST(SUBSTRING(@BARCODE,6,7) AS INT);
 
    DECLARE @CHQ_TYPE    TINYINT = CAST(SUBSTRING(@BARCODE,3,2) AS TINYINT);
    DECLARE @CHQ_YEAR    TINYINT = CAST(SUBSTRING(@BARCODE,5,1) AS TINYINT);
    DECLARE @CHQ_SERIAL INT = CAST(SUBSTRING(@BARCODE,6,7) AS INT);
 
    IF LEFT(@BARCODE,2) = '21' --CHEQUE
    BEGIN
        --CREATION DU CHEQUE S'IL N'EXISTE PAS
        IF NOT EXISTS    (
                        SELECT
                                *
                        FROM
                                DBO.T_CHEQUE_CHQ
                        WHERE
                                CHQ_PREFIX = 21
                            AND    CHQ_TYPE = @CHQ_TYPE
                            AND    CHQ_YEAR = @CHQ_YEAR
                            AND    CHQ_SERIAL = @CHQ_SERIAL
                        )
        BEGIN
            INSERT INTO    DBO.T_GIFT_GFT DEFAULT VALUES;
            SET @GFT_ID = SCOPE_IDENTITY();
 
            INSERT INTO    DBO.T_CHEQUE_CHQ(GFT_ID, CHQ_PREFIX, CHQ_TYPE, CHQ_YEAR, CHQ_SERIAL, CHQ_AMOUNT)
            VALUES(@GFT_ID, 21, @CHQ_TYPE, @CHQ_YEAR, @CHQ_SERIAL, CAST(@CHQ_TYPE AS DECIMAL(6,2)));
        END
        ELSE
            SELECT
                    @GFT_ID = GFT_ID
            FROM
                    DBO.T_CHEQUE_CHQ
            WHERE
                    CHQ_TYPE = @CHQ_TYPE
                AND    CHQ_YEAR = @CHQ_YEAR
                AND    CHQ_SERIAL = @CHQ_SERIAL
    END
    ELSE
        IF LEFT(@BARCODE,2) = '24' --CARTE
        BEGIN
            --CREATION DU TYPE DE CARTE S'IL N'EXISTE PAS
            IF NOT EXISTS    (
                            SELECT
                                    *
                            FROM
                                    DBO.T_TYPE_CARD_TCA
                            WHERE
                                    TCA_CODE = @TCA_CODE
                            )
            BEGIN
                INSERT INTO    DBO.T_TYPE_CARD_TCA(TCA_STANDARD, TCA_CODE)
                VALUES(0, @TCA_CODE);
                SET @TCA_ID = SCOPE_IDENTITY();
            END
            ELSE
                SELECT    
                        @TCA_ID = TCA_ID
                FROM
                        DBO.T_TYPE_CARD_TCA
                WHERE
                        TCA_CODE = @TCA_CODE
 
            --CREATION DE LA CARTE SI ELLE N'EXISTE PAS
            IF NOT EXISTS    (
                            SELECT
                                    *
                            FROM
                                    DBO.T_CARD_CRD
                            WHERE
                                    TCA_ID = @TCA_ID
                                AND    CRD_SERIAL = @CRD_SERIAL
                            )
            BEGIN
                INSERT INTO DBO.T_GIFT_GFT DEFAULT VALUES;
                SET @GFT_ID = SCOPE_IDENTITY();
 
                INSERT INTO DBO.T_CARD_CRD(GFT_ID, CRD_PREFIX, TCA_ID, CRD_SERIAL, CRD_RECHARGEABLE)
                VALUES(@GFT_ID, 24, @TCA_ID, @CRD_SERIAL, 0) 
            END
            ELSE
                SELECT
                        @GFT_ID = GFT_ID
                FROM
                        DBO.T_CARD_CRD
                WHERE
                        TCA_ID = @TCA_ID
                    AND    CRD_SERIAL = @CRD_SERIAL
        END
        ELSE
            IF LEFT(@BARCODE,2) = '22' --CHEQUE VARIABLE
            BEGIN
                --CREATION DU CHEQUE S'IL N'EXISTE PAS
                IF NOT EXISTS    (
                                SELECT
                                        *
                                FROM
                                        DBO.T_CHEQUE_CHQ
                                WHERE
                                        CHQ_PREFIX = 22
                                    AND    CHQ_TYPE = @CHQ_TYPE
                                    AND    CHQ_YEAR = @CHQ_YEAR
                                    AND    CHQ_SERIAL = @CHQ_SERIAL
                                )
                BEGIN
                    INSERT INTO    DBO.T_GIFT_GFT DEFAULT VALUES;
                    SET @GFT_ID = SCOPE_IDENTITY();
 
                    DECLARE @CHQ_AMOUNT DECIMAL(6,2);
                    SET @CHQ_AMOUNT = CAST(@VALUE AS DECIMAL(6,2)) / 100;
                    INSERT INTO    DBO.T_CHEQUE_CHQ(GFT_ID, CHQ_PREFIX, CHQ_TYPE, CHQ_YEAR, CHQ_SERIAL, CHQ_AMOUNT)
                    VALUES(@GFT_ID, 22, @CHQ_TYPE, @CHQ_YEAR, @CHQ_SERIAL, @CHQ_AMOUNT);
                END
                ELSE
                    SELECT
                            @GFT_ID = GFT_ID
                    FROM
                            DBO.T_CHEQUE_CHQ
                    WHERE
                            CHQ_TYPE = @CHQ_TYPE
                        AND    CHQ_YEAR = @CHQ_YEAR
                        AND    CHQ_SERIAL = @CHQ_SERIAL
            END
END
 
GO

J'ai beau retourné dans tout les sens, je ne vois pas...
Au niveau des paramètres, je passe exactement le même type de variable. Le problème n'est donc pas là.

J'ai pensé au découpage du barcode mais pareil, un tinyint allant de 0 à 255, en prenant 2 digit du barcode, j'ai de la marge...

Le message d'erreur parle de CHQ_AMOUNT, j'ai donc regardé de ce côté mais tous les casts que je fais pour traiter la valeur que je mets dans cette colonne semblent fonctionner...

Please help

**aieeeuuuuu** · 21/01/2013, 12h40

Envoyé par Kropernic

Personne n'aurait une petite idée de où et cette procédure pourrait générer un arithmetic overflow ?

peut être ici ?

Code :

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

 SET @ORIGIN = @ORIGIN * 10000

quel est le type de votre colonne ORIGIN ?
et que donne ceci :

Code :

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

1
2
3
4
5
6
7
 
SELECT
        CAST(CAST(ORIGIN AS INT) * 10000 AS INT)
FROM
        DBO.T_CHEQUE_ACHAT_HISTORY
WHERE
        ACTION = 'T'

J'ai pensé au découpage du barcode mais pareil, un tinyint allant de 0 à 255, en prenant 2 digit du barcode, j'ai de la marge...

sauf si votre barcode contient un moins !

**Kropernic** · 21/01/2013, 13h58

Envoyé par aieeeuuuuu

peut être ici ?

Code :

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

 SET @ORIGIN = @ORIGIN * 10000

quel est le type de votre colonne ORIGIN ?

La colonne est de type INT. J'y ai pensé bien sûr. Mais pour les cas qui sont traités par ce step, le nombre dans origin est TOUJOURS au maximum sur 2 digits.
Pour info, voici la ligne une ligne qui provoque l'erreur :

SeqNrHist = 1070031
Barcode = 228880082806
DatTim = 2007-08-04 17:13:00.000
Action = T
Origin =17
DestType = 3
Dest = 410067
Value = 2500
SaleType = 0
UserType = 0

(edit : mise en relation des colonnes et de leur valeur)

Envoyé par aieeeuuuuu

et que donne ceci :

Code :

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

1
2
3
4
5
6
7
 
SELECT
        CAST(CAST(ORIGIN AS INT) * 10000 AS INT)
FROM
        DBO.T_CHEQUE_ACHAT_HISTORY
WHERE
        ACTION = 'T'

Et donc ceci ne donne bien sûr aucune erreur vu que l'entier est toujours inférieur à 100. Donc même en multipliant par 10000, on a est encore loin des 2 milliards ^^.

Envoyé par aieeeuuuuu

sauf si votre barcode contient un moins !

Ca je n'y avait pas pensé... C'est vrai que, du point de vue du type de donnée, rien ne l'interdit. Heureusement, ce n'est jamais le cas. Dans l'exemple que j'ai donné plus haut, il y a un barcode.
Il s'agit d'un EAN13 sans le check digit. Donc pas de signe dans le barcode.

Toujours avec la ligne donnée plus haut, on voit que je mets dans des tinyint les valeurs 88 et 8. C'est donc bien dans la plage de donnée de ce type.

Pareil pour la valeur ou je caste 2500 en decimal(6,2) et que je divise par 100 après.

... divisé un décimal par un entier... Pas top ça maintenant que j'y pense...

Pourtant, si je fais un exec de la procédure stockée en passant le barcode et la valeur, elle ne donne aucune erreur... Y aurait-il une différence de tolérance entre un job et un script exécuté dans SSMS ?

**Rei Ichido** · 22/01/2013, 10h37

Est-il normal que d'un côté vous fassiez :

Code :

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

1
2
 
 CAST(@CHQ_TYPE AS DECIMAL(6,2)));

et de l'autre :

Code :

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

1
2
3
 
DECLARE @CHQ_AMOUNT DECIMAL(6,2);
                    SET @CHQ_AMOUNT = CAST(@VALUE AS DECIMAL(6,2)) / 100;

?

**Kropernic** · 22/01/2013, 10h54

Malheureusement oui.

Cette ancienne DB ne respecte même pas la 1NF... En fait, c'est même pire que ça. Si je ne me trompe pas, la 1NF dit que chaque colonne doit être atomique. Ici, en plus de ne pas être atomique, une colonne contient dans certains cas un numéro de caisse et un numéro de ticket, et dans d'autre, il contient une date au format YYMMDD. Bref, la joie quoi !

Alors oui, il y a une colonne VALUE mais elle n'est pas toujours remplie... (parfois à 0). Alors pour les gifts ou il est possible de déterminer la valeur via leur barcode (cas pour les gift-cheques commençant par '21'), j'utilise ce procédé pour ne pas devoir tester si la colonne VALUE est à 0 ou non.

Pour les gift-cheques commençant par '22', comme on ne peut déterminer leur valeur via le barcode, ils ont quand même eu le bon goût de toujours mettre quelque chose dans la colonne VALUE.

Ce qui explique que, dans certains cas, je fais un cast de CHQ_TYPE et dans d'autre, j'utilise la donnée passée en paramètre pour insérer une valeur dans la colonne CHQ_AMOUNT.

**aieeeuuuuu** · 22/01/2013, 11h41

Envoyé par Kropernic

Pire ! Si je modifie le code du step pour qu'il reprenne à la ligne qui a planté, il continue le traitement avec succès pour quelques milliers de lignes et replante avec la même erreur et toujours apparemment sans raison... Du moins, je ne vois pas...

Êtes-vous sûr qu'il reprend à la ligne qui a planté ?

Peut être pouvez vous placer votre code (le script principal) dans SSMS, l'entourer d'un try catch avec un point d’arrêt dans le catch... on pourra peut être mieux localiser l'erreur

Une autre piste : ça plante pendant le fetch. Dans ce cas, certaines variables auront les valeurs de la ligne qui plante, les autres (située après la variable posant problème, ainsi qu'elle même) auront les valeurs de la ligne précédente, ça pourrait perturber le traitement, ainsi que l'analyse du bug. Pouvez vous nous poster le DDL de la table T_CHEQUE_ACHAT_HISTORY

**Kropernic** · 22/01/2013, 11h56

Envoyé par aieeeuuuuu

Êtes-vous sûr qu'il reprend à la ligne qui a planté ?

Oui oui, en fait, j'ajoute un filtre à la clause where du step sur la colonne SEQNRHIST dont j'ai le dernier numéro traité dans la table T_CURRENT_LINE_PROCESSING que j'ai mentionné plutôt dans laquelle je stocke normalement la ligne en cours de traitement. Mais après vérification, j'ai remarqué que la ligne marqué dans cette table était traitée. J'en ai donc conclu que le bug se situait dans la procédure stockée se trouvant juste avant (j'aurais bien sûr dû placer le stockage de la ligne traitée en premier lieu, c'est une erreur de ma part mais pour le coup, c'est plutôt pratique). En ajoutant donc un filtre sur SEQNRHIST, je suis sûr qu'il reprend à la bonne ligne.

Envoyé par aieeeuuuuu

Peut être pouvez vous placer votre code (le script principal) dans SSMS, l'entourer d'un try catch avec un point d’arrêt dans le catch... on pourra peut être mieux localiser l'erreur

Je peux essayer mais je ne suis pas à l'aise avec les try-catch en SQL SERVER... Je tenterai ça... (va falloir que j'aille vérifier la syntaxe).

Envoyé par aieeeuuuuu

Une autre piste : ça plante pendant le fetch. Dans ce cas, certaines variables auront les valeurs de la ligne qui plante, les autres (située après la variable posant problème, ainsi qu'elle même) auront les valeurs de la ligne précédente, ça pourrait perturber le traitement, ainsi que l'analyse du bug.

Ah, je n'avais pas songé à cela. Mais je ne pense pas que cela vienne de la.
J'ai déjà fait le teste de copier le code dans SSMS en gardant le curseur mais en ne sélectionnant que la ligne fautive pour voir si le traitement passait et c'était concluant...

Envoyé par aieeeuuuuu

Pouvez vous nous poster le DDL de la table T_CHEQUE_ACHAT_HISTORY

Voici le DDL de la table T_CHEQUE_ACHAT_HISTORY:

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
27
28
29
30
USE [GIFT_MANAGEMENT]
GO
 
/****** Object:  Table [dbo].[T_CHEQUE_ACHAT_HISTORY]    Script Date: 01/22/2013 10:45:48 ******/
SET ANSI_NULLS ON
GO
 
SET QUOTED_IDENTIFIER ON
GO
 
SET ANSI_PADDING ON
GO
 
CREATE TABLE [dbo].[T_CHEQUE_ACHAT_HISTORY](
    [SeqNrHist] [int] NOT NULL,
    [Barcode] [varchar](13) NOT NULL,
    [DatTim] [datetime] NOT NULL,
    [Action] [varchar](1) NOT NULL,
    [Origin] [int] NULL,
    [DestType] [int] NULL,
    [Dest] [int] NULL,
    [Value] [int] NULL,
    [SaleType] [int] NULL,
    [UserType] [int] NULL
) ON [PRIMARY]
 
GO
 
SET ANSI_PADDING OFF
GO

Magnifique table n'est-ce pas ? Aucune contrainte d'aucune sorte... La porte ouverte à tout et n'importe

J'ai même trouvé des gifts qui subissent 2 transactions au même instant... (heureusement, il s'agissait de "vrais" doublons et j'ai supprimé cette cochonnerie)

Quand je pense que ça a été fait par des ingénieurs... C'est triste quand même...

**aieeeuuuuu** · 22/01/2013, 12h49

Envoyé par Kropernic

Pour les gift-cheques commençant par '22', comme on ne peut déterminer leur valeur via le barcode, ils ont quand même eu le bon goût de toujours mettre quelque chose dans la colonne VALUE.

Êtes vous certains qu'il y a bien quelque chose, et que ce quelque chose n'est jamais supérieur = 9999, sinon le CAST(VALUE AS DECIMAL(6,2)) ne passera pas

ça mérite vérification :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT *
FROM   T_CHEQUE_ACHAT_HISTORY
WHERE  barcode like '22%' 
AND    value > 9999
 
--
SELECT *
FROM   T_CHEQUE_ACHAT_HISTORY
WHERE  barcode like '22%' 
AND    CAST(VALUE AS DECIMAL(6,2)) / 100 IS NULL

**Kropernic** · 22/01/2013, 13h04

Vous avez en effet mis le doigt sur un bug.

J'ai, pour les gift 22, des valeurs supérieur à 9999. Par exemple 12900. Ce qui donne en réalité 129€ (c'est toujours exprimé en centimes). Il faut donc je fasse la division par 100 avant le cast.

Je vais bien sûr corriger cela.

Il y a à parier que ce soit LE bug que je cherche. Quel bug à la c**...

Cependant, je m'interroge... Quand je fais les vérifications et que je recherche quelle ligne à planté, à moins bien sûr que je me trompe et que je regarde la mauvaise ligne (c'est toujours possible), j'ai une valeur de 2500 (soit 25€) et pour celle-là, le cast ne devrait pas poser de problème.

Bref, trève de bavardage, je corrige, je vide les tables et je recommence ce step à 0.

On verra bien si ça plante encore !

Quoi qu'il en soit, déjà un grand merci pour tout ce temps que vous m'avez accordé. Je crois que sans vous, jamais je n'aurais vu cette erreur de cast sur laquelle j'avais pourtant le nez dessus depuis le début.

**Kropernic** · 23/01/2013, 10h13

Bon bin d'après le job activity monitor, le job est passé au step suivant.

Le problème venait donc bien de ce cast hasardeux.

Je vous remercie grandement pour votre aide et passe cette discussion en résolue.

Je continue cependant de m'interroger sur le contenu de la table où la ligne en cours est sensé être contenue...

**aieeeuuuuu** · 23/01/2013, 10h38

bonjour,

comme vous le disiez, la mise à jour de cette table a lieu après le plantage. Les données qu'elle contient sont donc celles de la ligne précédente...

**Kropernic** · 23/01/2013, 10h45

Oui oui, ça j'avais bien compris que c'était la ligne précédente.

Mais en faisant la même requête que celle du curseur, requête qui contient bien sûr une clause ORDER BY, et en prenant la ligne d'après celle qui stockée, je n'obtenais pas une ligne obtenant une valeur provoquant l'erreur.

D'où le fait que je cherchais le bug dans le vent car mes tests ne provoquaient jamais l'erreur...

**aieeeuuuuu** · 23/01/2013, 11h03

bonjour,

N'avez-vous pas de doublons sur le couple (SeqNrHist, DatTim) ?

Code :

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

1
2
3
4
5
 
SELECT   SeqNrHist, DatTim
FROM     T_CHEQUE_ACHAT_HISTORY
GROUP BY SeqNrHist, DatTim
HAVING   COUNT(*) > 1

ça pourrait expliquer...

**Kropernic** · 23/01/2013, 11h10

Oui oui, des doublons il y a.

J'y avais pensé mais "par chance", la ligne stockée dans la table de log n'était pas de cas.

Pour l'info, les doublons viennent du fait que cette table T_CHEQUE_ACHAT_HISTORY est produite à partir d'une autre table où les actions se font sur des plages de gifts plutôt que sur un gift à la fois.
Du coup, en "découplant" ces plages de gifts, on obtient plusieurs lignes jumelles avec juste la valeur de la colonne barcode qui varie.

[2008 R2] Arithmetic overflow

Développement SQL Server

Discussions similaires

Partager

Partager