![]() |
#8
|
||||
|
||||
![]() Quote:
Yes, poorly tested. Solved (hopefully) by the addition of: Code:
Set SicHrs = Union(SicHrs, DataBody.Columns(3).Cells(DataBody.Rows.Count).Offset(1)) As to timing, well, I'd expect my offering to be hundreds of times slower - the two codes were designed to handle different scenarios. In general, I would expect the time taken iterating and testing every cell in a contiguous range as opposed to iterating and not testing every cell in a non-contiguous range such as AnnHrs to be slower however, how much slower would be dependent on the proportion of cells testing positive for action; If only 3 cells in 30k cells needed action I suspect the non-contiguous range method would be faster than if 10k cells needed action. I haven't tested. |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
macro to move cells | ewso | Excel Programming | 1 | 10-11-2017 12:23 AM |
![]() |
JayBird24 | Word VBA | 1 | 08-19-2016 04:26 PM |
How to move data - Macro not working | ecarter312 | Excel Programming | 1 | 08-10-2016 11:26 PM |
![]() |
Phil H | Excel Programming | 3 | 06-18-2015 01:20 PM |
Macro - Can I move information from a website and excel to word? | redzan | Word VBA | 1 | 03-13-2013 07:39 AM |