Question

我的Form1中有12个按钮，每个按钮旁边都有一个文本框。 button事件调用一个名为dialogueOpen的方法，该方法处理从form2获取一个对象并将一个字符串值放在一个文本框中。

如何根据用户点击的按钮放置在文本框中返回的值？因此，如果用户单击了button1，则返回的文本应放在textbox1中，如果是button2，则用户单击，然后返回的文本应放在textbox2中。重点是避免使用字符串名称进行检查，因为按钮都可以称为“浏览”。

现在我的代码可以做到这一点，但重复性是否有更好的做法？

    private void dailogueOpen(String btnName)
    {
        if (listBox1.SelectedItem == null)
        {
            MessageBox.Show("Please Select a form");
        }
        else
        {
            var selectedItem = (FormItems)listBox1.SelectedItem;
            var form2result = new Form2(myDataSet, selectedItem);
            var resulOfForm2 = form2result.ShowDialog();

            if (resulOfForm2 == DialogResult.OK)
            {
                switch (btnName)
                {
                    case "btn1":
                        textBox1.Text = form2result.getValue();
                        break;
                    case "btn2":
                        textBox2.Text = form2result.getValue();
                        break;
                    case "btn3":
                        textBox3.Text = form2result.getValue();
                        break;
                    case "btn4":
                        textBox4.Text = form2result.getValue();
                        break;
                    case "btn5":
                        textBox5.Text = form2result.getValue();
                        break;
                }
            }
        }
    }


    private void button1_Click(object sender, EventArgs e)
    {
        String name = "btn1";
        dailogueOpen(name);
    }

    private void button2_Click(object sender, EventArgs e)
    {
        String name = "btn2";
        dailogueOpen(name);
    }

    private void button3_Click(object sender, EventArgs e)
    {
        String name = "btn3";
        dailogueOpen(name);
    }

    private void button4_Click(object sender, EventArgs e)
    {
        String name = "btn4";
        dailogueOpen(name);
    }

    private void button5_Click(object sender, EventArgs e)
    {
        String name = "btn5";
        dailogueOpen(name);
    }

Answer 1

编辑：我刚注意到你的事件处理程序。接下来会有更多的重构：

是的，有。您需要以某种方式将文本框与按钮相关联。例如，像这样创建一个字典：

Dictionary<Button, TextBox> _dict;

_dict[button1] = textBox1;
_dict[button2] = textBox2; 
...

为所有事件使用一个事件处理程序：

private void button_click(object sender, EventArgs e)
{
    dialogeOpen((Button)sender);
}

将dialogueOpen更改为接受按钮而不是字符串和

 _dict[btn].Text = form2Result.getValue();

Answer 2

1您在所有same delegate

上使用button

