You are putting them in a module - in the worksheet (class) module to be precise. You may find it useful to call subs/functions sat within a standard module from the worksheet module (eg in response to Events) as this then makes it easy for different sheets/objects to call the same code, but you haven't done anything horribly wrong