Is this the best way for me to set events for my controls?-Collection of common programming errors

There is no problem with this, and it won’t bog down your application…

Except…

If you call this function more than one time, for a single instance of your class, you could cause problems. Each time you call SetPictureBoxEvents(), you’ll get a new set of delegates attached to your event handler. If you call this four times, and don’t unsubscribe, your routines will run 4 times each when an event is raised.