ExcelHome技术论坛

 找回密码
 免费注册

QQ登录

只需一步,快速开始

快捷登录

搜索
EH技术汇-专业的职场技能充电站 妙哉!函数段子手趣味讲函数 Excel服务器-会Excel,做管理系统 效率神器,一键搞定繁琐工作
HR薪酬管理数字化实战 Excel 2021函数公式学习大典 Excel数据透视表实战秘技 打造核心竞争力的职场宝典
让更多数据处理,一键完成 数据工作者的案头书 免费直播课集锦 ExcelHome出品 - VBA代码宝免费下载
用ChatGPT与VBA一键搞定Excel WPS表格从入门到精通 Excel VBA经典代码实践指南
查看: 2405|回复: 11

[求助] 跪求各位大侠,我的代码太啰嗦,运行太慢,请帮忙优化

[复制链接]

TA的精华主题

TA的得分主题

发表于 2015-11-5 09:43 | 显示全部楼层 |阅读模式
以下是代码 ,
附有文件。

Sub 写入多工作簿()
    Dim mypath$, myfile$, AK As Workbook, aRow%, tRow%, i As Integer, sht As Worksheet, rt, J As Long
    Application.ScreenUpdating = False
    mypath = ThisWorkbook.Path & "\"
       myfile = Dir(mypath & "*.xls")
       Do While myfile <> ""
          If myfile <> ThisWorkbook.Name Then
             Set AK = Workbooks.Open(mypath & myfile)
              Application.DisplayAlerts = False
                For Each sht In Worksheets
                If sht.Name Like "*(*左右*)" Then sht.Activate
                Next
                Application.DisplayAlerts = True
                  rt = ActiveSheet.Range("R65536").End(xlUp).Row  '以下21行是删除赔率的最大最小值,减少大数据的负面干扰.
                  For J = rt To 7 Step -1
                  If Range("i" & J) >= 1.06 Then
                  Rows(J).Delete
                  End If
                  If Range("i" & J) <= 0.85 Then
                  Rows(J).Delete
                  End If
                  If Range("j" & J) > 1.12 Then
                  Rows(J).Delete
                  End If
                  If Range("j" & J) < 0.69 Then
                  Rows(J).Delete
                  End If
                  If Range("k" & J) >= 1.06 Then
                  Rows(J).Delete
                  End If
                  If Range("k" & J) <= 0.85 Then
                  Rows(J).Delete
                  End If
                  If Range("h" & J) < [H5] Then
                  Rows(J).Delete
                  End If
                  If [H5] < 0.92 And Range("h" & J) >= 0.96 Then
                  Rows(J).Delete
                  Else
                  If [H5] > 0.92 And Range("h" & J) >= 0.97 Then
                  Rows(J).Delete
                  End If
                  End If
                  Next J
             Select Case ActiveSheet.[A9].CurrentRegion.Rows.Count
             Case Is > 50
             lr = Range("a65536").End(xlUp).Row + 1
             mr = Range("U65536").End(xlUp).Row
        Workbooks("布莱克本VS维冈竞技(英冠)欧洲数据2014.9.2").Sheets(2).Range("I7:Q" & mr).Copy
             ActiveSheet.Range("I7:Q" & mr).Select
             ActiveSheet.Paste
        'Workbooks("布莱克本VS维冈竞技(英冠)欧洲数据2014.9.2").Sheets(2).Range("L7:N350").Copy
            ' ActiveSheet.Range("L7:N350").Select
            ' ActiveSheet.Paste
        Workbooks("布莱克本VS维冈竞技(英冠)欧洲数据2014.9.2").Sheets(2).Range("V7:AD" & mr).Copy
             ActiveSheet.Range("V7:AD" & mr).Select
             ActiveSheet.Paste
        Workbooks("布莱克本VS维冈竞技(英冠)欧洲数据2014.9.2").Sheets(2).Range("A391:AL473").Copy
             ActiveSheet.Range("A" & lr).Select
             ActiveSheet.Paste
        ActiveSheet.Range("b65536").End(xlUp).Offset(-61, 0).Resize(1, 3).Copy
             ActiveSheet.[e5:g5].Select
             ActiveSheet.Paste
             Selection.PasteSpecial Paste:=xlPasteValues
        ActiveSheet.Range("b65536").End(xlUp).Offset(-61, 23).Resize(1, 3).Copy
             ActiveSheet.[e6:g6].Select
             ActiveSheet.Paste
             Selection.PasteSpecial Paste:=xlPasteValues
        ActiveSheet.Range("b65536").End(xlUp).Offset(-60, 0).Resize(1, 3).Copy
             ActiveSheet.[I5:K5].Select
             ActiveSheet.Paste
             Selection.PasteSpecial Paste:=xlPasteValues
             ActiveSheet.Cells.Columns("A:A").ColumnWidth = 13
             ActiveSheet.Cells.Columns("B:AD").ColumnWidth = 6.18
             Workbooks(myfile).Close True
             ActiveWorkbook.Save
             Case Is < 50
        Workbooks("布莱克本VS维冈竞技(英冠)欧洲数据2014.9.2").Sheets(3).Range("A20:BA230").Copy
             ActiveSheet.Range("A20:BA230").Select
             ActiveSheet.Paste
             Workbooks(myfile).Close True
             ActiveWorkbook.Save
             End Select
           End If
          myfile = Dir
       Loop
   Application.ScreenUpdating = True
   MsgBox "数据更新完毕!", 164, "提示"
