Discussion :

[babacan]

Bonjour;
Ma procédure permet de faire un enregistrement ou une modification ou une supression selon le choix de l'utilisateur.Lorsque j'exécuter ma procédure dans mon AGL ,ma procédure me renvoye un message vide qui ne contient aucune indication.
Voici ma procédure

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
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
 
CREATE PROCEDURE [dbo].[PrGarantieAutomobile]
      @CodeCaracteristic int,
      @Zonecode varchar(10)=null,
      @CarosserieCode varchar(10)=null,
      @EnergieCode varchar(10)=null,
      @StatutCode varchar(10)=null,
      @NumeroImmatriculation varchar(15)=null,
      @DatePremiereImmatriculation datetime=null,
      @DateImmatriculation datetime=null,
      @ChargeUtile int=null,
      @PoidsVide int=null,
      @NombrePlace int=null,
      @PuissanceAdministrative int=null,
      @PuissanceReelle int=null,
      @GenreCode varchar(10)=null,
      @MarqueCode varchar(10)=null,
      @Serie varchar(20)=null,
      @ValeurVenale money=null,
      @ValeurNeuf money=null,
      @Types varchar(20)=null,
      @Conducteur varchar(30)=null,
      @TransportCode varchar(10)=null,
      @AjustementCode varchar(10)=null,
      @DateNaissanceConducteur datetime=null,
      @Datepermis datetime=null,
      @Numeropermis varchar(10)=null,
      @Profession varchar(40)=null,
      @CouleurCode varchar(10)=null,
      @ProtectionCode varchar(10)=null,
      @Assure varchar(60)=null,
      @Adresse varchar(60)=null,
      @Gage char(2)=null,
      @NumeroMoteur varchar(20)=null,
      @NumeroImmatPrec varchar(15)=null,
      @DateImmatPrec datetime=null,
      @TypeCommercial char(20)=null,
      @NombrePlaceHC int=null,
      @DureeConduite int=null,
      @CategorieCode varchar(10)=null,
      @SousCategorieCode varchar(10)=null,
        @NatureCode varchar(10)=null,
        @DepartementCode varchar(10)=null,
        @ClasseProfessionnelCode varchar(10)=null,
      @Rep int,
      @sms varchar(100) output
AS
 BEGIN
   begin transaction
    declare @erro int
    declare @er int
    declare @numCaract int
    begin
  set @er=(SELECT count(SaisieCode) FROM EnBufGarantieAuto where FlagGarantie=1)
    if @er = 0
      begin
       set @sms='Veuillez choisir des garanties'
       rollback
      end
    else
     begin /*Mise à jour de la table temporaire garantie auto*/
      update EnBufGarantieAuto
       set NumeroImmatriculation=@NumeroImmatriculation,
           CodeCategorie=@CategorieCode,
           CaracteristicAutoCode=@CodeCaracteristic
       if @@error=0
         begin
          set @sms='Maj EnBufGarantieAuto OK'
          commit
         end
       else
        begin
         set @sms='Maj echouée'
         rollback
        end
      /*enregistrement de la caracteristique auto dans la table temporaire*/
     if (@Rep=1)
        begin
          begin transaction
     set @erro=(SELECT CaracteristicAutoCode FROM EnCaracteristicAuto where    CaracteristicAutoCode=@CodeCaracteristic)
            if @erro!=0
             begin
               set @sms='Ce code existe déjà,entrez-en un autre'
               return
             end
            else
             begin
    set @numCaract=(SELECT count(CaracteristicAutoCode) FROM EnBufCaracteristicAuto)
                if @numCaract!=0
                   delete from EnBufCaracteristicAuto
                begin
                   begin transaction
