I'm working on a fairly large Excel project.

One of the larger bits is a repair feature, where clicking on the toolbar
button brings up a screen with a list of all the available worksheets in the
workbook. The user checks off which sheets will be effected then can either
clear or repair the selected items. Clear blanks out the user editable
cells, repair restores all the formulas for the selected worksheets.
Currently the code is like this:

The user checks off the sheets they want to effect and hits the clear or
repair button. The code checks to see which sheets were selected and for
every one it finds it runs that sheet's clear/repair sub procedure which is
contained in a general "ClearRepair" module.

Generally speaking is it better to have long bits of code in a form or in a
module. I'm wondering if I'm being smart by doing it the way I'm doing it
or if it will run better/cleaner/smaller to pull all the subs out of the
module and put them into forms code.

Thanks for any advice.
CB Hamlyn