Discussion :

[Guigsounet]

Bonjour j'ai deux soucis avec une procédure stockée dont j'ai hérité
Je me permet de prévenir que je ne suis pas dev donc les questions que je vais poser ont peut-être des réponses évidentes

La procédure stockée ressemble à ceci :

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
begin procedure XXX @id varchar(50)
as 
begin
insert into table_de_log (datemaj) values (@id)
begin try
begin tran
 
truncate T1
truncate T2
truncate ...
... T20
 
insert into T1 select * from TT1
insert into T2 select * from TT2
insert into ...
... from TT20
 
commit tran
update table_de_log  set isactive=0,Status=0 where datemaj=@id
end try
begin catch
update table_de_log set isactive=0,errmsg=convert(vachar(50),error_number())+'-'+convert(varchar(5),error_line())+'-'+error_message() where datemaj=@id
end catch;
end;

Mon premier pb : lors de la phase de tests nous avons simulé différentes erreurs et avons ajouté un insert sur une table inexistante : si la table n'existe pas la procédure se stop immédiatement, j'aimerais donc trouver un moyen de finir la procédure stockée quoiqu'il arrive et de voir ensuite la ou les erreurs

Second problème : par moment la procédure plante l'instance de BDD pourant le nombre de lignes traitées au total est des plus raisonnable, est-il possible et intelligent (si c'est possible) d'ajouter des commit après chaque insert ?

[SQLpro]

1) Vous ne devez pas placer de code entre le END TRY et le BEGIN CATCH

2) vous ne terminez pas un END CATCH

3) votre contrôle transactionnel est mal placé

4) il vous faut renvoyer un code d'erreur si passage dans le CATCH.

A +

[Guigsounet]

Merci de votre réponse
1- Je n'en vois pas
2 et 4- Si mais je ne l'ai pas recopié (du coup j'edit le premier post)
3-pourriez vous précisez en quoi il est mal placé ?

Qu'en est-il des commits à votre avis ?
Comment faire en sorte qu'il aille au bout de la procédure même si il y a une erreur dans un des 20 inserts ?

[SQLpro]

Voici quel est le squelette de code correct :

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
CREATE procedure P ...
AS
 
-- code non transactionné
 
BEGIN TRANSACTION
 
-- code transactionné
 
BEGIN TRY
 
-- code vérifié
 
 
   COMMIT --> finalisation de la transaction par validation
 
END TRY
--> AUCUN code ne doit figurer END TRY et BEGIN CATCH
BEGIN CATCH
   IF XACT_STATE() <> 0 
      ROLLBACK --> finalisation de la transaction par annulation,
 
-- code de gestion d'erreur
 
   RAISERROR('Traitement transactionnel annulé dans la procédure P.', 16, 1); --> renvoi d'un code d'erreur
 
-- code non transactionné suite à erreur
 
END CATCH
 
-- code non transactionné final
 
GO

A +

[Guigsounet]

J ai suivi vos conseils et ai crée 2 procédures stockées

une avec les truncate insert
et la seconde qui lance la premiere avec le try/catch

C'est effectivement mieux, ma procédure ne se stop pas en plein vol mais ne va pas après une eventuelle erreur

y a til un moyen cependant de commiter les insert les uns après les autres ?

[SQLpro]

J ai suivi vos conseils et ai crée 2 procédures stockées

une avec les truncate insert
et la seconde qui lance la premiere avec le try/catch

C'est effectivement mieux, ma procédure ne se stop pas en plein vol mais ne va pas après une eventuelle erreur

y a til un moyen cependant de commiter les insert les uns après les autres ?

Vous n'êtes pas cohérent, Si vous voulez une transaction, alors le COMMIT et le ROLLBACK doit se faire globalement.
Si vous voulez valider à chaque instruction SQL, alors il ne faut pas gérer de transaction explicite.

Bref, il y a quelque chose qui tourne pas rond dans votre tête !

Si au lieux de nous fournir un exemple théorique, incomplet et syntaxiquement incohérent vous nous fournissiez le vrai code ce serait peut être plus intéressant ne croyez vous pas ???

A +