Установить значение TextBox

@Saulq · Регистрация: 03.12.2018

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

В моей программе находится множество текстовых полей. И если в поле стоит знак '-' то это число -1, а если поле пустое то это число 1. Как можно оптимизировать код так, что бы постоянно не писать это для каждого TextBox'a:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
            if (textBoxA.Text == "")
                a = 1;
            else if (textBoxA.Text == "-")
                a = -1;
            else
                a = Convert.ToDouble(textBoxA.Text);
 
            if (textBoxB.Text == "")
                b = 1;
            else if (textBoxA.Text == "-")
                b = -1;
            else
                b = Convert.ToDouble(textBoxB.Text);

@QuakerRUS · 09.11.2020, 18:15

Saulq, создайте массив TextBox и пройдитесь циклом.

@Kyra90 · 09.11.2020, 18:25

Согласна с QuakerRUS, но можно без массива

C#
1
2
3
4
5
            foreach (TextBox c in Controls.OfType<TextBox>())
            {
                if (c.Text == "-") c.Text = "-1";
                if (c.Text == "") c.Text = "1";
            }

@Enifan · 09.11.2020, 20:27

А инициализация переменных ?

Добавлено через 26 минут
Предложу свой вариант с инициализацией переменных. Тут нужно учитывать названия TextBox'ов и переменных

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
using System;
using System.Reflection;
using System.Windows.Forms;
 
namespace WinForm
{
    public partial class Form1 : Form
    {
        double a, b, c;
 
        public Form1()
        {
            InitializeComponent();
 
            textBoxA.TextChanged += TextBox_TextChanged;
            textBoxB.TextChanged += TextBox_TextChanged;
            textBoxC.TextChanged += TextBox_TextChanged;
        }
 
        private void TextBox_TextChanged(object sender, System.EventArgs e)
        {
            TextBox tb = sender as TextBox;
            FieldInfo fi = Type.GetType(GetType().FullName).GetField(tb.Name.ToLower().Replace("textbox", ""), BindingFlags.NonPublic | BindingFlags.Instance);
            fi.SetValue(this, tb.Text == "" ? 1 : tb.Text == "-" ? -1 : double.Parse(tb.Text));
            MessageBox.Show($"{fi.Name} = {fi.GetValue(this)}");
        }
    }
}

@Kyra90 · 09.11.2020, 20:41

Enifan, Хорошо, а если так обходить ненужные текстбоксы?

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

В моей программе находится множество текстовых полей

может их там сотня

C#
1
2
3
4
5
6
7
          foreach (TextBox c in Controls.OfType<TextBox>())
            {
                if(c.Name = "textBox2")continue;
 
                if (c.Text == "-") c.Text = "-1";
                if (c.Text == "") c.Text = "1";
            }

@QuakerRUS · 09.11.2020, 21:05

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

C#
1
if(c.Name = "textBox2")continue;

Зачем задавать исключения, когда можно создать коллекцию/массив необходимых для обработки TextBox? Так вы повышаете риск ошибки при добавлении в форму нового TextBox, который не будет иметь отношения к логике групповой обработки.

Enifan, не перебор ли использовать рефлексию? Рекомендуют использовать ее как можно меньше, да и медленно она работает.

@Enifan · 10.11.2020, 05:32

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

а если так обходить ненужные текстбоксы?

в любом перечисленном варианте ставить на текстбоксе "метку", от нее отталкиваться

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

не перебор ли использовать рефлексию?

как будем инициализировать переменные double a, double b ?
Вообще перебор, но тогда можно обойтись связкой данных (ТекстБокс и double), пойдет ли такой вариант ТС ?

@QuakerRUS · 10.11.2020, 11:38

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

как будем инициализировать переменные double a, double b ?

В массив их загнать как и текстбоксы.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
using System;
using System.Collections.Generic;
using System.Windows.Forms;
 
namespace WindowsFormsApp1
{
    public partial class Form1 : Form
    {
        double[] a = new double[3];
        TextBox[] t;
        SortedDictionary<string, int> d = new SortedDictionary<string, int>();
 
        public Form1()
        {
            InitializeComponent();
        }
 
        private void Form1_Load(object sender, EventArgs e)
        {
            t = new TextBox[] { textBox1, textBox2, textBox3 };
 
            for (int i = 0; i < 3; i++)
            {
                t[i].TextChanged += textBox_TextChanged;
                d.Add(t[i].Name, i);
            }
        }
 
        private void textBox_TextChanged(object sender, EventArgs e)
        {
            TextBox s = sender as TextBox;
            int n = 0;
 
            if (s.Text == "")
                n = 1;
            else if (s.Text == "-")
                n = -1;
 
            a[d[s.Name]] = n;
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            MessageBox.Show($"{a[0]} {a[1]} {a[2]}");
        }
    }
}

@Enifan · 10.11.2020, 12:20

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

В массив их загнать как и текстбоксы.

Я думал над этим. Соглашусь, такой вариант лучше, но устроит ли это ТС.

@QuakerRUS · 10.11.2020, 13:02

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

но устроит ли это ТС.

Можно сделать массив указателей или прописать ссылки. Но я не вижу объективных причин не использовать просто массив. Вариант со ссылками:

C#
1
2
3
4
double[] m = new double[3];
ref double a = ref m[0];
ref double b = ref m[1];
ref double c = ref m[2];

Новые блоги и статьи Все статьи Все блоги /
Вывод данных через динамический список в справочнике Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .	Функция заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .
Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем	10 пpимет, которые всегда сбываются Maks 31.03.2026 1. Чтобы, наконец, пришла маршрутка, надо закурить. Если сигарета последняя, маршрутка придет еще до второй затяжки даже вопреки расписанию. 2. Нaдоели зима и снег? Не надо переезжать. Достаточно. . .	Перемещение выделенных строк ТЧ из одного документа в другой Maks 31.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .

@QuakerRUS 1469 / 1010 / 456 Регистрация: 30.10.2017 Сообщений: 2,799
	09.11.2020, 18:15
	Saulq, создайте массив TextBox и пройдитесь циклом. 0

Установить значение TextBox

Решение