区分附加到同一事件处理程序的多个对象

Question

我很难弄清楚如何搜索/表达我想要完成的任务。我正在寻找是否有更安全/更智能的方法来实现以下示例。

假设我有 3 个带有名称的按钮：

btnOne
btnTwo
btn三

每个按钮的目的是在单击时将唯一的字符串写入某个文本框。由于所有三个按钮都执行非常相似的操作，因此为所有三个按钮创建一个事件处理程序似乎是合乎逻辑的，但是区分三个按钮以便正确的文本框可以获得正确的字符串的聪明方法是什么？我不喜欢目前的做法：

伪，伪代码：

private void Clicked(object sender, EventArgs e)
{
    string buttonName = (sender as Button).Name;
    switch(buttonName)
    {
         case "btnOne":
             tbOne.Text = "This text is from button one";
             break;
         case "btnTwo":
             tbTwo.Text = "This is some different text";
             break;
         case "btnThree":
             tbThree.Text = "Button three text";
             break;
    }
}

另一种方法是为每个按钮设置一个单击事件处理程序，然后该处理程序具有一个通用方法，例如：

private void AfterButtonClicked(Textbox tb, string text)
{
    tb.Text = text;
}

Answer 1

这是一个如此简单的案例，为了简单起见，我想我会将它们全部放在自己的事件处理程序中。

如果您想举办一个活动，您可以使用以下方法测试sender：

private void Clicked(object sender, EventArgs e)
{
    if (sender is btnOne)
        tbOne.Text = "This text is from button one";
    else if (sender is btnTwo)
        tbTwo.Text = "This is some different text";
    else if (sender is btnThree)
         tbThree.Text = "Button three text";
}

至少如果你重命名按钮，你的程序将无法编译，你会来这里修复它。

你现在的样子，如果你将“btnOne”重命名为“btn1”，tbOne 中的文本将无法更新，你可能不会注意到它。

Answer 2

在您的情况下，重用代码将 some 文本框的 Text 属性设置为 some 字符串。就我个人而言，我没有看到以您演示的方式编写单个事件处理程序的好处为您演示的操作。恕我直言，它使代码复杂化而不是简化它。

最好使用三个单独的处理程序，将三个单独的文本框属性设置为三个单独的字符串。这样您就无需进行任何进一步的比较来找出哪个按钮发送了事件。

根据实际应用程序和处理程序内的代码，此观察可能有效也可能无效。

---

但是，您可以重构它的一种方法是：

Dictionary<Button, TextBox> buttonTextboxMap = new Dictionary<Button, TextBox> 
{
    {btnOne, tbOne}, 
    {btnTwo, tbTwo}, 
    {btnThree, tbThree}
};

Dictionary<Button, string> buttonStringMap = new Dictionary<Button, string> 
{
    {btnOne, "This text is from button one"},
    {btnTwo, "This is some different text"},
    {btnThree, "Button three text"}
};

void Clicked(object sender, EventArgs e)
{
    Button btn = (Button)sender;

    buttonTextboxMap[btn].Text = buttonStringMap[btn];
}

---

第二种方法是使用每个按钮的Tag 属性。我喜欢这个，因为它将所有内容都封装在按钮本身中。但是，缺点是只有一个Tag 属性，并且您不能保证它包含适当的信息。 （您也可以将Button 子类化以保存正确的信息，但我想这取决于您所追求的可重用性以及您将使用该特定按钮的次数）：

class TextBoxAndString
{
    public TextBox tb {get; set;}
    public String s {get; set;}
}

.ctor() //the form's constructor
{
    btnOne.Tag = new TextBoxAndString {tb = tbOne, s = "This text is from button one"};
    btnTwo.Tag = new TextBoxAndString {tb = tbTwo, s = "This is some different text"};
    btnThree.Tag = new TextBoxAndString {tb = tbThree, s = "Button three text"};   
} 

void Clicked(object sender, EventArgs e)
{
    Button btn = (Button)sender;
    TextBoxAndString tbs = (TextBoxAndString)btn.Tag;

    tbs.tb.Text = tbs.s;
}