2013-02-24 6 views
4

Я написал следующий код, который удаляет выбранное значение из спискаBox. Я также хочу удалить его из списка словарей, а затем обновить/записать его в текстовый файл, поэтому, когда я снова запустил программу и загрузил текстовый файл, он будет обновлен, если не будет отображаться удаленный элемент каждый раз, когда я запускаю свой приложение снова.Почему я получаю исключение InvalidOperationException?

private void listBox1_KeyDown(object sender, KeyEventArgs e) 
     { 
      string sb; 
      if (e.KeyCode == Keys.Delete) 
      { 
       if (this.listBox1.SelectedIndex >= 0) 
       { 
        string obj = this.listBox1.SelectedValue.ToString(); 
        data.Remove(obj); 
        listBox1.DataSource = null; 
        listBox1.DataSource = data; 
        foreach (KeyValuePair<string, List<string>> kvp in LocalyKeyWords) 
        { 
         for (int i = 0; i < kvp.Value.Count(); i++) 
         { 
          sb = "Url: " + kvp.Key + " --- " + "Local KeyWord: " + kvp.Value[i] + Environment.NewLine; 
          LocalyKeyWords.Remove(kvp.Key); 
         } 
        } 

       } 
      } 

     } 

LocalyKeyWords является Словарь>

В этом случае содержит два элемента/клавиши я удалить один и я вижу, с точки останова, что это один был удален.

Вопрос заключается в том, если мне нужно, чтобы удалить kvp.Key или удалить как-то деталь я удален из ListBox который является самым один я хочу, чтобы удалить из LocalyKeywords и это переменная OBJ, так как им делать:

data.Remove(obj); 

Возможно, мне нужно удалить obj также из localyKeyWords?

excpetion им ошибка при получении после его удаления элемента из LocalyKeyWords клику продолжают на этой линии:

foreach (KeyValuePair<string, List<string>> kvp in LocalyKeyWords) 

Im получаю ошибку:

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

System.InvalidOperationException was unhandled 
    HResult=-2146233079 
    Message=Collection was modified; enumeration operation may not execute. 
    Source=mscorlib 
    StackTrace: 
     at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource) 
     at System.Collections.Generic.Dictionary`2.Enumerator.MoveNext() 
     at GatherLinks.Form1.listBox1_KeyDown(Object sender, KeyEventArgs e) in d:\C-Sharp\GatherLinks\GatherLinks\GatherLinks\Form1.cs:line 959 
     at System.Windows.Forms.Control.OnKeyDown(KeyEventArgs e) 
     at System.Windows.Forms.Control.ProcessKeyEventArgs(Message& m) 
     at System.Windows.Forms.Control.ProcessKeyMessage(Message& m) 
     at System.Windows.Forms.Control.WmKeyChar(Message& m) 
     at System.Windows.Forms.Control.WndProc(Message& m) 
     at System.Windows.Forms.ListBox.WndProc(Message& m) 
     at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m) 
     at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m) 
     at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam) 
     at System.Windows.Forms.UnsafeNativeMethods.DispatchMessageW(MSG& msg) 
     at System.Windows.Forms.Application.ComponentManager.System.Windows.Forms.UnsafeNativeMethods.IMsoComponentManager.FPushMessageLoop(IntPtr dwComponentID, Int32 reason, Int32 pvLoopData) 
     at System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(Int32 reason, ApplicationContext context) 
     at System.Windows.Forms.Application.ThreadContext.RunMessageLoop(Int32 reason, ApplicationContext context) 
     at System.Windows.Forms.Application.Run(Form mainForm) 
     at GatherLinks.Program.Main() in d:\C-Sharp\GatherLinks\GatherLinks\GatherLinks\Program.cs:line 18 
     at System.AppDomain._nExecuteAssembly(RuntimeAssembly assembly, String[] args) 
     at System.AppDomain.ExecuteAssembly(String assemblyFile, Evidence assemblySecurity, String[] args) 
     at Microsoft.VisualStudio.HostingProcess.HostProc.RunUsersAssembly() 
     at System.Threading.ThreadHelper.ThreadStart_Context(Object state) 
     at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx) 
     at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx) 
     at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state) 
     at System.Threading.ThreadHelper.ThreadStart() 
    InnerException: 
+0

Ваши два цикла не имеют смысла. Вы перебираете ключи словаря, затем вы перебираете элементы в списке и пытаетесь удалить ключ словаря для каждого элемента в списке. Здесь что-то не так. – Steve

ответ

8

Вы удаляете элементы из LocalyKeyWords времени посреди итерации над ним; это недопустимо, как говорится в сообщении об исключении.

Я не уверен, что представляет собой большая картина, но локализованное решение состоит в том, чтобы сделать временную копию LocalyKeyWords и перебрать ее. Затем вы можете без проблем изменить «исходную» коллекцию.

Пример:

foreach (var kvp in LocalyKeyWords.ToList()) // .ToList() makes a temp copy 
1

Вы не можете изменить IEnumerable коллекции внутри foreach.

Вы можете создать копию словаря и пропустить его, модифицируя оригинал.

Смежные вопросы