Entity Framework. Как правильно использовать транзакции?

Question

[Егор Лаврентьев]

Коллеги, приветствую!
У меня в приложении есть примерно вот такой код

public static bool DoLargeOperation()
        {
            bool res = true;

            res = res && DoFirstSubOperation();
            res = res && DoSecondSubOperation();
            res = res && DoThirdSubOperation();

            return res;
        }

Каждая из внутренних функций имеет примерно такой вид:

public static bool DoFirstSubOperation()
        {
            using (var context = new EntityFrameworkContext())
            {
                // Изменение данных.
                context.SaveChanges();
            }
        }

Допустим, что первые две внутренних функции выполнились без ошибок, а вот третья вызвала исключение.
Каким образом мне отменить результат выполнения первых двух функций?

Вот такой подход не принёс результатов:

using (var transaction = new TransactionScope())
            {
                res = res && DoFirstSubOperation();
                res = res && DoSecondSubOperation();
                res = res && DoThirdSubOperation();
            }

Единственное решение, которое я вижу - это объявлять контекст в методе вот так:

public static bool DoLargeOperation()
        {
            bool res = true;

            using (var context = new EntityFrameworkContext())
            {
                using (var transaction = context.Database.BeginTransaction())
                {

                    res = res && DoFirstSubOperation(context);
                    res = res && DoSecondSubOperation(context);
                    res = res && DoThirdSubOperation(context);
                    if (res)
                    {
                        transaction.Commit();
                    }
                    else
                    {
                        transaction.Rollback();
                    }
                }
            }

            return res;
        }

Но допустим ли такой вариант, или есть какой-то другой путь?

Answer 1

[Сергей]

Да, при таких условиях это правильный вариант.
Если функции являются членами одного класса и их можно сделать не статическими, то можно сделать контекст также членом класса. Делать статическим контекст я бы не советовал.

Comments

[Егор Лаврентьев]

Спасибо за ответ. Да, контекст глобальным или статическим делать неправильно. Больше всего мне было интересно, допустимо ли передавать его в качестве параметра метода.

[Oxoron]

Сергей , а если создать контекст, передавать его в методы, и только после методов вызвать SaveChanges() (внутри методов не вызывать)? В этом случае можно сэкономить на быстродействии, за счет двух SaveChanges() и обработку транзакции.

[Сергей]

Oxoron: Да немного времени можно сэкономить т.к. каждый SaveChanges() ищет изменения в контексте. Отмечу, что EF не очень подходит для массовых операций и держать в нем много записей плохо, а при небольшом числе записей время затрачиваемое на поиск изменений очень небольшое.

[Oxoron]

Сергей, а нужна ли вообще транзакция, если у нас лишь один вызов SaveChanges()?
И что используют для массовых операций? NHibernate?

[Сергей]

Oxoron: Вызов SaveChanges() выполняется в рамках одной транзакции таким образом открываеть, закрывать или откатывать смысла не имеет, а вот например SaveChanges(false) не открывает транзакцию.

Внутренностей NHibernate я не знаю и сказать что происходит не могу. Можно подключить sql profiler и посмотреть, что он шлет.

Если простое обновление/удаление/вставка множества записей не стесняемся использовать SQLCommand (коннект можно получить из контекста или прочитать строку из конфига), для чего то более сложного пишем хранимую процедуру и вызываем её.