Code is generally slowed considerably by 'activating' and 'selecting'. Most time that is not necessary.
I don't know whether you actually need to activate ALL.xls to start with as I don't know what is in FTALLV however these lines of code
Can be replaced by these (not exactly sure of the name of that second workbook - spelling or extension - so check), avoiding any activation or selection or copy.VBA Code:Range("A11:BD11").Select Range(Selection, Selection.End(xlDown)).Select Selection.Copy
VBA Code:Dim rws As Long With Workbooks("ALL.xls").Sheets("FT") rws = .Range("A11:BD11").End(xlDown).Row - 10 Workbooks("COMPARSION.xls").Sheets("FT COMPARE").Range("B23").Resize(rws, 56).Value = .Range("A11:BD11").Resize(rws).Value End With
Sorry for entering the conversation, but we are always learning. @Peter_SSs can you answer me two questions? 1. I always use .activate instead of .select, because .select doesn't work, do you know why? 2. I didn't know this propoerty of range, "resize", it's great! But, the .value=.value only works if you are copying just the values, right?? Because I always can use range(A1:A3").copy Sheets("sheet2").range("B1:B3"). Is it faster? Or the same?
Thank you!