Simplify VBA code for hiding Column and range based on a cell value

I have written a VBA code that does 2 things. 1) It hides Column based on a cell value. 2) hides a range of row based on a cell value of another sheet. The issue is I have quite a lot of range of row that needs to be hidden so my code become quite lenghtly for doing 2). The code is working fine but I was wondering if there were a better more efficient way of doing this.

Please Login or Register  to view this content.

You can reduce the amount of code like

Please Login or Register  to view this content.

Hi DFHK,

You could build the part that repeats all the time into a Sub with arguments you can shorten the whole thing to single line for each column


Grtz

Thanks for the hints. However there is a bug preventing to run the macro proerperly. Range "Section1" is in "proposal" worksheet whereas Range "S1Controller" is in resources worksheet. Could it be the problem?

You just need to remove the . from the Section ranges like

Please Login or Register  to view this content.

How about like this

Please Login or Register  to view this content.

Hi FLuff13, Thanks for this, much appreciated! It works like a charm and it is much easier to read! Thanks so much!

Thanks mehmetick, this is much appreciated! I am still trying to get my head around this, but I can see the value of your approach. Thanks!

Glad we could help & thanks for the feedback