Simplify or clean a macro that sets the interior color of a non-contiguous range

[TrafficGuy]

Hi folks, I’ve gotten great help here before and am hoping that you can help me again. I have a piece of code that sets the interior color of a non-contiguous range of cells to green, 35. It works but I’m sure it could be a lot cleaner, I just don’t have the experience to make it cleaner. This piece of code runs within a looping procedure that loops through several worksheets. MyWrkSht is a variable passed to the procedure that represents the worksheet name, column is a variable that represents a column number that the procedure determines is the correct column to set the interior color. So, any simplified code would need to be able to use these two variables. Thanks in advance for the help.

Please Login or Register  to view this content.

[Domski]

I don't see a pattern that you could use in a loop so other than creating an array of the row numbers and looping through that I don't see much that can be changed.

Dom

[JBeaucaire]

Not sure this is really much better than what you had originally. It's still the same number of iterations, just less lines of code, and some would find this particular syntax much harder to read.

Please Login or Register  to view this content.

[shg]

You could select the cells of interest, enter a name in the Names box (left of the formula bar), and then

Please Login or Register  to view this content.

[JBeaucaire]

And the points for a great example that made me chuckle today goes to: shg