Avoid Multiple Click Handlers

Question

0.00/5 (No votes)

See more:

I have several buttons emulating a keyboard...well..everytime you click on a key, the program must redirect to the page of its key, ok?...Is there a better way of doing this than putting this on every click() event?

C#

private void btnA_Click(object sender, EventArgs e)
        {
            ObtenerResultados(((Button) sender).Text);
        }

C#

private void btnB_Click(object sender, EventArgs e)
        {
            ObtenerResultados(((Button) sender).Text);
            Application.DoEvents();
        }

Is the same call for all buttons...maybe there is a faster way?

Thanks

Posted 21-Dec-10 7:25am

W1SHM4ST3R

Updated 21-Dec-10 7:44am

AspDotNetDev

v2

Add a Solution

4 solutions

Solution 1

I don't really follow you, but your buttons can all share the same click handler (you only need a single "btn_Click" rather than a "btnA_Click", "btnB_Click", and so on). I'm not sure if you're using ASP.Net or Windows Forms (or WPF), so I don't really want to provide you with sample code that might confuse you. Basically, go to where the click handler is assigned and choose a different one.

Posted 21-Dec-10 7:36am

AspDotNetDev

Solution 4

Ok, more exactly, you need one handler for all buttons representing a keyboard, but you still need information on what exactly key was pressed. Also, you need to use better syntax -- the auto-generated syntax is gravely obsolete (you're interested in v.4.0, huh?) and inferior relative the lambda syntax.

The code will be something like this:

C#

for (Button button in KeyboardButtons)
   //in the following line, you're sure sender can be cast to Button:
   button.Click += (sender, args) => { ClickHandler((Button)sender); };

//...

//you don't have to use args parameter:
void ClickHandler(Button sender) {
    string text = sender.Text; //in WPF it would be (string)sender.Content
    Debug.Assert(text.Length > 0);
    char key = text[1];
   //now you know which key is represented by the button pressed
   //...
} //ClickHandler

..//

Posted 21-Dec-10 17:17pm

Sergey Alexandrovich Kryukov

Updated 21-Dec-10 17:24pm

v5

Comments

Espen Harlinn 26-Feb-11 11:02am

Right, my 5

Sergey Alexandrovich Kryukov 26-Feb-11 19:35pm

Thank you.
--SA

_Ashish 27-Feb-11 6:32am

its simpler and thoughful +5

Solution 5

Some new references on Virtual Keyboard topic; I answered few related Questions, so some important aspects are summarized here: Application focus getting and losing[^] and here: Programming on BACKSPACE button[^].

—SA

Posted 26-Feb-11 13:40pm

Sergey Alexandrovich Kryukov

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)

#realJSOP · Accepted Answer · 2010-12-21T08:19:00

Solution 3

Why are you calling DoEvents from an event handler?

Just create one button click handler and do what you're already doing:

C#

private void btnLetters_Click(object sender, EventArgs e)        
{
    ObtenerResultados(((Button) sender).Text);        
}

And then attach that event to all of the applicable buttons.

Posted 21-Dec-10 8:19am

#realJSOP

Updated 21-Dec-10 8:24am

v5

Comments

Sergey Alexandrovich Kryukov 21-Dec-10 23:08pm

DoEvents skips event cycle of the UI thread until all pending message are processed. In almost all cases, it's a bad idea to use it.