Ввод массивов через множество textBox, подсчёт суммы, и вывод через listBox. Ошибка при вводе через textBox

@DarkSinner · Регистрация: 23.02.2015

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

Создал я кучу текст боксов, там происходит ввод каждого элемента массива. И вывод через листбокс
По идее должен высчитывать сумму ряда и показывать чему равна эта сумма.
Программа такая работает тока если ввести все поля.
Но консольная версия без использования double.Parse() работает даже если какое-то поле не введено.
в sum = new double[13]; по идее sum = new double[12] должна быть а работает ток с 13.
В цикле использую 12 хотя можно и 13,
если ввод заменить числами, работает как часы.
Подскажите как исправить. Я ток осваиваю шарп желательно попроще обьясните пожалуста

C#

                  int i;
                  double summ;
                  double[] sum = new double[13];
                  sum[0] = double.Parse(tb_met.Text);               
                  sum[1] = double.Parse(tb_ugld.Text);
                  sum[2] = double.Parse(tb_tol.Text);
                  sum[3] = double.Parse(tb_am.Text);
                  sum[4] = double.Parse(tb_ksi.Text);
                  sum[5] = double.Parse(tb_uglok.Text);
                  sum[6] = double.Parse(tb_azdi.Text);
                  sum[7] = double.Parse(tb_form.Text);
                  sum[8] = double.Parse(tb_etben.Text);
                  sum[9] = double.Parse(tb_andser.Text);
                  sum[10] = double.Parse(tb_serv.Text);
                  sum[11] = double.Parse(tb_benz.Text);
                  sum[12] = double.Parse(tb_fen.Text);
              summ = 0;
 
                for (i = 0; i <= 12; i++)
                {
                    summ += sum[i];
                    if (i > 11)
                    {
                        listBox1.Items.Add(string.Format("Summ={0}", summ));
                    }

Добавлено через 16 минут
Проблема решена поставил значение text 0. Теперь он по умолчанию ставит 0 значение и тем самым программа работает коректно

@chumich · 23.02.2015, 18:08

13 - потому что у вас тринадцать текстбоксов. Уберете один - сможете сделать размерность массива -12.
Уберите if - он не нужен.
Попробуйте заменить double.Parse на Convert.ToDouble.
Если с числами работает, значит, возможно, проблема в именах текстбоксов. Попробуйте поставить 13 новых и не меняйте имена: пусть в коде будут textBox1.Text и т. д. Проверьте, если будет работать - значит проблема в этом.

C#

int i;
double summ;
double[] sum = new double[13];
sum[0] = Convert.ToDouble(tb_met.Text); 
sum[1] = Convert.ToDouble(tb_ugld.Text);
sum[2] = Convert.ToDouble(tb_tol.Text);
sum[3] = Convert.ToDouble(tb_am.Text);
sum[4] = Convert.ToDouble(tb_ksi.Text);
sum[5] = Convert.ToDouble(tb_uglok.Text);
sum[6] = Convert.ToDouble(tb_azdi.Text);
sum[7] = Convert.ToDouble(tb_form.Text);
sum[8] = Convert.ToDouble(tb_etben.Text);
sum[9] = Convert.ToDouble(tb_andser.Text);
sum[10] = Convert.ToDouble(tb_serv.Text);
sum[11] = Convert.ToDouble(tb_benz.Text);
sum[12] = Convert.ToDouble(tb_fen.Text);
summ = 0;
for (i = 0; i <= 12; i++)
{
    summ += sum[i];
}
listBox1.Items.Add(string.Format("Summ={0}", summ));

@DarkSinner · 23.02.2015, 18:49 **[ТС]**

Ну if нужен для того что бы сумма конечная была, а то он выдаст 13 значении с разными числами ;
13 новых textBox не помогли. И Convert.ToDouble не меняет данной ситуации.
И разве цикл не считываеться по порядку 0,1,2,3,4...12 ,то есть 13 textBoxs?

@chumich · 23.02.2015, 19:00

Сообщение от DarkSinner

Ну if нужен для того что бы сумма конечная была, а то он выдаст 13 значении с разными числами

Не выдаст (в моем коде). Цикл заканчивается на 21 строке. И в листбокс пойдет конечный результат.
Цикл считывается по порядку с 0 до 12.

@~~Ev_Hyper~~ · 23.02.2015, 19:34

DarkSinner, если вам нужно посчитать сумму значений во всех текстбоксах, то можно сделать проще:

C#

            listBox1.Items.Add(string.Format("Сумма = {0}",
                 Controls.OfType<TextBox>() //выбираем из всех элементов формы только textBox
                .Select(x => x.Text)//получаем введенный текст
                .Where(x => !string.IsNullOrWhiteSpace(x))//проверяем, что строка не пустая
                .Select(x => double.Parse(x))//преобразовываем строку в число
                .Sum()//суммируем
                ));

можно записать немного короче, но я решил рассписать, чтобы было более понятно.

@DarkSinner · 23.02.2015, 20:22 **[ТС]**

Ev_Hyper Спасибо, но мне не надо суму всех текс боксов мне надо сумму определённых.

@~~Ev_Hyper~~ · 23.02.2015, 21:12

DarkSinner, в таком случае можно использовать точно такой-же код, только добавить строчку отбора п критерию (например, те, чьи названия начинаются с: "tb_".

C#

            listBox1.Items.Add(string.Format("Сумма = {0}",
                Controls.OfType<TextBox>() //выбираем из всех элементов формы только textBox
                .Where(x => x.Name.StartsWith("tb_")) //условие отбора
                .Select(x => x.Text)//получаем введенный текст
                .Where(x => !string.IsNullOrWhiteSpace(x))//проверяем, что строка не пустая
                .Select(x => double.Parse(x))//преобразовываем строку в число
                .Sum()//суммируем
                ));

Или сгруппировать текстбоксы в один groupBox или другой группирующий элемент управления

C#

            listBox1.Items.Add(string.Format("Сумма = {0}",
                 groupBox1.Controls.OfType<TextBox>() //выбираем из всех элементов groupBox1 только textBox
                .Select(x => x.Text)//получаем введенный текст
                .Where(x => !string.IsNullOrWhiteSpace(x))//проверяем, что строка не пустая
                .Select(x => double.Parse(x))//преобразовываем строку в число
                .Sum()//суммируем
                ));

как видите вариантов может быть много, но изменения в коде происходит всего в 1 строчке!

@DarkSinner · 24.02.2015, 09:06 **[ТС]**

Ev_Hyper Спасибо воспользовался с условием отбора, а с группбоксом взял на заметку.
А не подскажите как вернуть быстрые клавиши ctrl +c , ctrl +v, ctrl +x.
А то заблочил и не знаю как вернуть.
кейпресс

C#

 char ch = e.KeyChar;
            if (!Char.IsDigit(ch) && ch != ',' && ch != 8)
            {
                e.Handled = true;
            }

@DarkSinner 0 / 0 / 0 Регистрация: 23.02.2015 Сообщений: 6
	23.02.2015, 18:49 [ТС]	3
	Ну if нужен для того что бы сумма конечная была, а то он выдаст 13 значении с разными числами ; 13 новых textBox не помогли. И Convert.ToDouble не меняет данной ситуации. И разве цикл не считываеться по порядку 0,1,2,3,4...12 ,то есть 13 textBoxs? 0

@chumich 2080 / 1238 / 464 Регистрация: 20.12.2014 Сообщений: 3,237
	23.02.2015, 19:00	4
	Сообщение от DarkSinner Ну if нужен для того что бы сумма конечная была, а то он выдаст 13 значении с разными числами Не выдаст (в моем коде). Цикл заканчивается на 21 строке. И в листбокс пойдет конечный результат. Цикл считывается по порядку с 0 до 12. 0

@DarkSinner 0 / 0 / 0 Регистрация: 23.02.2015 Сообщений: 6
	23.02.2015, 20:22 [ТС]	6
	Ev_Hyper Спасибо, но мне не надо суму всех текс боксов мне надо сумму определённых. 0

	24.02.2015, 09:06