 Autor
|
Tema: Limpiar datagrid vinculado a un ObservableCollection - WPF (Leído 1,467 veces)
|
TrashAmbishion
 Desconectado
Mensajes: 756
|
Tengo un datagrid (CalculoRutas) que se alimenta de un ObservableCollection con el siguiente método: Metodo: void miMetodo private void MatrizRutas_Checked(object sender, RoutedEventArgs e) { try { if (GenerarRutas.SelectedItems.Count > 0) { foreach (var selectedItem in GenerarRutas.SelectedItems) { PatrullasRutas.Items.Add(selectedItem); vmRutas.LimpiarRutasPatrullas(); } } } catch (Exception a) { MessageBox.Show(a.Message); } }
Clase: public class ViewModelRutas { public ObservableCollection<Patrulla> RutasPatrullas { get; set; } public ViewModelRutas() { RutasPatrullas = GeneratePatrullas(); } private ObservableCollection<Patrulla> GeneratePatrullas() { try { //Más código public void LimpiarRutasPatrullas() { RutasPatrullas.Clear(); } } catch (Exception e) { MessageBox.Show("MainWindowViewModel " + e.Message); return null; } } }
El problema es que cuando llamo al LimpiarRutasPatrullas, me limpia el DataGrid pero me lanza un error: "Colección modificada; puede que no se ejecute la operación de enumeración" ocurre cuando intentas modificar una colección mientras estás en medio de un bucle de enumeración (por ejemplo, mientras estás iterando sobre los elementos de la colección)" He probado poner el datagrid (CalculoRutas.ItemSource=null) y da el mismo error. ¿Alguna idea de como puedo solventarlo? Saludos
|
|
|
|
« Última modificación: 6 Marzo 2024, 12:47 pm por TrashAmbishion »
|
En línea
|
|
|
|
TrashAmbishion
Desconectado
Mensajes: 756
|
Solo añadí un break; seguido de vmRutas.LimpiarRutasPatrullas(); y listo..
Saludos
|
|
|
|
|
En línea
|
|
|
|
Eleкtro
Ex-Staff
Desconectado
Mensajes: 9.821
|
añadí un break; seguido de vmRutas.LimpiarRutasPatrullas(); y listo.. Hola. ¿Te das cuenta que al añadir la sentencia break para interrumpir el ciclo del bucle foreach, solamente acabarás iterando y añadiendo el primer elemento de la colección GenerarRutas.SelectedItems del DataGrid?. Usar un bucle pierde todo el sentido si añades un break de la forma en la que lo detallas, sin criterio / condición específica. Indiferentemente de eso, el código que has mostrado es algo confuso ya que hay dos miembros con nombre idéntico donde solo cambia el orden de las palabras: PatrullasRutas y RutasPatrullas. En cualquier caso, con el método vmRutas.LimpiarRutasPatrullas() alteras una colección eliminando todos sus elementos, lo haces en cada iteración del bucle. No hay necesidad. Simplemente mueve ese método fuera del bucle foreach y el problema se debería solucionar. No puedes iterar una colección al mismo tiempo que la alteras añadiendo o eliminando elementos en la misma. En esos casos se puede recurrir a generar una copia de la colección original para iterar los elementos de la copia y realizar las operaciones de adición o eliminación de elementos en la original. No manejo WPF, pero si te siguiese dando error entonces haz una copia del objeto selectedItem al añadir dicho elemento en el bucle: PatrullasRutas.Items.Add(copia de selectedItem);. Aténtamente, Elektro.
|
|
|
|
« Última modificación: 6 Marzo 2024, 17:34 pm por Eleкtro »
|
En línea
|
|
|
|
|
Serapis
|
Si en un bucle quieres eliminar todos los ítems, tienes que hacerlo de una de estas dos formas:
- Desde el final hacia atrás... (porque no queda ninguno huérfano por culpa de una eliminación en medio, que sea preciso atender... según sea el caso conforme al tipo de colección operada).
- Eliminado siempre el item primero (de dicha colección). Porque tras ser eliminado, el siguiente pasa a ser el primero (Esto sin embargo suele comportar más trabajo interno de reasignación de valores (internos), el caso previo este tipo de operación minimiza esto, pues suele limitarse a descontar la cantidad total de ítems y ya).
Nota que añadir y eliminar en un mismo bucle aunque es posible debes tener bien claro la posición que tiene en todo momento cada cual, de lo contrario, es posible que acabes eliminado items que previamente has añadido y por lo tanto cuando revises el resultado final, no consten los que tú pretendías que quedaran.
Considera, piensa en qué sucede con los ítems por encima (o hijos de) a uno dado cuando eliminas o cuando insertas conforme al tipo de colección que estés usando. Por ejemplo si es una lista enlazada, un array, un árbol, una tabla hash, etc...
...y sí, mirando el código ocurre como te dice Elektro, recorres en un bucle enumerado añadiendo ítems, para acto seguido dar una orden de borrar (el contenido de) la colección en la que se ha insertado el item seleccionado, no tierne sentido, o bien no precisas añadir nada, o bien no tiene sentido que los borres, al menos no con cada añadido, en todo caso al final (y una sola vez), además estás borrando de 'PatrullasRutas', no de la colección 'GenerarRutas'... que es quizás tu objetivo...
Quizás no tengas muy claro que quieres hacer, o por lo menos como debes hacerlo. si al menos indicaras cual es tu objetivo final, se te podría indicar las pautas a seguir, pero preguntas por un error sin dejar claro que es lo que pretendes que haga tu código.
|
|
|
|
|
En línea
|
|
|
|
TrashAmbishion
Desconectado
Mensajes: 756
|
Hola. ¿Te das cuenta que al añadir la sentencia break para interrumpir el ciclo del bucle foreach, solamente acabarás iterando y añadiendo el primer elemento de la colección GenerarRutas.SelectedItems del DataGrid?. Usar un bucle pierde todo el sentido si añades un break de la forma en la que lo detallas, sin criterio / condición específica.
Indiferentemente de eso, el código que has mostrado es algo confuso ya que hay dos miembros con nombre idéntico donde solo cambia el orden de las palabras: PatrullasRutas y RutasPatrullas.
En cualquier caso, con el método vmRutas.LimpiarRutasPatrullas() alteras una colección eliminando todos sus elementos, lo haces en cada iteración del bucle. No hay necesidad. Simplemente mueve ese método fuera del bucle foreach y el problema se debería solucionar.
No puedes iterar una colección al mismo tiempo que la alteras añadiendo o eliminando elementos en la misma. En esos casos se puede recurrir a generar una copia de la colección original para iterar los elementos de la copia y realizar las operaciones de adición o eliminación de elementos en la original.
No manejo WPF, pero si te siguiese dando error entonces haz una copia del objeto selectedItem al añadir dicho elemento en el bucle: PatrullasRutas.Items.Add(copia de selectedItem);.
Aténtamente,
Elektro.
Gracias por los tips... le estoy dando una vuelta para darle el enfoque que dices..
|
|
|
|
|
En línea
|
|
|
|
TrashAmbishion
Desconectado
Mensajes: 756
|
Si en un bucle quieres eliminar todos los ítems, tienes que hacerlo de una de estas dos formas:
- Desde el final hacia atrás... (porque no queda ninguno huérfano por culpa de una eliminación en medio, que sea preciso atender... según sea el caso conforme al tipo de colección operada).
- Eliminado siempre el item primero (de dicha colección). Porque tras ser eliminado, el siguiente pasa a ser el primero (Esto sin embargo suele comportar más trabajo interno de reasignación de valores (internos), el caso previo este tipo de operación minimiza esto, pues suele limitarse a descontar la cantidad total de ítems y ya).
Nota que añadir y eliminar en un mismo bucle aunque es posible debes tener bien claro la posición que tiene en todo momento cada cual, de lo contrario, es posible que acabes eliminado items que previamente has añadido y por lo tanto cuando revises el resultado final, no consten los que tú pretendías que quedaran.
Considera, piensa en qué sucede con los ítems por encima (o hijos de) a uno dado cuando eliminas o cuando insertas conforme al tipo de colección que estés usando. Por ejemplo si es una lista enlazada, un array, un árbol, una tabla hash, etc...
...y sí, mirando el código ocurre como te dice Elektro, recorres en un bucle enumerado añadiendo ítems, para acto seguido dar una orden de borrar (el contenido de) la colección en la que se ha insertado el item seleccionado, no tierne sentido, o bien no precisas añadir nada, o bien no tiene sentido que los borres, al menos no con cada añadido, en todo caso al final (y una sola vez), además estás borrando de 'PatrullasRutas', no de la colección 'GenerarRutas'... que es quizás tu objetivo...
Quizás no tengas muy claro que quieres hacer, o por lo menos como debes hacerlo. si al menos indicaras cual es tu objetivo final, se te podría indicar las pautas a seguir, pero preguntas por un error sin dejar claro que es lo que pretendes que haga tu código.
Gracias, le doy una vuelta para revisar nuevamente...
|
|
|
|
|
En línea
|
|
|
|
|
 |
