One event for multiple buttons

Question

1.00/5 (2 votes)

See more:

How do I make one event for multiple buttons?
I have a custom Russian keyboard with 33BigLetters and 33SmallLetters
(total=66Letters and also 66 buttons for each letter)
I know I did it some time ago, but right now I cant find anywhere and I don't remember how to do it.

I want to create a single event, that for each button pressed, a textbox.text add the button.text value.
Thanks.

its goes like this but I cant really do anything useful with this:

void MultiBtn_Click(Object sender,
                           EventArgs e)
        {

            Button clickedButton = (Button)sender;
            textBox1.Text += clickedButton.Text;
        }

Posted 24-Nov-11 3:47am

_Q12_

Updated 31-Mar-23 20:45pm

v2

Add a Solution

Comments

[no name] 24-Nov-11 9:51am

What more do you want? Clarify what you are trying to accomplish.

2 solutions

Solution 1

All you have to do is set all of the Click event handlers to use that event handler. Inside the handler, do this:

C#

Button button = sender as Button;
switch (button.Name)
{
    case "button1":
        // do something for button1
        break;
    case "button2":
        // do something for button2
        break;
    // and so on

}

EDIT ====================================

Why was this 1-voted? it's a viable solution to the posted question.

Posted 24-Nov-11 4:00am

#realJSOP

Updated 25-Nov-11 3:07am

v2

Comments

_Q12_ 24-Nov-11 10:13am

I am sure that its a much simpler way...
Error 1 Control cannot fall through from one case label ('case "button1":') to another,for this part:
case "button1": textBox1.Text += button1.Text;

#realJSOP 25-Nov-11 9:06am

I don't understand what you're on about. The code I gave you will compile and run.

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Accepted Answer · 2011-11-24T04:04:00

Solution 2

The code you have is pretty much what you need, all you need to do is add that to the Click event of each button. You can do that either in the designer via the Click drop down, or in code:

C#

myButton1.Click += new EventHandler(MultiBtn_Click);
myButton2.Click += new EventHandler(MultiBtn_Click);
...

If it makes it easier though, there is also the Tag property which you could store the character in, if the Text property gives you problems.

Posted 24-Nov-11 4:04am

OriginalGriff

Comments

_Q12_ 24-Nov-11 10:07am

for each button from all 66 to make EventHandlers ?
I dont want to do that... I want ONE EventHandler for ALL 66 buttons.

[no name] 24-Nov-11 10:16am

There is only ONE EventHandler. You need to assign it to all 66 buttons, there is no other way. Though you could try something like this

foreach(Control ctrl in Controls)
{
if(ctrl is Button)
{
ctrl.Click += new EventHandler(MultiBtn_Click);
}
}

_Q12_ 24-Nov-11 10:38am

That's What I was looking for, Mark.
Many thanks.

OriginalGriff 24-Nov-11 10:31am

As Mark said that is a single event handler - you have to hook them in somehow!

LanFanNinja 24-Nov-11 11:19am

+5