C# WinForms用户输入表单值回传异常及最佳实践咨询

刚接触System.Windows.Forms，我做了一个包含两个文本框（tbox_H、tbox_V）和一个按钮的简单表单，想要让用户输入的值回传给启动该表单的代码。两个文本框默认值都是“1”，我已经确认在TextChanged事件和MouseClick事件里，值会随用户输入变化，但回传时只能拿到默认值。我怀疑把文本框声明为Public不是最佳实现方式，想了解从用户输入表单传递值的最佳实践。

相关代码 #

表单Designer生成代码

namespace MyCommands
{
    partial class MyForm
    {
        //Windows Form Designer generated code
        //some other generated code omitted for brevity
        private System.Windows.Forms.Button button1;
        public System.Windows.Forms.TextBox tbox_H;
        public System.Windows.Forms.TextBox tbox_V;

        private void InitializeComponent()
        {
            this.button1 = new System.Windows.Forms.Button();
            this.tbox_H = new System.Windows.Forms.TextBox();
            this.tbox_V = new System.Windows.Forms.TextBox();
            // button1
            this.button1.DialogResult = System.Windows.Forms.DialogResult.OK;
            this.button1.Name = "button1";
            this.button1.Text = "OK";
            this.button1.MouseClick += new System.Windows.Forms.MouseEventHandler(this.button1_MouseClick);
            // tbox_H
            this.tbox_H.Name = "tbox_H";
            this.tbox_H.Text = "1";       //default value
            this.tbox_H.TextChanged += new System.EventHandler(this.tbox_H_TextChanged);
            // tbox_V
            this.tbox_V.Name = "tbox_V";
            this.tbox_V.Text = "1"; //default value
            this.tbox_V.TextChanged += new System.EventHandler(this.tbox_V_TextChanged);
            // Form
            this.AcceptButton = this.button1;
            this.Controls.Add(this.button1);
            this.Controls.Add(tbox_H);
            this.Controls.Add(tbox_V);
        }
    }
}

表单逻辑代码

using System.Windows.Forms;

namespace MyCommands
{
    public partial class MyForm : Form
    {
        public MyForm()
        {
            InitializeComponent();
        }

        private void tbox_H_TextChanged(object sender, EventArgs e)
        {
            MessageBox.Show("At TextBox", tbox_H.Text);   //Confirmed value changes
        }

        private void tbox_V_TextChanged(object sender, EventArgs e)
        {
            MessageBox.Show("At TextBox", tbox_V.Text);  //Confirmed value changes
        }


        private void button1_MouseClick(object sender, MouseEventArgs e)
        {
            MessageBox.Show("At MouseClick", tbox_H.Text + ", " + tbox_V.Text); //Values confirmed
            this.Close();
        }
    }
}

启动表单的代码

namespace MyCommands
{
    public class MyCommand1
    {
        MyForm form1 = new MyForm();
        form1.ShowDialog();
       int var1 = Convert.ToInt32(form1.tbox_H.Text); //only default value passed
       int var2 = Convert.ToInt32(form1.tbox_H.Text); //only default value passed
    }
}

问题原因分析 #

你的MyCommand1代码存在语法错误——类的直接作用域内不能直接编写执行语句（比如form1.ShowDialog()），这些代码必须放在方法中。另外，你在获取var2时错误引用了tbox_H，应该是tbox_V。还有，按钮已经设置了DialogResult = OK，不需要在MouseClick事件中调用this.Close()，手动调用可能导致状态异常，点击按钮会自动关闭表单并返回对应的DialogResult。

最佳实践方案 #

1. 封装数据，避免直接暴露控件

不要把文本框声明为public，而是在表单类中添加属性来暴露输入值，既隐藏控件细节，又方便做数据验证：

public partial class MyForm : Form
{
    // 封装属性，对外暴露输入值
    public int HorizontalValue 
    { 
        get 
        { 
            if(int.TryParse(tbox_H.Text, out int value))
                return value;
            return 1; // 默认值
        } 
    }

    public int VerticalValue 
    { 
        get 
        { 
            if(int.TryParse(tbox_V.Text, out int value))
                return value;
            return 1; // 默认值
        } 
    }

    // ... 其他代码不变
}

2. 正确使用DialogResult判断用户操作

启动表单时，通过ShowDialog()的返回值判断用户是否确认输入，再安全获取数据：

namespace MyCommands
{
    public class MyCommand1
    {
        public void Execute()
        {
            using(MyForm form1 = new MyForm())
            {
                DialogResult result = form1.ShowDialog();
                if(result == DialogResult.OK)
                {
                    int var1 = form1.HorizontalValue;
                    int var2 = form1.VerticalValue;
                    // 在这里处理获取到的值
                }
            }
        }
    }
}

使用using语句可确保表单关闭后被正确释放资源。

3. 移除不必要的Close()调用

按钮已设置DialogResult = OK，点击后会自动关闭表单，删除button1_MouseClick事件中的this.Close()：

private void button1_MouseClick(object sender, MouseEventArgs e)
{
    MessageBox.Show("At MouseClick", tbox_H.Text + ", " + tbox_V.Text); //Values confirmed
    // 移除this.Close();
}

4. 添加输入验证（可选但推荐）

在用户确认输入时验证合法性，避免转换失败：

private void button1_MouseClick(object sender, MouseEventArgs e)
{
    if(!int.TryParse(tbox_H.Text, out _) || !int.TryParse(tbox_V.Text, out _))
    {
        MessageBox.Show("请输入有效的整数", "输入错误");
        // 阻止表单关闭
        this.DialogResult = DialogResult.None;
        return;
    }
    MessageBox.Show("At MouseClick", tbox_H.Text + ", " + tbox_V.Text);
}

内容的提问来源于stack exchange，提问作者dengebre