Медленно работает код

@hands0me · Регистрация: 18.07.2011

Author24 — интернет-сервис помощи студентам

Здравствуйте. Я только начал работать на C# и сразу столкнулся с проблемой. Программа выполняется очень медленно (зависает на минуту или больше), хотя подобные решения не раз использовал на С++(код работал без промедления). Подскажите, пожалуйста, в чем может быть дело.

C#

//в textbox1 множество строк(тысячи) информации вроде "3009;  0.954921;0;0; 0;0;0;0;0;0;0;0;    0;0;0;1;0;0;0;0;     0.00;    0.02;   -0.02;   0.000;    0.26;   -0.26;    0.00;   0.000;     0.3;    -0.2;"
//все эти числа хочу поместить в двумерный массив Array для дальнейшей работы с ними.
 
int sh = 1;
            for (int i = 1; i < textBox1.Lines[0].Length; i++)
            {
                if (textBox1.Lines[0][i] == ';') sh++;
            }
            double[,] Array;
            StringBuilder str=new StringBuilder();
            StringBuilder obr = new StringBuilder("           ",11);
            Array = new double[textBox1.Lines.Count(), sh + 1];
            for (int i = 1; i <= textBox1.Lines.Count() - 6; i++)
            {
                int u = 0;
                int sh1 = 0;
                for (int j = 1; j <= sh; j++)
                {
                    str.Clear();
                    str.Append(textBox1.Lines[i + 4]);
                    label1.Text = Convert.ToString(i);
                    int l = 0;
                    do
                    {
                        u++;
                        if (str[u] != ' ')
                        {
                            obr[l] = str[u];
                            if (str[u] == '.') obr[l] = ',';
                            l++;
                        }
                        if (str[u + 1] == ';')
                        {
                            sh1++;
                            u++;
                        }
 
                    } while (sh1 != j);
                    Array[i, j] = Convert.ToDouble(Convert.ToString(obr));
                    for (int z = 0; z < obr.Length; z++) obr[z] = ' ';
                }
            }

@znake · 18.07.2011, 20:26

при условии, что я правильно понял то, что ты хочешь увидеть, попробуй вот так, как результаты?

C#

//не забыть подключить System.Globalization
            var numberFormat = (NumberFormatInfo) CultureInfo.CurrentCulture.NumberFormat.Clone();
                        numberFormat.NumberDecimalSeparator = ".";
            textBox1.Text = textBox1.Text.Replace(" ","");
            var inLine = 0;
            foreach (var ch in textBox1.Lines[0])
                if (ch==';') inLine++;
            var array = new double [textBox1.Lines.Length, inLine];
            var temp_str = "";
            var str = new string[inLine];
            for (var i=0; i<textBox1.Lines.Length; i++)
            {
                temp_str = textBox1.Lines[i].Remove(textBox1.Lines[i].Length-2,1);
                str = textBox1.Lines[i].Split(';');
                for(var j=0; j<str.Length-1; j++)
                    array[i,j] = double.Parse(str[j],numberFormat);
            }

Добавлено через 11 минут
то же самое, только убрано лишнее ненужное действие на каждом шаге:

код

C#

            var numberFormat = (NumberFormatInfo) CultureInfo.CurrentCulture.NumberFormat.Clone();
            numberFormat.NumberDecimalSeparator = ".";
            textBox1.Text = textBox1.Text.Replace(" ","");
            var inLine = 0;
            foreach (var ch in textBox1.Lines[0])
                if (ch==';') inLine++;
            var array = new double [textBox1.Lines.Length, inLine];
            var str = new string[inLine];
            for (var i=0; i<textBox1.Lines.Length; i++)
            {
                str = textBox1.Lines[i].Split(';');
                for(var j=0; j<str.Length-1; j++)
                    array[i,j] = double.Parse(str[j],numberFormat);
            }

@hands0me · 20.07.2011, 12:17 **[ТС]**

Спасибо за ответ. Во время компиляции, как я понял во время заполнения массива данными, выдает ошибку "FormatException was unhandled"
на участке

C#

1 2	for (var j = 0; j < str.Length - 1; j++) array[i, j] = double.Parse(str[j], numberFormat);

попробую разобраться)

Добавлено через 3 часа 44 минуты
Ему что-то не нравится в формате строки str[j] ... со строкой str= "123" работает прекрасно, хотя split должен вернуть массив тех же строк. Буду признателен любым предложениям)

Добавлено через 40 минут
Не заметил простой ошибки, преобразование необходимо было проводить с 5 строки текстбокса.
Код летает) вот конечный вариант:

C#

            textBox1.Text = textBox1.Text.Replace(" ", "");
            textBox1.Text = textBox1.Text.Replace(".", ",");
            var inLine = 0;
            foreach (var ch in textBox1.Lines[0])
                if (ch == ';') inLine++;
            var array = new double[textBox1.Lines.Length, inLine];
            var str = new string[inLine];           
            for (var i = 0; i < textBox1.Lines.Length-5; i++)
            {
                str = textBox1.Lines[i+5].Split(';');                
                for (var j = 0; j < str.Length-2; j++)                    
                array[i, j] = double.Parse(str[j]);                
            }

@sh · 25.07.2011, 09:17

Еще быстрее и правильнее сделать так:

C#

// берем все из textBox1
textBox1.SelectAll();
string fullText = textBox1.SelectedText;
 
// замены
fullText = fullText.Replace(" ", "");
fullText = fullText.Replace(".", ",");
 
string [] sstr = {"\r\n"};
 
// разбиваем на строки
string [] lines = fullText.Split(sstr, System.StringSplitOptions.RemoveEmptyEntries);
 
// а вот с этим даже первый вариант летать будет
.
.
.
//

Часто встречаюсь с такой ситуацией: человек пишет веб интерфейс к БД.
Ему нужно сделать выборку, что реализуется на уровне самой базы. Но он деает это в веб коде -
перебирает значения. Естественно соединение с базой в это время постоянно активно.
Это нормально для корп. ресурса малого предприятия 10 - 50 человек. А если интернет магазин???

Смысл в том, что источник данных ВСЕГДА (ну почти) лучше освободить как можно раньше.

PS: to znake использовать var там, где хватит простых типов не есть гуд для программиста.

	25.07.2011, 09:17