Question

here’s my problem .. i’m doing a calculator in C# and i don’t want to click every single button to make a operation, i wanna handle it with my num pad .. like
if i press “1” , show me in the textbox “1”.

i changed

private void cmd1_Click(object sender, KeyPressEventArgs e)
        {
            if (e.KeyChar == '1')
            {
                txtShow.Text='1';
            }
        }

and i’m having this error :

No overload for ‘cmd1_Click’ matches delegate “System.EventHandler”

this.cmd1.Click += new System.EventHandler(this.cmd1_Click);

what the hack is wrong with this?

Cheers.

Answer 1

change

this.cmd1.Click += new System.EventHandler(this.cmd1_Click);

to

this.cmd1.KeyPress += new System.EventHandler(this.cmd1_Click);

You’ll probably want to rename cmd1_Click too.

And as mentioned in the answer above, this would be better on the Form itself, rather than each button.