EF 4.1 и "Коллекция была изменена, операция перечисления не может выполняться". исключение

Это заставляло меня гать в течение последних 2 дней. У меня есть 3 довольно простых класса (ну, для удобства чтения)

public class Employee 
{
    public string Name { set; get; }
    virtual public Employer Employer { set; get; }

    public Employee(string name)
    {
        this.Name = name;
    }
}

,

// this basically ties Employee and his role in a company.
public class EmployeeRole{
    public int Id { set; get; }
    virtual public Employee Employee { set; get; }
    public string Role { set; get; }

    public EmployeeRole(Employee employee, string role){
        this.Employee = employee;
        this.Role = role;
    }
}

и

public class Employer{

    public string Name { set; get; }
    List<EmployeeRole> employees = new List<EmployeeRole>();
    virtual public List<EmployeeRole> Employees { get { return this.employees; } }

    public Employer(string name, Employee creator){
        this.Name = name;
        this.Employees.Add(new EmployeeRole(creator, "Creator"));
        creator.Employer = this;
    }
}

Кажется довольно простым. Не выполнять какую-либо конкретную конфигурацию для этих классов в DbContext.

Но когда я запускаю следующий код

using (DbContext db = DbContext.GetNewDbContext()){

    Employee creator = new Employee("Bob");
    db.Employees.Add(creator);
    db.SaveChanges();

    Employer employer = new Employer("employer", creator);
    db.Employers.Add(employer);
    db.SaveChanges();
   // I know I can call SaveChanges once (and it actually works in this case), 
   // but I want to make sure this would work with saved entities.
}

Он выдает следующее исключение:

Коллекция была изменена; операция перечисления не может выполняться.

Трассировка стека:

в System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource ресурс) в System.Collections.Generic.List 1.Enumerator.MoveNextRare() at System.Collections.Generic.List 1.Enumerator.MoveNext() at System.Data.Objects.ObjectStateManager.PerformAdd(IList 1 entries)
at System.Data.Objects.ObjectStateManager.AlignChangesInRelationships(IList
1 записи) в System.Data.Objects.ObjectStateManager.DetectChanges() в System.Data.Objects.ObjectContext.DetectChanges() at System.Data.Entity.Internal.InternalContext.DetectChanges(Boolean силы) при System.Data.Entity.Internal.Linq.InternalSet 1.ActOnSet(Action action, EntityState newState, Object entity, String methodName) at System.Data.Entity.Internal.Linq.InternalSet 1.Add(объект объекта)
в System.Data.Entity.DbSet`1.Add(объект TEntity)

У кого-то есть идея, что происходит, и, может быть, как это исправить? Спасибо!

Ответы

Ответ 1

У меня была та же проблема. Это похоже на ошибку в EF.

Я изменил свой код, чтобы явно добавить объект в контекст EF, прежде чем устанавливать его в любое другое свойство.

Ответ 2

Для меня это выглядит как ошибка в Entity Framework. Я подготовил ваш пример к более простой, но с той же структурой:

public class TestA // corresponds to your Employee
{
    public int Id { get; set; }
    public TestB TestB { get; set; } // your Employer
}

public class TestB // your Employer
{
    public TestB()
    {
        TestCs = new List<TestC>();
    }

    public int Id { get; set; }
    public ICollection<TestC> TestCs { get; set; } // your EmployeeRoles
}

public class TestC // your EmployeeRole
{
    public int Id { get; set; }
    public TestA TestA { get; set; } // your Employee
}

Это три объекта с циклическими отношениями:

TestA -> TestB -> TestC -> TestA

Если я использую теперь соответствующий код с той же структурой, что и ваш, я получаю одно и то же исключение:

var testA = new TestA();
var testB = new TestB();
var testC = new TestC();

context.TestAs.Add(testA);

testA.TestB = testB;
testB.TestCs.Add(testC);
testC.TestA = testA;

context.ChangeTracker.DetectChanges();

Обратите внимание, что я использовал DetectChanges вместо SaveChanges, потому что трассировка стека в исключении делает ясно, что на самом деле DetectChanges вызывает исключение (которое внутренне называется SaveChanges). Я также обнаружил, что вызов SaveChanges дважды не является проблемой. Проблема здесь - это только "раннее" добавление в контекст до завершения всего графика объекта.

Коллекция, которая была изменена (как исключение жалуется), представляет собой не коллекцию TestB.TestCs в модели. Кажется, это набор записей в ObjectStateManager. Я мог бы проверить это, заменив ICollection<TestC> TestCs на одну ссылку TestC TestC в классе TestB. Таким образом, модель вообще не содержит какой-либо коллекции, но она по-прежнему вызывает одно и то же исключение в отношении модифицированной коллекции. (SaveChanges будет работать с тремя одиночными ссылками, потому что EF не знает, в каком порядке сохранить объекты из-за цикла. Но это еще одна проблема.)

Я бы счел это ошибкой, что обнаружение изменения EF (DetectChanges), по-видимому, изменяет собственную внутреннюю коллекцию, которую она просто выполняет.

Теперь исправить эту проблему легко: просто Add сущности в контексте в качестве последнего шага перед вызовом SaveChanges:

var testA = new TestA();
var testB = new TestB();
var testC = new TestC();

testA.TestB = testB;
testB.TestCs.Add(testC);
testC.TestA = testA;

context.TestAs.Add(testA);

context.ChangeTracker.DetectChanges();

EF добавит весь связанный объектный граф в контекст. Этот код преуспевает (также используя SaveChanges вместо DetectChanges).

Или ваш пример:

using (DbContext db = DbContext.GetNewDbContext()){
    Employee creator = new Employee("Bob");
    Employer employer = new Employer("employer", creator);

    db.Employees.Add(creator);
    db.SaveChanges();
}

Edit

Это было то же исключение: Черты Entity Framework "Коллекция была изменена" при использовании наблюдаемой коллекции. После кода в этом примере ситуация была схожа: добавление объекта в контекст, а затем изменение/добавление отношений к этому объекту.

Edit2

Интересно, что это вызывает одно и то же исключение:

var testA = context.TestAs.Find(1); // assuming there is already one in the DB
var testB = new TestB();
var testC = new TestC();

testA.TestB = testB;
testB.TestCs.Add(testC);
testC.TestA = testA;

context.SaveChanges(); // or DetectChanges, it doesn't matter

Итак, я хочу добавить отношения с новыми объектами к существующей сущности. Исправление этой проблемы кажется менее очевидным.

Ответ 3

Недавно я столкнулся с этой проблемой. Я обнаружил, что EF ненавидит косвенные круговые ссылки. В вашем случае кажется. Работодатель должен владеть отношениями с сотрудником. Поэтому верните ссылку работодателю из класса сотрудника.

Ответ 4

Я столкнулся с тем же вопросом сегодня утром, в моем случае у меня была ассоциация "Сотрудник-менеджер", определенная с круговыми отношениями. Например:

public class Employee 
{
    public string Name { set; get; }
    virtual public Employee Manager { set; get; }

    public Employee()
    {

    }
}

Приложение сбой с ошибкой выше при настройке свойства Manager. В конце он оказался плохим внедрением метода GetHashCode() в классе Employee. Казалось, что EF не смог обнаружить модифицированный объект, поскольку сравнение между объектами не удавалось; таким образом, EF думал, что коллекция была изменена.