Runtime error 1004 application defined or object defined error

**Wyliecody** · 02-01-2012, 09:44 AM

I have tried everything I think.

I am using a workbook open event to start two different Inputboxes to change a cell that activates a worksheet change event and then runs a set of several inputboxes.

When I run the Open event I get the above error on the second macro.

Currently the code is simple. Each macro after these runs seperately and together. This is the only hiccup and it was working fine but something changed.

I have tried it everywhich way and got a different Run time 1004 that said some thing about not finding the macro.

I am also new to VBA but learning very quickly, any help would be great.

Please Login or Register  to view this content.

**TMS** · 02-01-2012, 09:45 AM

Please post the workbook with the full code.

Regards, TMS

**Wyliecody** · 02-01-2012, 10:13 AM

BoilerCalctooltest.xlsm

Its huge but here you go.

**TMS** · 02-01-2012, 11:31 AM

Change the name of the module from Tower_Number to just TowerNumber.

You can't/shouldn't have a module and a subroutine/function with the same name.

I think you'd be far better off with a User Form and ask all the questions at once and then validate that you have all the information you need. But that's a different story.

Regards, TMS

**Simon Lloyd** · 02-01-2012, 11:50 AM

I have to agree with TMS, the workbook is highly annoying having constant pop ups, just one userform would suffice, you can always add error handling to check if someone has left a box empty where you require a value

Google Me

Find me located here Simon Lloyd and what i'm about Here
The above is NOT a link to a forum so is NOT against Rule 13

**Wyliecody** · 02-02-2012, 07:08 PM

that was simple. It worked. I do have some method behind my madness. The thought process is User ease of use from a tablet. That way it can be punched in simply and quickly and then get a price. However, still being new to this I am curious about what you are suggesting. A userform meaning what? the only userform I have used is a Button to calculate all the info. That still requires an extra step. I want it a idiot proof as possible, You think a user form could do that?

**TMS** · 02-02-2012, 07:46 PM

You're welcome. Thanks for the rep.

You're using code already. I assume, but I could easily be wrong, that if the existing code works on a tablet, then a user form would.

Under normal circumstances, I would think a form would be the best way. You'd have a label for each field with a description and a text box for each field. Then you'd have two buttons, process and cancel. If process is pressed, you can validate all the data fields. That's the theory, anyway.

Regards, TMS