EventHandler not Fire
-
I want to know why this does not work. It is very simple, but when I click on it, it does not show the message box. I want to know why.
MainMenu mainMenu1 = new MainMenu(); MenuItem menuItem1 = new MenuItem("&File"); menuItem1.MenuItems.Add("Data"); menuItem1.MenuItems.Add("Computer"); menuItem1.MenuItems.Add("Keyboard"); menuItem1.MenuItems.Add("Telephone"); menuItem1.Click += new EventHandler(OnComputer); void OnComputer(object sender, EventArgs e) { MessageBox.Show("You click on computer", "Computer Click"); } -
I want to know why this does not work. It is very simple, but when I click on it, it does not show the message box. I want to know why.
MainMenu mainMenu1 = new MainMenu(); MenuItem menuItem1 = new MenuItem("&File"); menuItem1.MenuItems.Add("Data"); menuItem1.MenuItems.Add("Computer"); menuItem1.MenuItems.Add("Keyboard"); menuItem1.MenuItems.Add("Telephone"); menuItem1.Click += new EventHandler(OnComputer); void OnComputer(object sender, EventArgs e) { MessageBox.Show("You click on computer", "Computer Click"); }That looks like you've assigned it to the File item, not the Computer item.
Upcoming events: * Glasgow: Geek Dinner (5th March) * Edinburgh: Web Security Conference Day for Windows Developers (12th April) My: Website | Blog | Photos
-
That looks like you've assigned it to the File item, not the Computer item.
Upcoming events: * Glasgow: Geek Dinner (5th March) * Edinburgh: Web Security Conference Day for Windows Developers (12th April) My: Website | Blog | Photos
-
The computer item is a subitem of the file menu. So how can I assign it to the computer item. How can I fix that?
mfcuser wrote:
how can I assign it to the computer item.
MainMenu mainMenu1 = new MainMenu();
MenuItem menuItem1 = new MenuItem("&File");
menuItem1.MenuItems.Add("Data");
MenuItem computerMenuItem = new MenuItem("Computer"); menuItem1.MenuItems.Add(computerMenuItem);
menuItem1.MenuItems.Add("Keyboard");
menuItem1.MenuItems.Add("Telephone");computerMenuItem.Click += new EventHandler(OnComputer);void OnComputer(object sender, EventArgs e)
{
MessageBox.Show("You click on computer", "Computer Click");
}/ravi
This is your brain on Celcius Home | Music | Articles | Freeware | Trips ravib(at)ravib(dot)com
-
mfcuser wrote:
how can I assign it to the computer item.
MainMenu mainMenu1 = new MainMenu();
MenuItem menuItem1 = new MenuItem("&File");
menuItem1.MenuItems.Add("Data");
MenuItem computerMenuItem = new MenuItem("Computer"); menuItem1.MenuItems.Add(computerMenuItem);
menuItem1.MenuItems.Add("Keyboard");
menuItem1.MenuItems.Add("Telephone");computerMenuItem.Click += new EventHandler(OnComputer);void OnComputer(object sender, EventArgs e)
{
MessageBox.Show("You click on computer", "Computer Click");
}/ravi
This is your brain on Celcius Home | Music | Articles | Freeware | Trips ravib(at)ravib(dot)com
It is working fine now. About if I have several menu items. There must be a way to pack them to an array rather than doing it one by one. So the way I had "Data", "Keyboard", "Telephone" was incorrect. The menu items need to have their own variable to identify them rather than single string.