ExcelHome技术论坛

 找回密码
 免费注册

QQ登录

只需一步,快速开始

快捷登录

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

[求助] 怎么优化代码

[复制链接]

TA的精华主题

TA的得分主题

发表于 2019-9-11 11:07 | 显示全部楼层 |阅读模式
在一个工作簿里有两张表;sheet2用VBA公式取销量等材料表的数据;代码如下;
这三段怎么循环?
如果要cells里的列+18循环到363后结束?;从E循环到相应的次数后结束?怎么做循环公式?怎么优化代码



Sheet2.Cells(4, 3) = "=SUMPRODUCT((C$3=销量等材料!$ad$3:$ad$8000)*($A4=销量等材料!$z$3:$z$8000)*(销量等材料!$e$3:$e$8000))"
    Range("C4").Select
    Selection.AutoFill Destination:=Range("C4:t4"), Type:=xlFillDefault
    Range("C4:t4").Select
    Selection.AutoFill Destination:=Range("C4:t203"), Type:=xlFillDefault
    Range("C4:t203").Select
    Selection.Copy '复制
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False '数值化
    Application.CutCopyMode = False 'ESC

Sheet2.Cells(4, 21) = "=SUMPRODUCT((C$3=销量等材料!$ad$3:$ad$8000)*($A4=销量等材料!$z$3:$z$8000)*(销量等材料!$f$3:$f$8000))"
    Range("u4").Select
    Selection.AutoFill Destination:=Range("u4:al4"), Type:=xlFillDefault
    Range("u4:al4").Select
    Selection.AutoFill Destination:=Range("u4:al203"), Type:=xlFillDefault
    Range("u4:al203").Select
    Selection.Copy '复制
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False '数值化
    Application.CutCopyMode = False 'ESC

Sheet2.Cells(4, 39) = "=SUMPRODUCT((C$3=销量等材料!$ad$3:$ad$8000)*($A4=销量等材料!$z$3:$z$8000)*(销量等材料!$g$3:$g$8000))"
     Range("am4").Select
     Selection.AutoFill Destination:=Range("am4:bd4"), Type:=xlFillDefault
     Range("am4:bd4").Select
     Selection.AutoFill Destination:=Range("am4:bd203"), Type:=xlFillDefault
     Range("am4:bd203").Select
     Selection.Copy '复制
     Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
     :=False, Transpose:=False '数值化
     Application.CutCopyMode = False 'ESC

Sheet2.Cells(4, 57) = "=SUMPRODUCT((C$3=销量等材料!$ad$3:$ad$8000)*($A4=销量等材料!$z$3:$z$8000)*(销量等材料!$h$3:$h$8000))"
     Range("be4").Select
     Selection.AutoFill Destination:=Range("be4:bv4"), Type:=xlFillDefault
     Range("be4:bv4").Select
     Selection.AutoFill Destination:=Range("be4:bv203"), Type:=xlFillDefault
     Range("be4:bv203").Select
     Selection.Copy '复制
     Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
     :=False, Transpose:=False '数值化
     Application.CutCopyMode = False 'ESC

汇总数据.rar

113.09 KB, 下载次数: 15

TA的精华主题

TA的得分主题

发表于 2019-9-11 12:35 | 显示全部楼层
[广告] VBA代码宝 - VBA编程加强工具 · VBA代码随查随用  · 内置多项VBA编程加强工具       ★ 免费下载 ★      ★使用手册
for r=3 to 363 step 18 这样,至于你的列变化是在公式里,那么需要把你的公式改成R1C1引用格式,循环C后面的数字就是了。

TA的精华主题

TA的得分主题

发表于 2019-9-11 14:47 | 显示全部楼层
[广告] Excel易用宝 - 提升Excel的操作效率 · Excel / WPS表格插件       ★免费下载 ★       ★ 使用帮助
你的代码完全是录制的宏,看起来太累
不如你整理一下基础数据,说明数据运算方法,同时举个例子,别人帮你重新写代码。

TA的精华主题

