Can this code be tidied up? Macro recorder appears to need some organizational skills

I used macro recorder, and this code was the result. Is there a way to shorten this code up? The more I use macro recorder, the more I'm beginning to realize that the code generated from macro recorder can be very sloppy.

By seeing you all work your magic, I can analysis what you did, and hopefully start shortening these myself

Please Login or Register  to view this content.

Hello duugg,

It would help to know what you are doing. Part of the code applies borders to the selected cells and the other part formats column "C" and then selects "C1". What to you want to do exactly?

Perhaps the BorderAround method - have a look at Help. That said, I think it would only do outside borders.

As far as I am aware 'With' only operates on single objects / collections at a time, so you are unable to combine the borders within the 'With' statement.
The macro recorder, whilst giving you the 'idea' of what you want to right is not, as you have found, particularly efficient. For one thing it will never use an array, or loops, both of which could be used to 'condense' this code.
However what you can do if you need to apply the same settings to mulitple objects is something like this:

Please Login or Register  to view this content.

You can replace your Borders code with:

Please Login or Register  to view this content.