在我的excel中,C列总是会有文本,要么是 response
或 resolution
. 我的目标是在此基础上将A:C列分开。如果C列有文本 response
,将A:C列复制到 E:G
否则 A:C
到 I:K
我现在使用以下代码。
Sub SLACalc()
Dim DTA As Workbook
Dim SLADATA As Worksheet
Set DTA = Excel.Workbooks("main.xlsm")
Set SLADATA = DTA.Worksheets("SLA DATA")
For i = 2 To SLADATA.Cells(Rows.Count, "A").End(xlUp).Row
If InStr(Cells(i, "C").Value, "response") > 0 Then
SLADATA.Cells(i, "E").Value = SLADATA.Cells(i, "A").Value
SLADATA.Cells(i, "F").Value = SLADATA.Cells(i, "B").Value
SLADATA.Cells(i, "G").Value = SLADATA.Cells(i, "C").Value
Else
SLADATA.Cells(i, "I").Value = SLADATA.Cells(i, "A").Value
SLADATA.Cells(i, "J").Value = SLADATA.Cells(i, "B").Value
SLADATA.Cells(i, "K").Value = SLADATA.Cells(i, "C").Value
End If
Next i
End Sub
当我的行数较少的时候,这很好用 A:C
. 现在,我有接近20,000行和面临很多性能问题与Excel。有没有什么办法,我可以改进代码,使其运行速度更快。
Assuming you want to split the table on the same row as per you code
First,
你可以减少你的循环代码,比如
For i = 2 To SLADATA.Cells(Rows.Count, "A").End(xlUp).Row
If InStr(Cells(i, "C").Value, "response") > 0 Then
SLADATA.Range(Cells(i, "E"), Cells(i, "G")).Value = SLADATA.Range(Cells(i, "A"), Cells(i, "C")).Value
Else
SLADATA.Range(Cells(i, "I"), Cells(i, "K")).Value = SLADATA.Range(Cells(i, "A"), Cells(i, "C")).Value
End If
Next i
Second
试试阵列。阵列有助于大幅缩短处理时间
Sub SLACalc2()
Dim DTA As Workbook
Dim SLADATA As Worksheet
Set DTA = Excel.Workbooks("main.xlsm")
Set SLADATA = DTA.Worksheets("SLA DATA")
LRow = SLADATA.Cells(Rows.Count, "A").End(xlUp).Row
DataArr = SLADATA.Range("A2:C" & LRow).Value
For i = 1 To UBound(DataArr)
If Application.Index(DataArr, i, 3) = "response" Then
SLADATA.Range(Cells(i + 1, "E"), Cells(i + 1, "G")).Value = Application.Index(DataArr, i)
Else
SLADATA.Range(Cells(i + 1, "I"), Cells(i + 1, "K")).Value = Application.Index(DataArr, i)
End If
Next i
End Sub
有了 该计时器 ;我可以检查处理时间。第一种方法更快。可能是因为,它避免了从数组中存储和检索数据。
但是,如果你只是想要单独的表,就像Ron Rosenfeld在他对问题的评论中所建议的那样,最好使用自动过滤。它的工作速度会比数组快。
Sub Macro1()
Dim DataRng As Range
Set DataRng = Range("A1:C" & Cells(Rows.Count, "A").End(xlUp).Row)
DataRng.AutoFilter Field:=3, Criteria1:="=*response*"
DataRng.Cells.SpecialCells(xlCellTypeVisible).Copy
Range("E1").Select
ActiveSheet.Paste
Application.CutCopyMode = False
DataRng.AutoFilter Field:=3, Criteria1:="=*resolution*"
DataRng.Cells.SpecialCells(xlCellTypeVisible).Copy
Range("I1").Select
ActiveSheet.Paste
Application.CutCopyMode = False
ActiveSheet.ShowAllData
End Sub