Please help me with this one!!!

Question

I am trying to use the toolbar function in my application. I have found that whenever one of the toolbar buttons is clicked, all toolbar butten events get executed. I do realize that there is only a single click_event for all buttons on the toolbar, so what I have done is as follows... private void tbMainApp_ButtonClick(object sender, System.Windows.Forms.ToolBarButtonClickEventArgs e) { if (e.Button == tbbEvent1) try { Event1 if (e.Button == tbbEvent2) try { Event2 if (e.Button == tbbEvent3) try { Event3 Given my apparent lack of knowledge, I am assuming that given each one of the buttons will be found on the toolbar, each will execute in succession. What should I do to ensure that when a button is clicked, ONLY THAT section of code gets executed. Any assistance you all could provide would be EXTREMELY appreciated. Thanks, Bob

kovan · Answer

using indexof is a problem as stated when a user changes or addes a button or removes one instead of using == use if (e.button is tbBtnSave) { //save } this will compare buttons not indexes

jimteeuwen · Answer

Tis possible this way but doesn't work very well if the order of the buttons can be changed by the user at runtime. Using the if( e.button == myButton ) is only marginally slower compared to the switch statement as it doesnt concern 100'ds of buttons to check and it doesnt care in which order the buttons exist in the toolbar or exist in the toolbar at all.

I_Pogi · Answer

That action can also be performed quite nicely with a switch statement. i.e. switch(toolBar.Buttons.IndexOf(e.Button) { case 0: //enter in the what should be executed when the first button is pressed break; case 1: //enter in what should be executed with the second button break; //etc. } This is actually a little better because you don't have to compare each if statement. Hope this helps.

Mahesh Chand · Answer

This is the way to do it. There is no performance issues here by checking a few if statements. The code will be executed for the button that was clicked.