LinkBack URL
About LinkBacks
This code simply formats my sheet but it takes forever and a day to finish the job....is there a more effective way to write this?
(a lot of it has been taken from recording macros and using that code)
Please Login or Register to view this content.
This code simply formats my sheet but it takes forever and a day to finish the job....is there a more effective way to write this?
(a lot of it has been taken from recording macros and using that code)
Please Login or Register to view this content.
Not sure if it would make much difference, depends how big your named ranges are but you can change your loops such as:
toPlease Login or Register to view this content.
Please Login or Register to view this content.
that has sped it up a bit,
how do you change my loop to match your code
My attemptPlease Login or Register to view this content.
doesnt work...(type mismatch error)Please Login or Register to view this content.
Ah, I guess you can't use it with the UCase function.
if Account,AccountAPRP and Comment are not contiguous, using Application.proper(Range("Account,AccountAPRP,Comment").value) will convert them all to the values in the first range
Josie
if at first you don't succeed try doing it the way your wife told you to
JosephP
thanks you have just solved my brain scratcher, I didnt understand why that was happening!!!
have to revert back to my original (but very slow) code from the first post......any other ideas?
you can loop the areas
for ucase you can't do that (and application.upper doesn't exist) so you would be better off using arraysPlease Login or Register to view this content.
for instancePlease Login or Register to view this content.
untested of course but perhaps
Please Login or Register to view this content.
Wow, you have reduced my macro running time from 25 seconds to 1 second!! Legend!
have some rep!
There are currently 1 users browsing this thread. (0 members and 1 guests)
Posting Permissions
Register To Reply
Bookmarks