Discussion :

[ikeas]

bonjour
je tente d'utiliser pour la première fois les transactions avec cet exemple

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
 using (DbContextTransaction transaction = Context.XXX.Database.BeginTransaction())
            {
                for (int i = 0; i < 10; i++)
                {
                    LogCompteur lc = new LogCompteur()
                    {
                        Compteur = 25,
                        Date = DateTime.Now,
                        Machine = 160,
                        Utilisateur = "m001" + "fgddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddd",
                        Equipe = "j",
                    };
                    Context.XXX.LogCompteur.Add(lc);
                }
                try
                {
                    Context.XXX.SaveChanges();
                    transaction.Commit();
                }
                catch
                {
                    transaction.Rollback();
                }
            }
             Context.XXX.SaveChanges();

je crée délibérément une erreur sur "utilisateur" pour tester le rollback
le problème c'est que mes objets ne sont manifestement pas enlevé de LogCompteur
ce qui fait qu'au Context.XXX.SaveChanges(); (le final en sortant de la transaction) ça plante

aurais je oublié quelque chose ?

[Momoth]

Bonjour,

Le rollback d'une transaction annule les actions entreprises dans celles ci. A savoir dans ton cas, aucune. Le add d'un item à la collection du DbSet va juste permettre à EF de tracker ton objet (et donc d’exécuter les actions adéquates lors du SaveChanges).

Dans ton cas, vu que tu pète sur le SaveChanges, ton rollback n'a aucune action à revert. La transaction ne gère pas les états des objets par EF.

[ikeas]

donc si j'ai bien compris
comme ca marche pas avec les objets
la solution serait de faire un remove de l'objet qu'on a ajouté
(c'est ce que je fais d'ailleurs)

ce qui m'a trompé c'est cet exemple
http://www.entityframeworktutorial.n...framework.aspx

donc si je reformule, les transactions ne fonctionnent pas avec les objets

[Momoth]

Désolé, je viens de voir que tu avais répondu ^^".

Hum je maitrise pas assez le bousin pour avoir une réponse précise la dessus. Après il faut voir en détail l'erreur sur le SaveChanges().

Je suis tombé sur un cas cette semaine (avec des transactions dans l'histoire), où le SaveChanges() a levé une exception parce qu'il n'y avait aucune action à exécuter.

Je pense donc que ton rollback à bien fait le taf et tout annulé et que ton SaveChanges() final pète parce qu'il n'a aucune action à exécuter.

Par contre, je me demande à quoi sert ce SaveChanges(). Vu que tu en fait un plus haut, s'il passe, tu n'aura rien à SaveChanger (oui j'invente des verbes ). Et si ton premier SaveChanges() plante, tu ne veux justement pas sauvegarder les données, vu que tu fais un rollback.

[ikeas]

j'ai compris ce que tu voulais dire
toutefois ce que je voulais montrer
c'est que le principe est le suivant
- j'ajoute 10 elements
- je tente le save change
- si ca marche je fais un commit (pas de soucis) dans ce cas la le savechanges en gras n'a aucun intérêt et peut éventuellement planter)
- si ca marche pas je fais un rollback, c'est la que j'ai un probleme car tous les elements ajouté a LogCompteur ne sont pas enlevé et il ne me sort pas une erreur sur rien a sauver mais la meme erreur que celle qui a conduit au rollback

[Momoth]

Je vois ce que tu veux dire.

Après, dans ce cas précis, si tu veux continuer de faire des actions après que la transaction ai échoué et sans être pollué par tes objets, après ton rallback tu peux disposer ton dbcontext et le réinstancier.

Juste faire des remove() sur les objets est assez contraignant car à chaque fois que tu vas modifier ta transaction (rajouter d'autres objets dans une autre table par exemple), tu devra modifier ton catch pour le prendre en compte.

Et ca sera plus simplement réutilisable dans d'autres endroits de l'appli.