Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > .NET для новичков > Коллекция была изменена


Автор: ДобренькийПапаша 11.5.2010, 11:01
Есть такой метод:

Код

private Dictionary<double, double> GetArrangement(double noiseLevel)
        {
            double lowest = ConvertDataPoint(currentChart.Series[0].Points.FindMinByValue());
            double highest = ConvertDataPoint(currentChart.Series[0].Points.FindMaxByValue());

            double current = lowest;

            List<double> x = new List<double>();    //Значения

            Dictionary<double, double> arrangementTable = new Dictionary<double, double>();

            while(current<=highest)
            {
                x.Add(current);
                current += noiseLevel;
            }

            List<int> p = new List<int>(); //Вероятности
            for (int i = 0; i < x.Count; i++)
            {
                p.Add(0);
            }
            for (int i = 0; i < x.Count; i++)
                {
                    for (int j = 0; j < currentChart.Series[0].Points.Count; j++)
                    {
                        if (Math.Abs(x[i] - ConvertDataPoint(currentChart.Series[0].Points[j])) <= noiseLevel)
                        {
                            p[i] += 1;
                        }
                    }
                    arrangementTable.Add(x[i], p[i]);  //Изменена коллекция
                }

            int total = 0;
            foreach(KeyValuePair<double, double> item in arrangementTable)  //ИСКЛЮЧЕНИЕ
            {
                total += (int)item.Value;
            }

            foreach (KeyValuePair<double, double> item in arrangementTable)
            {
                arrangementTable[item.Key] = (item.Value*100)/total;
            }

            return arrangementTable;
        }


Исключение вылетает и пишет:"Коллекция была изменена; невозможно выполнить операцию перечисления"
У Dictinary нет метода типа Update() так как тогда быть?

Автор: mrbrooks 11.5.2010, 12:35
Цитата

The foreach statement repeats a group of embedded statements for each element in an array or an object collection. The foreach statement is used to iterate through the collection to get the desired information, but should not be used to change the contents of the collection to avoid unpredictable side effects. 


т.е. foreach используется только для итерации коллекции (это же вам не С++). Используйте for или LINQ

Автор: ДобренькийПапаша 11.5.2010, 13:38
Спасибо, что раъяснили наконец-то саму суть. Хотя я это должен был знать, но, видимо, внимания не обращал. Сделал вот как-то так, не знаю насколько эффективно это, но работает нормально.
Код

double[] keys = new double[arrangementTable.Keys.Count];
            arrangementTable.Keys.CopyTo(keys, 0);

            for(int i=0; i<keys.Length; i++)
            {
                arrangementTable[keys[i]] = (arrangementTable[keys[i]] * 100) / total;
            }

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)