Сравнение textBox1.Text и textBox2.Text

Question

Сравнение textBox1.Text и textBox2.Text

1

поэтому у меня этот код

public partial class Form1 : Form
{
    public string str;
    public string str2;
    public Form1()
    {

        InitializeComponent();
        str = textBox1.Text;
        str2 = textBox2.Text;
    }

    private void button1_Click(object sender, EventArgs e)
    {
        if (String.IsNullOrEmpty(textBox1.Text))
        {
            MessageBox.Show("Enter Material Name Please.");     
        }

        if (str == str2)
        {
            MessageBox.Show("Materials are equal.");
        }

        else if (str != str2)
        {
            MessageBox.Show("Materials don't match.");
        }
    }
}

я думаю, вы догадаетесь, что я пытаюсь сделать здесь. Не могли бы вы сказать, что с ним не так? Я начал пару дней назад, так что простите мне недостаток знаний.

Shako 07 окт. 2014, в 10:18

Источник

0

что вы пытаетесь сравнить с двумя textbox ?
spajce 07 окт. 2014, в 07:37
0

Вы должны включить в свой вопрос только соответствующие части вашего кода, пустые методы не будут причиной проблемы.
Sayse 07 окт. 2014, в 07:42

Теги:

c#

1 ответ

Ещё вопросы

что вы пытаетесь сравнить с двумя textbox ?
Вы должны включить в свой вопрос только соответствующие части вашего кода, пустые методы не будут причиной проблемы.

Tim Schmelter · Accepted Answer · 2014-10-07T06-01-00.000Z

Вы инициализируете эти строки только один раз в конструкторе, а не в обработчике событий click. Вы также можете использовать свойства для сопоставления элементов управления со строками, которые делают код более читабельным и надежным:

// use meaningful control/variable names
string Material1 
{
    get { return textBox1.Text; }
    set { textBox1.Text = value; }
}

string Material2
{
    get { return textBox2.Text; }
    set { textBox2.Text = value; }
}

private void button1_Click(object sender, EventArgs e)
{ 
    if (String.IsNullOrWhiteSpace(Material1)) // handles also multiple spaces
    {
        MessageBox.Show("Enter Material Name Please.");  
        // return; <-- perhaps?   
    }

    if (Material1 == Material2)
    {
        MessageBox.Show("Materials are equal.");
    }
    else
    {
        MessageBox.Show("Materials don't match.");
    }
}

Вероятно, самое подходящее время для операции, чтобы узнать о ссылках и значениях, другой альтернативой, конечно, было бы для операции обновить str и str2 в пустых событиях text_changed, которые я отредактировал
Это не обязательно, но оператор возврата после окна сообщения в тесте нулевой проверки может быть улучшением. Предполагая, что OP не хочет проверять равенство, если material1 равен нулю.
@failedprogramming: я уверен, что это может быть улучшено многими способами, но это не ошибка, чтобы продолжить здесь. Я также хотел сосредоточиться на основных моментах (я уже использовал свойства для сопоставления строк с элементами управления).