Nicht alle leeren Listen werden aus ObservableCollection entfernt?

Question

Ich habe Code zum Erstellen einer gruppierten Liste für eine ListView in Xamarin Forms, die aus irgendeinem Grund nur manchmal entfernt eine Gruppe aus der Liste, wenn es leer ist.

char[] alpha = "ABCDEFGHIJKLMNOPQRSTUVWXYZ".ToCharArray(); 
var animals = new List<string>() { "Jaguar", "Elephant", "Nemo", "Cat", "Dog", "Iguana", "Puma", "Crow", "Hawk", "Owl", "Badger", "Meerkat", "Lion", "Tiger", "Rabbit", "Pig" }; 

var groups = new ObservableCollection<GroupedItemModel>(); 

for (int i = 0; i < alpha.Length; i++) 
{ 
    groups.Add(new GroupedItemModel(alpha[i].ToString())); 
} 

foreach (var _group in groups) 
{ 
    foreach (var _animal in animals) 
    { 
     if (_animal[0].ToString().ToUpper() == _group.GroupName.ToUpper()) 
     { 
      _group.Add(_animal); 
     } 
    } 
} 

for (int i = 0; i < groups.Count; i++) 
{ 
    if (groups[i].Count == 0) 
    { 
     groups.RemoveAt(i); 
    } 
} 

ListSource = groups; 

Dies geschieht jedoch:

Warum diese Gruppen nicht entfernt werden? Irgendwelche Lösungen?

Answer 1

Beim Entfernen von einer Liste müssen Sie rückwärts arbeiten.

wie:

wenn Sie 1,2,3,4,5,6,7,8,9,10 haben, und Sie sagen, entfernen gerade Zahlen als 1-10, auf 1, das ist ok Bei 2 entfernst du es, also verschiebt sich 3 an seinen Platz .. also wirst du nicht 3 überprüfen ... du sprichst zu 4 .. in diesem Beispiel würdest du damit durchkommen, aber was wäre, wenn deine Liste bereits fehlte sagen 3, wäre es Nummer 4 an den Ort bewegt haben, wo 2 gewesen war, und weitermachen, würde 4

ändern for-Schleife absteigend sein

Answer 2

Wie oben durch BugFinder beschrieben verpasst werden, das Problem, das Sie bekommen ist caus Ändern Sie die Liste, während Sie sie durchlaufen. Problem gelöst. Es könnte sich aber auch lohnen, den Gesamtansatz zu betrachten. Derzeit wir:

1. erstellen Liste aller möglichen Gruppen (AZ)
2. Iterate obwohl die Liste der Tiere und die jeweils auf dem ersten Buchstaben
3. Iterate obwohl die Gruppen auf der Basis einer Gruppe hinzufügen und jede wegzuwerfen die sind leer.

Dies alles kann mit ein paar Zeilen von Linq-Code und einer zwicken zum Wir GroupedItemModel

public class GroupedItemModel 
{ 
    public GroupedItemModel(string name, IEnumerable<string> values){ 
     Name = name; 
     Values = new List<string>(values); 
    } 

    public string Name { get; } 
    public List<string> Values { get; } 

} 

getan werden jetzt eine ObservableCollection<GroupedItem> mit

new ObservableCollection<GroupedItem>(animals.GroupBy(a => char.ToUpper(a[0]).ToString()).OrderBy(g => g.Key).Select(g => new GroupedItem(g.Key, g))); 

Sehr gering Nachwort versehen können: Voranstellen Variablennamen mit Unterstrichen werden normalerweise für Membervariablen einer Klasse verwendet, nicht für lokale Variablen (_group, _animal). Es ändert nicht die Funktionsweise des Codes, aber wenn Sie Code mit anderen teilen, hilft Ihnen die Verwendung der allgemeinen Konventionen, den Code schneller zu lesen und zu verstehen.