INSERT INTO EnBufCaracteristicAuto ([CaracteristicAutoCode],[Zonecode],[CarosserieCode],[EnergieCode],[StatutCode],[NumeroImmatriculation],[DatePremiereImmatriculation],[DateImmatriculation],[ChargeUtile],[PoidsVide], [NombrePlace],[PuissanceAdministrative],[PuissanceReelle],[GenreCode],[MarqueCode],[Serie], [ValeurVenale],[ValeurNeuf],[Types],[Conducteur],[TransportCode],[AjustementCode],[DateNaissanceConducteur] ,[Datepermis],[Numeropermis],[Profession],[CouleurCode],[ProtectionCode], [Assure],[Adresse] ,[GageCode],[NumeroMoteur],[NumeroImmatPrec],[DateImmatPrec],[TypeCommercial], [NombrePlaceHC] ,[DureeConduite],[CategorieCode],[SousCategorieCode],[NatureCode], [DepartementCode],[ClasseProfessionnelCode])
   VALUES (@CodeCaracteristic,@Zonecode,@CarosserieCode,@EnergieCode,@StatutCode,@NumeroImmatriculation  ,@DatePremiereImmatriculation,@DateImmatriculation,@ChargeUtile,@PoidsVide, @NombrePlace                      ,@PuissanceAdministrative,@PuissanceReelle,@GenreCode,@MarqueCode,@Serie, @ValeurVenale  ,@ValeurNeuf,@Types,@Conducteur,@TransportCode,@AjustementCode, @DateNaissanceConducteur ,@Datepermis,@Numeropermis,@Profession,@CouleurCode,@ProtectionCode,@Assure, @Adresse,@Gage ,@NumeroMoteur,@NumeroImmatPrec,@DateImmatPrec,@TypeCommercial,@NombrePlaceHC,@CategorieCode,@SousCategorieCode,@NatureCode,@DepartementCode, @ClasseProfessionnelCode)
 if @@error=0
    begin
     set @sms=(select DescMessage From EnMessage Where CodeMessage=3)
     commit transaction
    end
 else
     begin
      set @sms='Enregistrement non effectué'
      return
     end
      end
     end
      commit tran
    end
 
--------------------------------------------------------------------------
 if (@Rep=2)
  begin
   begin transaction
           UPDATE [EnBufCaracteristicAuto]
           SET [Zonecode] = @Zonecode
              ,[CarosserieCode] = @CarosserieCode
              ,[EnergieCode] = @EnergieCode
              ,[StatutCode] = @StatutCode
              ,[NumeroImmatriculation] = @NumeroImmatriculation
              ,[DatePremiereImmatriculation] = @DatePremiereImmatriculation
              ,[DateImmatriculation] = @DateImmatriculation
              ,[ChargeUtile] = @ChargeUtile
              ,[PoidsVide] = @PoidsVide
              ,[NombrePlace] = @NombrePlace
              ,[PuissanceAdministrative] = @PuissanceAdministrative
              ,[PuissanceReelle] = @PuissanceReelle
              ,[GenreCode] = @GenreCode
              ,[MarqueCode] = @MarqueCode
              ,[Serie] = @Serie
              ,[ValeurVenale] = @ValeurVenale
              ,[ValeurNeuf] = @ValeurNeuf
              ,[Types] = @Types
              ,[Conducteur] = @Conducteur
              ,[TransportCode] = @TransportCode
              ,[AjustementCode] = @AjustementCode
              ,[DateNaissanceConducteur] = @DateNaissanceConducteur
              ,[Datepermis] = @Datepermis
              ,[Numeropermis] = @Numeropermis
              ,[Profession] = @Profession
              ,[CouleurCode] = @CouleurCode
              ,[ProtectionCode] = @ProtectionCode
              ,[Assure] = @Assure
              ,[Adresse] = @Adresse
              ,[GageCode] = @Gage
              ,[NumeroMoteur] = @NumeroMoteur
              ,[NumeroImmatPrec] = @NumeroImmatPrec
              ,[DateImmatPrec] = @DateImmatPrec
              ,[TypeCommercial] = @TypeCommercial
              ,[NombrePlaceHC] = @NombrePlaceHC
              ,[DureeConduite] = @DureeConduite
                  ,[CategorieCode] = @CategorieCode
                  ,[SousCategorieCode] = @SousCategorieCode
                  ,[NatureCode]=@NatureCode
                  ,[DepartementCode]=@DepartementCode
                  ,[ClasseProfessionnelCode]= @ClasseProfessionnelCode
 WHERE [CaracteristicAutoCode] = @CodeCaracteristic