TA的得分主题

发表于 2019-9-11 15:31 | 显示全部楼层
本帖最后由 yylucke 于 2019-9-11 15:54 编辑

问题:
1、既然动用了VBA,尽量不要用函数,那样工作表运算奇慢,《销量等材料》表中检查校验最好换成代码完成。
2、《品名资料》表中貌似有问题:产品1难道只能对应区域1?有无可能产品1对应区域2、区域3等,如果是,那你的《销量等材料》表中检查校验用vlookup函数就是错误的结果。
3、《品名资料》表中“区域”是恒定的?还是会动态增加的?
4、《销量等材料》表中Z列是录入的,还是从《品名资料》表中抓取的?
5、预留字段做什么?既然用VBA了,那么结果表《销量等材料提取》可以动态生成,没必要预留,否则造成《销量等材料提取》爆大无比。
6、看了你的计算过程,完全可以用函数解决(前提是:你不怕运算奇慢无比,不怕你的电脑宕机)
C4单元格公式,拖拽填充
=SUMPRODUCT((C$3=销量等材料!$AD$3:$AD$8000)*($A4=销量等材料!$Z$3:$Z$8000)*(OFFSET(销量等材料!$A$3,0,MATCH(C$2,销量等材料!$2:$2,0)-1,7998,1)))



TA的精华主题

TA的得分主题

 楼主| 发表于 2019-10-2 09:43 | 显示全部楼层
[广告] VBA代码宝 - VBA编程加强工具 · VBA代码随查随用  · 内置多项VBA编程加强工具       ★ 免费下载 ★      ★使用手册
yylucke 发表于 2019-9-11 15:31
问题:
1、既然动用了VBA,尽量不要用函数,那样工作表运算奇慢,《销量等材料》表中检查校验最好换成代码 ...

就是把《销量等材料》表的数据转化为提取表里的数;重新写个VBA怎么弄?公式弄的话因为数据大很慢的

TA的精华主题

TA的得分主题

 楼主| 发表于 2019-10-27 11:32 | 显示全部楼层
machunze 发表于 2019-9-11 12:35
for r=3 to 363 step 18 这样,至于你的列变化是在公式里,那么需要把你的公式改成R1C1引用格式,循环C后面 ...

求解怎么破

TA的精华主题

TA的得分主题

发表于 2019-10-28 16:32 | 显示全部楼层

一般思路不按你那么破,用字典,下面是个简单的程序,仍然有点费时间。
Sub 录入销量()


If MsgBox("  录入销量(箱) " & vbNewLine & "  这只要一点时间", vbYesNo) <> vbYes Then Exit Sub

Dim d As Object, i&, j&
Set d = CreateObject("scripting.dictionary")
With Sheets("销量等材料")
    For i = 3 To .[a65536].End(xlUp).Row
        For j = 4 To 25
            d(.Cells(i, 1).Value & .Cells(2, j).Value & .Cells(i, 26).Value) = _
             d(.Cells(i, 1).Value & .Cells(2, j).Value & .Cells(i, 26).Value) + .Cells(i, j).Value
        Next
    Next
End With
            
Dim k&, p&
With Sheets("销量等材料提取")
    For k = 4 To .[a65536].End(xlUp).Row
        For p = 3 To .[iiv2].End(xlToLeft).Column
            If d.exists(.Cells(3, p).Value & .Cells(2, p).Value & .Cells(k, 1).Value) Then
                .Cells(k, p) = d(.Cells(3, p).Value & .Cells(2, p).Value & .Cells(k, 1).Value)
            Else
                .Cells(k, p) = 0
            End If
        Next
    Next
End With

MsgBox "  导入结束注意核对"
End Sub
您需要登录后才可以回帖 登录 | 免费注册

本版积分规则

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

GMT+8, 2024-4-27 13:05 , Processed in 0.029827 second(s), 12 queries , Gzip On, MemCache On.

Powered by Discuz! X3.4

© 1999-2023 Wooffice Inc.

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

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

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