Same advice everyone else asking "speed up VBA macro": work with an in-memory array, not with the worksheet. And when you work with the worksheet, don't use
Select
and
Activate
, especially not in a loop -
it's the single slowest thing you can do in VBA. As you can see, turning off
ScreenUpdating
is a cargo-cultist's performance fix for poorly-written code. Fix the blatant issues, see what other problems
Rubberduck finds, fix them, then get useful feedback on
Code Review. This question isn't really a good fit for Stack Oveflow. —
Mat's Mug just now