End Sub



布莱克本VS维冈竞技(英冠)欧洲数据2014.9.2.zip

154.02 KB, 下载次数: 46

TA的精华主题

TA的得分主题

发表于 2015-11-5 10:08

TA的精华主题

TA的得分主题

发表于 2015-11-5 13:21 | 显示全部楼层
[广告] VBA代码宝 - VBA编程加强工具 · VBA代码随查随用  · 内置多项VBA编程加强工具       ★ 免费下载 ★      ★使用手册
建议使用数组吧,另外删除行,使用一次性删除

TA的精华主题

TA的得分主题

 楼主| 发表于 2015-11-5 17:01 | 显示全部楼层
liulang0808 发表于 2015-11-5 13:21
建议使用数组吧,另外删除行,使用一次性删除

Excel 跪求各位大侠,我的代码太啰嗦,运行太慢,请帮忙优化-ExcelVBA程序开发-ExcelHome技术论坛 -
http://club.excelhome.net/thread-1239709-1-1.html

TA的精华主题

TA的得分主题

发表于 2015-11-5 21:35 | 显示全部楼层
表格有点乱,请详细说明要求,最好模拟结果。

TA的精华主题

TA的得分主题

 楼主| 发表于 2015-11-5 21:39 | 显示全部楼层

TA的精华主题

TA的得分主题

 楼主| 发表于 2015-11-5 21:40 | 显示全部楼层
hlly888 发表于 2015-11-5 21:35
表格有点乱,请详细说明要求,最好模拟结果。

关键就是在多处复制时的啰嗦,最好用数组,但是动态数组我不会。

TA的精华主题

TA的得分主题

发表于 2015-11-5 21:45 | 显示全部楼层
[广告] VBA代码宝 - VBA编程加强工具 · VBA代码随查随用  · 内置多项VBA编程加强工具       ★ 免费下载 ★      ★使用手册
SYSMEX530 发表于 2015-11-5 21:39
我自己写这代码运行太慢,请帮忙优化。http://club.excelhome.net/thread-1239709-1-1.html

分析代码很头痛,请详细说明要求

TA的精华主题

TA的得分主题

 楼主| 发表于 2015-11-5 21:49 | 显示全部楼层
hlly888 发表于 2015-11-5 21:45
分析代码很头痛,请详细说明要求

我用到 COPY 高达7处,我想把它优化后,提高代码运行速度。

TA的精华主题

TA的得分主题

发表于 2015-11-5 22:12 | 显示全部楼层
(1)以下這個循環是沒有意義的。若有兩個以上的工作表符合條件,只有最後一個工作表會被激活。
For Each sht In Worksheets
        If sht.Name Like "*(*左右*)" Then sht.Activate
Next
(2)下列這些IF重複,應該改成用一個IF處理。例如If Range("i" & J) >= 1.06 Or Range("i" & J) <= 0.85 Or ....
If Range("i" & J) >= 1.06 Then
        Rows(J).Delete
End If
If Range("i" & J) <= 0.85 Then
        Rows(J).Delete
End If
您需要登录后才可以回帖 登录 | 免费注册

本版积分规则

手机版|关于我们|联系我们|ExcelHome

GMT+8, 2024-12-27 11:06 , Processed in 0.043567 second(s), 12 queries , Gzip On, MemCache On.

Powered by Discuz! X3.4

© 1999-2023 Wooffice Inc.

沪公网安备 31011702000001号 沪ICP备11019229号-2

本论坛言论纯属发表者个人意见,任何违反国家相关法律的言论,本站将协助国家相关部门追究发言者责任!     本站特聘法律顾问:李志群律师

快速回复 返回顶部 返回列表