score:92

Accepted answer

You can't use Contains with non-primitive values. Do

Where(l => l.Courses.Select(c => c.CourseId).Contains(courseId)

(or the Id field you use).

score:0

This can also happen when you pass a Func<T, bool> to Where() as a way to write a dynamic condition like here here For some reason the delegate can't be translated to SQL.

score:0

You cannot compare complex type, if you have not specified what you mean for equality.

As exception detail says, you need to check primitive values (like Integer in your case).

And better to use Any() method instead.

var toRemove = db.Lecturers
     .Where(l => l.Courses.Any(p=>p.Id == courseFromDb.Id)).ToList();

score:1

The Courses collection of below line should be null or empty.

 var toRemove = db.Lecturers
                        .Where(l => l.Courses.Contains(courseFromDb)).ToList();

score:8

If you are using a DbContext, you can query the .Local collection, and the == operator will work also with objects:

public void RemoveCourse(int courseId)
{
    using (var db = new AcademicTimetableDbContext())
    {
        var courseFromDb = db.Courses.Find(courseId);

        db.Lecturers.Load() //this is optional, it may take some time in the first load

        //Add .Local to this line
        var toRemove = db.Lecturers.Local 
                        .Where(l => l.Courses.Contains(courseFromDb)).ToList();

        foreach (var lecturer in toRemove)
        {
            lecturer.Courses.Remove(courseFromDb);
        }

        db.SaveChanges();
    }
}

The .Local is an ObservableCollection, so you can compare anything you like inside it (not limited to SQL queries which don't support object comparison). Just to make sure you get all your objects in the .Local collection you can call the db.Lecturers.Load() method before calling .Local, which brings all database entries into the Local collection.


Related Query

More Query from same tag