Nota (Thank's to Marty) :当您在表单设计器中时，选择所有按钮，然后为所有按钮分配“Generic_Click”，或者您可以使用下面的代码。

this.btn1.Click += new System.EventHandler(Generic_Click); //the same delegate
this.btn2.Click += new System.EventHandler(Generic_Click);
this.btn3.Click += new System.EventHandler(Generic_Click);
....


private void Generic_Click(object sender, EventArgs e)
{
     var control = (Button)sender;
     if(  control.Name == "btn1")
     {
        ....
     }
     else if(  control.Name == "btn2")
     {
        ....
     }
     else if(  control.Name == "btn3")
     {
        ....
     }


}

Answer 3

将您的事件处理程序替换为

private void ButtonClick(object sender, EventArgs e)
{
    var button = sender as Button;
    if (button == null) return;
    String name = button.Text;// Tag, name etc
    dailogueOpen(name);
}

Answer 4

我首先只为按钮使用一个事件处理程序，它看起来像这样：

protected void ButtonClick(object sender, EventArgs e)
{
    Button clickedButton = (Button) sender;
    string selectedId = clickedButton.ID;
    string[] idParameters = selectedId.Split('_');
    string textBoxId = "textbox" + idParameters[1];
    dailogueOpen(textBoxId);
}

我在这里做的是使用一个模式来显示文本框的名称，例如，如果您的按钮包含id：button_1，button_2，...，button_n，则可以推断相应的文本框是什么。

如果单击button_1，通过拆分其ID，您将知道其对应的文本框是id为textbox1的文本框。

然后dialogOpen函数看起来像这样：

private void dailogueOpen(string textBoxId)
{
    if (listBox1.SelectedItem == null)
    {
       MessageBox.Show("Please Select a form");
    }
    else
    {
        var selectedItem = (FormItems)listBox1.SelectedItem;
        var form2result = new Form2(myDataSet, selectedItem);
        var resulOfForm2 = form2result.ShowDialog();
        if (resulOfForm2 == DialogResult.OK)
        {
            TextBox textBox = (TextBox)this.Form.FindControl("MainContent").FindControl(textBoxId);
            textBox.Text = resulOfForm2.getValue();
    }
}

其中MainContent是文本框所在容器的id。

总而言之：

我会使用按钮和texboxes id的模式。
根据点击的按钮，我推断出相应的texbox id。
然后找到texbox并更新其值。

Answer 5

您可以在所有按钮点击上使用字典和一种事件方法

Dictionary<Button, TextBox> dx = new Dictionary<Button, TextBox>;

private void ButtonClick(object sender, EventArgs e)
{
    var button = sender as Button;
    if (button == null) return;
    dx[button].Text = form2result.getValue();
}

和这样的构造函数：

public ClassName()
{
   dx.Add(button1, textBox1);
   dx.Add(button2, textBox2);
   dx.Add(button3, textBox3);
}

Answer 6

我认为您可以做的第一件事是通过消除对switch语句的需要来提高可读性：

private void dailogueOpen(TextBox textBox)
{
    if (listBox1.SelectedItem == null)
    {
        MessageBox.Show("Please Select a form");
    }
    else
    {
        var selectedItem = (FormItems)listBox1.SelectedItem;
        var form2result = new Form2(myDataSet, selectedItem);
        var resulOfForm2 = form2result.ShowDialog();

        if (resulOfForm2 == DialogResult.OK)
        {
            textBox.Text = form2result.getValue();
        }
    }
}


private void button1_Click(object sender, EventArgs e)
{
    dailogueOpen(textBox1);
}

private void button2_Click(object sender, EventArgs e)
{
    dailogueOpen(textBox2);
}

private void button3_Click(object sender, EventArgs e)
{
    dailogueOpen(textBox3);
}

private void button4_Click(object sender, EventArgs e)
{
    dailogueOpen(textBox4);
}

private void button5_Click(object sender, EventArgs e)
{
    dailogueOpen(textBox5);
}

然后，这将为您提供合理的方法签名，以引入字典（由另外两个人建议）将Button映射到TextBox，这反过来允许您使用单个事件处理程序（由其他两个人建议）用于所有按钮

Answer 7

  private void button_Click(object sender, EventArgs e)
        {

            Button button = sender as Button;
            if (button == null) return;
            String name = button.Text;// Tag, name etc


            dailogueOpen(name);
        }


     private void dailogueOpen(String btnName)
        {
            if (listBox1.SelectedItem == null)
            {
                MessageBox.Show("Please Select a form");
            }
            else
            {
                var selectedItem = (FormItems)listBox1.SelectedItem;
                var form2result = new Form2(myDataSet, selectedItem);
                var resulOfForm2 = form2result.ShowDialog();

                if (resulOfForm2 == DialogResult.OK)
                { 
                   SetTxt(btnName,form2result.getValue());
                }
            }
    }



      private void SetTxt(string btnName, string value)
            {
                int lenght = "Button".Length;
                string index = btnName.Substring(lenght); //remove Button
                TextBox t = (TextBox)this.Controls.Find("textBox" + index, true)[0];

                if (t != null)
                    t.Text = value;
            }

我怎样才能重构下面的C＃代码？

7 个答案: