C# EF Doesn't save multiple records using ForEach

Question

Good day guys,

I'm trying to save a multiple records.

I've done my research but none of them actually works on my end.

Here's my code:

var dataToUpdate = context.Employees.ToList(); 

using(var db = context){
   var department= "IT";

   dataToUpdate.ForEach(x=>{
       x.department = department;
   });

   db.SaveChanges()
}

Source here: EF - Update multiple rows in database without using foreach loop but it doesn't work.

I tried to put breakpoints and try catch and my breakpoint goes to db.SaveChanges without error. But when I check the data from db, no changes have occurred.

It is clear that dataToUpdate did not come from this db. So it is not tracked here. Your question is far from complete. — Henk Holterman
– Henk Holterman, Commented Feb 12, 2019 at 7:51
It has to be about [entity-framework-6] or [entity-framework-core], it can't be about both. — Henk Holterman
– Henk Holterman, Commented Feb 12, 2019 at 8:01

TanvirArjel · Accepted Answer · 2019-02-12 08:22:39Z

2

Look at! you are pulling and updating data from two different contexts. Actually it should be from same context as follows:

using(var db = context)
{
  var dataToUpdate = db.Employees.ToList();

   var department= "IT";

   dataToUpdate.ForEach(x=>{
       x.department = department;
   });

   db.SaveChanges();
}

Now it should work.

edited Feb 12, 2019 at 8:22

answered Feb 12, 2019 at 8:10

TanvirArjel

32.5k14 gold badges89 silver badges120 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

BytesOfMetal Over a year ago

Typical missing semicolon after db.SaveChanges() :)

TanvirArjel Over a year ago

@Petaflop Oh! Thanks.

Collectives™ on Stack Overflow

C# EF Doesn't save multiple records using ForEach

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related