Many buttons on user form

Welcome to the Board!

Why not just have your code in a general module, so it's available to the whole VB Project? e.g.

<font face=Calibri><SPAN style="color:#00007F">Public</SPAN> i <SPAN style="color:#00007F">As</SPAN> <SPAN style="color:#00007F">Long</SPAN><br><br><SPAN style="color:#00007F">Sub</SPAN> foo()<br>    MsgBox "You clicked Button #" & i<br><SPAN style="color:#00007F">End</SPAN> <SPAN style="color:#00007F">Sub</SPAN></FONT>

If you need to know the button # to use in the code, you can delare a public variable and set it in each button.

<font face=Calibri><SPAN style="color:#00007F">Private</SPAN> <SPAN style="color:#00007F">Sub</SPAN> CommandButton1_Click()<br>    i = 1<br>    Application.Run "Foo"<br><SPAN style="color:#00007F">End</SPAN> <SPAN style="color:#00007F">Sub</SPAN></FONT>

How about posting your global code (the one piece you want to reference) so we can see exactly what you're trying to do?

Hope that helps,

Smitty

yeh this is how i started, but you end up with something like this...

sub commandbutton1_click
do something
end sub

sub commandbutton2_click
do something
end sub

sub commandbutton3_click
do something
end sub

....
sub commandbutton66_click
do something
end sub

there's got to be a more elegant way, surely?

Why do you have 50 command buttons?

The way you mention in the OP is to insert a class module. Put this code in the class module.
Then, this code in the userform's code module will assign each of the command buttons, other than the Close button (CommandButton1), as a property of a member of the new class and clicking a button on the userform will trigger Click events in both the Userform code module and the new Class module.

ah thx
this is just the ticket. and better still, i understand what you have done so i can use the method again rather than bother others.


the application is a map of storage sheds, each shed is a button which activates the accounts management etc for that particular shed. same code different shed number.

I glad my code helped, but I have to agree that 50 command buttons that call the same routine is excesive. Rather than button1, button2,...,button50, it might be easier if there was one textbox/cell/something that held a number between 1 and 50 and one command button.

I second Mike, well I second him seconding me if that makes any sense.

Couldn't you list the shed numbers in a control like a listbox or combobox?

the purpose of so many buttons rather than a combobox is to provide an interactive map for the user. i like to make things simple at the front end even if it costs me extra effort in logistics. i must say that it has come up really well and will be expanded to nearly 200 storage spaces in about 8 months time. thx again for your input.