ArgumentOutOfRangeException auf veränderte colletion

Question

//I have collection 
private static List<Sport> ChosenSports = new List<Sport>(); 

//and lock object for it 
private static object _lockSports = new object(); 

//I have checkbox that add or remove collection items 
private void CheckBoxSportZone_Checked(object sender, RoutedEventArgs e) 
{ 
    var chkZone = sender as CheckBox; 
    lock (_lockSports) 
    { 
     if (chkZone.IsChecked == true) 
      ChosenSports.Add(chkZone.DataContext as Sport); 
     else if (chkZone.IsChecked == false) 
      ChosenSports.Remove(chkZone.DataContext as Sport); 
    } 
} 

//And method that uses that collection 
private IEnumerable<Sport> FilterSports(HashSet<Sport> sports) 
{ 
    lock (_lockSports) 
     return sports. Where(x => ChosenSports.Contains(x)); 
} 

Wenn i checkBox Filter-Methode throw überprüfen

Eine nicht behandelte Ausnahme des Typs 'System.ArgumentOutOfRangeException' in mscorlib.dll

Zusätzliche Informationen aufgetreten: Index war außerhalb des Angebot. Muss nicht-negativ sein und weniger als die Größe der Sammlung.

Sieht aus wie ich ChechBox in der Zeit der Überprüfung der Sammlung markieren. Aber ich benutze das Lock-Schlüsselwort, also sollte es sicher sein, oder?

Answer 1

Dieser Code:

private IEnumerable<Sport> FilterSports(HashSet<Sport> sports) 
{ 
    lock (_lockSports) 
     return sports. Where(x => ChosenSports.Contains(x)); 
} 

sperrt nur die Instanziierung des Ausdrucks Linq. Die Sperre ist zu der Zeit weggegangen, zu der der Ausdruck tatsächlich aufgezählt wird.

Sie sollten wahrscheinlich dazu führen, die ChosenSports Sammlung innerhalb des Schlosses iteriert wird (und eine Kopie der Aufzählung zurück), durch .ToArray() Aufruf wie folgt:

private IEnumerable<Sport> FilterSports(HashSet<Sport> sports) 
{ 
    lock (_lockSports) 
     return sports.Where(x => ChosenSports.Contains(x)).ToArray(); 
}