C# для цикла, выходящего после 1 цикла

Question

В моем проекте я пытаюсь перебрать значения и вызвать функцию на них. Когда отладка Count говорит мне, что есть 2 значения. Моя функция работает в DispatcherTimer

Мой таймер в конструкторе:

DispatcherTimer dispatcherTimer = new System.Windows.Threading.DispatcherTimer(); 
dispatcherTimer.Tick += new EventHandler(runSync); 
dispatcherTimer.Interval = new TimeSpan(0, 0, syncTime); 
dispatcherTimer.Start(); 

Моя функция

private void runSync(object sender, EventArgs e) 
{ 

    //I can see the value of count is 2 when using break points 
    List<string> vals = repo.getRemovedAnswers(); 
    for (int i = 0; i < vals.Count(); i ++) 
    { 
     //i do something with the element in my database 

     // send back a confirmation that the delete is finished 
     repo.setAnswerDeleted(vals.ElementAt(i)); 
     Console.WriteLine(i + " removed"); 
     // 
    }    
    Console.WriteLine("syncing"); 

} 

Функция setAnswerDeleted в моем классе репо, это метод аннулируются так не возвращает перерыв или что-нибудь.

public List<String> getRemovedAnswers() 
{ 
    return _answersRemoved; 
} 

public void setAnswerDeleted(string uniqueIdAnswer) 
{ 
    _answersRemoved.RemoveAll(item => item == uniqueIdAnswer); 

} 

В журнале можно увидеть цикл запускается каждый dispatchtimer цикл и онил вызовы времени метод 1, почему это цикл не работает 2 раза, когда счетчик == 2?

Answer 1

Попробуйте изменить код таким образом:

private void runSync(object sender, EventArgs e) 
{ 
    //I can see the value of count is 2 when using break points 
    List<string> vals = repo.getRemovedAnswers(); 
    for (int i = vals.Count() - 1; i >= 0; i--) 
    { 
     repo.setAnswerDeleted(vals.ElementAt(i)); 
     Console.WriteLine(i + " removed"); 
    // 
    }    
    Console.WriteLine("syncing"); 

} 

Ваш для итерации один раз, потому что вы удалить элемент из списка и приращение ваш указательный одним, а элементы после удаления являются previus длина - 1, поэтому следующая проверка для vals.Count() возвращает 1, а ваш индекс равен 1. Таким образом, ваш индекс начинается с 1 и на втором этапе равен 0.

Answer 2

Проблема в том, что вы изменяете список, повторяя его, поэтому он удаляет элементы и результат Count() идет по каждому циклу. Лучшей практикой было бы вернуть копию списка, чтобы вы не изменяли ее, когда идете.

List<string> getRemovedAnswers() 
{ 
    .. logic 
    List<string> previousReturn = ... 
    return new List<string>(previousReturn);// Creates new list 
} 

Еще одна хорошая практика заключается в использовании foreach петлю вместо того, чтобы идти по индексу.

foreach(var element in vals) 
{ 
    repo.setAnswerDeleted(element); 
} 

Если вы это сделали, это могло бы вызвать исключение, сказав, что коллекция была изменена, итерации по ней. Это сразу предупредило вас об этой проблеме.