Macron running extremely slow after update to Excel 2016

Originally Posted by kalbasiatka

not use "select" and move on arrays

Thanks for youre reply!

How do you mean?

All the best,
Martin

I do not know if this will totally fix your problem (seems unlikely), but it should reduce running time.
This is not tested. Suggest you try this in a copy of your file first!

Please Login or Register  to view this content.

replaced with this notation

Please Login or Register  to view this content.

and replaced Cells(x,y) with .Cells(x,y)

Please Login or Register  to view this content.

Originally Posted by Kevin#

I do not know if this will totally fix your problem (seems unlikely), but it should reduce running time.
This is not tested. Suggest you try this in a copy of your file first!

Please Login or Register  to view this content.

replaced with this notation

Please Login or Register  to view this content.

and replaced Cells(x,y) with .Cells(x,y)

Please Login or Register  to view this content.

Hi Kevin,

Thanks for showing an interest in my problem. The new code fails, with the error message "Object doesn't support this property or method". It stops at

Please Login or Register  to view this content.

Any ideas?

Thanks,
Martin

oops - my error. Put the "." in the wrong place
try this

Please Login or Register  to view this content.

Originally Posted by Kevin#

oops - my error. Put the "." in the wrong place
try this

Please Login or Register  to view this content.

Thanks!!

Works now. The macro is running at least 10 times as faster now :-)

Great.

You usually do not need to select ranges in VBA. Just tell vba what to do with things. It can usually do that without having to "pick them up"

The "Select" command is very slow.
Your code contained "Select" inside big loops, so the code was "Selecting" every time it passed through the loop.
I would have expected this issue to have been there before you upgraded to Excel2016 too.

Please mark the thread as solved