Discussion :

[Shadam]

Bonjour tout le monde,

Je suis en train de développer un programme (étonnant non ? ) et j'ai fait une procédure qui fait elle même appel à d'autres procédures pour faire des insert en "rafale" surr plusieurs tables.

Extrait :

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
Dim myTransac As SqlTransaction
        myTransac = cnx.BeginTransaction
        cmd.Transaction = myTransac
 
        Try
            cmd.ExecuteNonQuery()
 
            'Si l'insertion du nouveau rapport est effectuée avec succès, on insère tout les lots
            LigneLot.insert(insertReussi, newRapport.ListeLot, newRapport.Id, cnx, myTransac)
 
            'Si les lots sont insérés avec succès, on insère les pertes
            EntetePerte.insert(insertReussi, newRapport.ListePerte, newRapport.Id, cnx, myTransac)
 
            'Si les pertes sont insérées avec succès, on commit la transaction, sinon on rollback
            myTransac.Commit()
        Catch
            insertReussi = False
            myTransac.Rollback()
        End Try
        cnx.Close()

Donc j'envoie aux procédures "filles" les paramètres dont elles ont besoins (connexion et transaction comprises) avec des try sur toutes les procédures impliquées.
Donc ma question est : Si une des ces procédures lève une exception, est ce que je vais bien revenir dans le try/catch d'origine ? Et donc est ce que le commit/rollback est utilisé à bon escient ici ?

[sinople]

Ca me semble correcte, on peut faire un poil plus propre:

- Utilisation d'un Using pour encapsuler la connexion (on est sur que la connexion est close à la sortie de l'using)

- Test de la transaction is not nothing avant le rollback pour éviter une erreur dans la gestion d'erreur

Après je reste dubitatif sur le insertReussi = False. Il me semble plus logique de continuer la propagation de l'Exception plus loin pour indiquer qu'il y a eu un problème en mettant un simple Throw à la fin du catch.

[Gurdil le nain]

Pour répondre à ta question, oui ton excpetion remontera normalement jusqu'au premier bloc try/catch qu'elle trouvera.

[Shadam]

Après je reste dubitatif sur le insertReussi = False. Il me semble plus logique de continuer la propagation de l'Exception plus loin pour indiquer qu'il y a eu un problème en mettant un simple Throw à la fin du catch.

Oui c'est vrai, mais insertReussi est une sale habitude que j'ai prise pour faire des tests sur l'échec ou le succès de mes requêtes:
- True --> requête réussie
- False --> requête foirée

après je mets des msgBox en fonction...

Sinon on est donc bien d'accord que mon rollback se propagera bien sur toutes les tables impactées par la transaction ?
J'insiste parce que c'est la première fois que j'utilise une transaction comme cela, d'habitude je m'en sers plutôt qu'en j'ai une requête à l'intérieur d'un For Each ou quelque chose comme ça.

[Pol63]

Oui c'est vrai, mais insertReussi est une sale habitude que j'ai prise pour faire des tests sur l'échec ou le succès de mes requêtes:
- True --> requête réussie
- False --> requête foirée

après je mets des msgBox en fonction...

personnellement je suis pas contre, la couche d'ui n'as pas besoin de détail de l'erreur, elle a juste besoin de savoir si c'est ok et éventuellement un message d'erreur à afficher
donc faire une fonction booléen n'est pas si affreux (nous on fait fct(byref msg as string) as boolean
ce qui est plus important qu'un throw dans un catch par contre c'est un log de l'erreur (ex.gettype.fullname, ex.message, ex.stacktrace), ca te permet de débugger ton programme rapidement après

Sinon on est donc bien d'accord que mon rollback se propagera bien sur toutes les tables impactées par la transaction ?
J'insiste parce que c'est la première fois que j'utilise une transaction comme cela, d'habitude je m'en sers plutôt qu'en j'ai une requête à l'intérieur d'un For Each ou quelque chose comme ça.

oui mais tu peux aussi faire des tests quand tu n'es pas sur de quelque chose
dans un projet de test avec à la même architecture d'appels de méthode tu peux faire une requete qui create table, puis une requete qui raiseerror 'msg',16,1 et voir si la table existe après l'exécution des 2

[Shadam]

Ok merci à tous pour votre aide

J'essaierai ton truc lundi Pol, pour l'insatnt c'est le week end