Ajout d'une collection, boucle nécessaire?

Question

J'ai ce code:

Schedule s = _entities.Schedules.Where(x => x.ScheduleID == schedule.ScheduleID && x.BookingObject.BookingObjectID == bookingObjectID).FirstOrDefault(); 

if(s == null) 
    s = new Schedule(); 

s.ScheduleStart = schedule.ScheduleStart; 
s.ScheduleEnd = schedule.ScheduleEnd; 

foreach (var t in schedule.Timetables) 
{ 
    s.Timetables.Add(t); 
} 

_entities.AddToSchedules(s); 

_entities.SaveChanges(); 

Exception: System.InvalidOperationException

le calendrier d'objet est transmis en tant que paramètre

EDIT:

changé:

Schedule sh = new Schedule(); 


    sh.ScheduleStart = schedule.ScheduleStart; 
    sh.ScheduleEnd = schedule.ScheduleEnd; 

    foreach (var t in schedule.Timetables) 
    { 
     //sh.Timetables.Add(t); // doesn't work 
     sh.Timetables.Add(new Timetable { DayOfWeek = t.DayOfWeek, StartTime = t.StartTime, EndTime = t.EndTime }); // works 
    } 

    _entities.AddToSchedules(sh); 

    _entities.SaveChanges(); 

est-il un peu logica l raison pour laquelle sh.Timetables.Add (t) ne fonctionne pas puisque c'est une collection de Timetable?

/M

Answer 1

MISE À JOUR: vient de confirmer que cela est susceptible d'être la raison. Et créé un nouveau tip en my tips series pour expliquer.

Je peux penser à une raison possible. Est la relation entre schedule.Timetables un à plusieurs? C'est à dire. Un horaire peut-il être affecté à un seul horaire à la fois?

Si oui, ce ne parvient pas:

foreach (var t in schedule.Timetables) {  
    sh.Timetables.Add(t); 
    } 

Parce que vous ajoutez le « t » à un calendrier différent, il est automatiquement retiré de l'ancienne collection schedule.Timetables, ce qui bien sûr modifie la collection que vous énumérez , qui est un non-non.

Si oui, la solution est tout simplement de faire quelque chose comme ceci:

var timetables = schedule.Timetables.ToArray(); 
    foreach (var t in timetables) {  
    sh.Timetables.Add(t); 
    } 

vous énumérez ici la collection, avant de commencer à le modifier.

Answer 2

s.Timetables.Add(schedule.Timetables) 

si ce n'est pas le cas, il est probable que votre question a besoin de plus d'informations. Qu'est-ce que s.Timetables, schedule.Timetables? (types)

Answer 3

foreach(var time in schedule.Timetables){ 
    s.Timetables.Add(time) 
} 

Vous pouvez boucler sur les tables de temps en schedule.Timetables et les ajouter à votre nouvelle entité.

Answer 4

Une autre option serait:

((List<TimeTable>)s.Timetables).AddRange(((List<TimeTable>)schedule.Timetables)); 

Answer 5

Regardez les premières lignes:

Schedule s = _entities.Schedules.Where(x => x.ScheduleID == schedule.ScheduleID && x.BookingObject.BookingObjectID == bookingObjectID).FirstOrDefault(); 

if(s == null) 
    s = new Schedule(); 

Sur la première ligne, vous interrogez la base de données pour un objet avec le même ScheduleId que schedule et stocker la référence renvoyée dans s. En supposant que cet objet est disponible dans la base de données, vous pointez effectivement s sur le même objet que schedule. Cela signifie que s.Timetables et schedule.Timetables sont la même collection, ce qui provoquera l'exception que vous obtenez pendant la foreach.

Je suppose que vous voulez seulement cloner l'objet pointé par la planification et l'insérer dans la base de données si ce n'est pas déjà dans la base de données. Dans ce cas:

Schedule s = _entities.Schedules.Where(x => x.ScheduleID == schedule.ScheduleID && x.BookingObject.BookingObjectID == bookingObjectID).FirstOrDefault(); 

if(s == null) 
{ 
    s = new Schedule(); 

    s.ScheduleStart = schedule.ScheduleStart; 
    s.ScheduleEnd = schedule.ScheduleEnd; 

    foreach (var t in schedule.Timetables) 
    { 
     s.Timetables.Add(t); 
    } 

    _entities.AddToSchedules(s); 

    _entities.SaveChanges(); 
}