Procedure too large

**JonathonHardy** · 10-26-2011, 12:24 PM

Would anyone be able to take a look at the attached file and see if I could streamline the code?

I am getting a "Compile Error" that the procedure is too large.

Essentially, what I would like the file to do is based on the dropdown selections, only show that particular month/week/day.

Thanks in advance for any assistance.

**shg** · 10-26-2011, 01:05 PM

You could replace the first 167 lines of your code with this:

Please Login or Register  to view this content.

More generally, though, you could add some columns to the right and use AutoFilter -- much faster and much simpler.

**Steffen Thomsen** · 10-26-2011, 01:08 PM

Hi,

I really dont get the build, whats it for?

First thing i noticed with your code is, your doing the same thing over and over.

Build this into a function

Create first a function that hides the select cells on the user input in the first dropdownmenu.

Next create a function that does the case.

The cells that needs to be hidden/visible are at regular intervals, large intervals, but nontheless intervals. So they could be set in a function.

You could build one function or 3 different, one for months, one for weeks and one for days.

Steffen Thomsen

**JonathonHardy** · 10-27-2011, 12:29 PM

Thank you for the suggestions. I've streamlined it enough to work.

**royUK** · 10-27-2011, 01:34 PM

If you are satisfied with the solution(s) provided, please mark your thread as Solved.

How to mark a thread Solved
Go to the first post
Click edit
Click Go Advanced
Just below the word Title you will see a dropdown with the word No prefix.
Change to Solved
Click Save