if @@error=0
  begin                          
    set @sms=(select DescMessage From EnMessage Where CodeMessage=4)
    commit transaction
    end
  else
    begin
      set @sms='Mise à jour non effectuée'
      rollback transaction
    end
       end
     if (@Rep=3)
       begin
        begin transaction
          DELETE FROM EnBufCaracteristicAuto
          WHERE CaracteristicAutoCode = @CodeCaracteristic
          begin
           if @@error = 0
             begin
               set @sms=(select DescMessage From EnMessage Where CodeMessage=5)
               commit transaction
             end
           else
             begin
               set @sms='Suppression non effectuée'
               rollback transaction
             end 
         end  
        end
     end
  end
 commit transaction
end

j'ai effectuer plusieurs test afin de savoir pourquoi il ne fonctionne pas comme prévu mais rien. Selon moi elle doit pouvoir bien fonctionner
Est ce que quelqu'un peut m'aider afin qu'elle puisse marcher convenablement ?
Merci d'avance

[SQLpro]

Mauvais :

SET @er=(SELECT count(SaisieCode) FROM EnBufGarantieAuto WHERE FlagGarantie=1)

Bon :

Code :

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

  SELECT @er=count(SaisieCode) FROM EnBufGarantieAuto WHERE FlagGarantie=1

D'autre part avec vos tests imbriqués vous ne pouvez pas retrouver le fil transactionnel.C'est pourquoi on utilise GOTO pour la gestion des exception conduisant à un ROLBALBACK ou bien encore un bloc TRY/CATCH.

A lire :
1) avec GOTO : http://sqlpro.developpez.com/cours/s...nsactsql/#L4.6
2) avec TRY CATCH : http://sqlpro.developpez.com/sqlserv...ge=tsql#LIII-D

A +

[Zabriskir]

Bonjour,

J'en profite pour rebondir sur ce que vous dites SQL Pro concernant la différence entre SET et SELECT :

Dans la documentation SQL Server, on trouve :
"Pour affecter les variables, il est recommandé d'utiliser SET @local_variable à la place de SELECT @local_variable".

On trouve aussi des informations intéressantes dans cet article.
Notamment :

When using a query to populate a variable, SET will fail with an error, if the query returns more than one value. But SELECT will assign one of the returned rows and mask the fact that the query returned more than one row. As a result, bugs in your code could go unnoticed with SELECT, and this type of bugs are hard to track down too.

Jusqu'à maintenant, j'utilise donc SET dans l'affectation de variables.
J'aurais donc dit que :

Code :

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

SET @er=(SELECT count(SaisieCode) FROM EnBufGarantieAuto WHERE FlagGarantie=1)

n'est pas si mal que ça.

Qu'en pensez-vous ?

Zabriskir

[agemis31]

Bonsoir,

When using a query to populate a variable, SET will fail with an error, if the query returns more than one value. But SELECT will assign one of the returned rows and mask the fact that the query returned more than one row. As a result, bugs in your code could go unnoticed with SELECT, and this type of bugs are hard to track down too.

Le SET est injustifié ici car ca ne risque pas d'arriver avec COUNT()

@+

[babacan]

bonjour,
merci pour votre aide,étant donnée que le tous me fatigue j'ai essai de diviser mes tâches(traitements) et cela marche à merveille.
mais je continue toujours de chercher car il me semble cela devrait être un problème de structure
Merci