Foxtable(狐表)用户栏目专家坐堂 → 请求指点代码低效的原因


  共有4649人关注过本帖树形打印复制链接

主题:请求指点代码低效的原因

帅哥哟,离线,有人找我吗?
东坡一剑
  1楼 | QQ | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:小狐 帖子:366 积分:2478 威望:0 精华:0 注册:2013/1/13 23:43:00
请求指点代码低效的原因  发帖心情 Post By:2013/10/24 21:09:00 [只看该作者]

为了根据货品出库计划中每种货品的需求数量在库存中找到每钟货品的最佳的取货位置,生成取货明细以便照单取货,我编写了如下代码。

蓝字部分是插入测试效率的代码;

红字和绿字部分分别是利用集合和利用字典来保存用于生成取货单新增行的代码,算是两种方案吧;

运行得结果是:不管采用哪种方案,在两三百行数据的情况下,前三段代码都在几乎不足0.1秒的时间内运行完毕,效率应该是可以接受的。但是第四段代码足足用了160秒之多,效率实在太低。把帮助文件相关的章节又看了一遍,仍然搞不懂问题所在。请高手赐教!

 

 

If Tables("货品出库计划").Rows.Count = 0 Then
    MessageBox.Show("本次出库信息为空,系统禁止提交!", "提示", MessageBoxButtons.OK, MessageBoxIcon.Warning)
    Return
End If
pause = True


Dim ttt As Date = Date.now
Dim sppp As TimeSpan


'计算本单号各产品的出库总数量
Tables("货品出库计划").filter = "单号 ='" & e.Form.Controls("单号").Text & "' and  单号 Is Not Null"
Dim dic As new Dictionary(Of String,Double)

Dim r As Row
Dim dr As DataRow
Dim v,v1,v2 As Double
Dim key As String
dic.Clear()
For Each r In Tables("货品出库计划").Rows
    If dic.ContainsKey(r("产品条码")) Then
        Continue For
    Else
        v = Tables("货品出库计划").Compute("Sum(数量)","产品条码 ='" & r("产品条码") & "'")
        dic.Add(r("产品条码"),v)
    End If
Next


sppp = Date.now - ttt
MessageBox.Show("第一段:" & sppp.TotalSeconds & "秒")
ttt = Date.Now

 

dr = DataTables("出库计划记录").Find("单号 ='" & e.Form.Controls("单号").text & "'")
If dr Is Nothing Then
    Dim st() As String = {"收货库房","发货单位","发货人","收货人","收货单位","发货库房","出库类型","计划日期"}
    r = Tables("出库计划记录").AddNew()
   
    For Each key In st
        r(key) = e.Form.Controls(key).Value
    Next
    r("单号") = e.Form.Controls("单号").text
    r.Save()
End If

 

sppp = Date.now - ttt
MessageBox.Show("第二段:" & sppp.TotalSeconds & "秒")
ttt = Date.Now


'根据本单各产品的本地取货数量计算取货位置(明细)
DataTables("取货明细").DataRows.Clear()
Dim sts As String() = {"货号","颜色","尺码号","产品条码","产品名称","吊牌价格","库房","库区","库位"}
If dic.Keys.count > 0 Then


    'Dim dic1 As new Dictionary(Of DataRow,Double)  
    Dim lst0,lst1,lst2,lst3,lst4,lst5,lst6,lst7,lst8 As new List(Of String)


    Dim lst As new List(Of Double)
    Dim cmd As  New SQLCommand
    cmd.C
    cmd.CommandText = "Select * From {库存表} Where 库房 ='" & User.Group & "库房'"   'And 产品条码 = '" & key & "'" 

    Dim dt As DataTable
    Dim drs As new List(Of DataRow)
    dt = cmd.ExecuteReader()    
    For Each key In dic.Keys        
        Dim id As String = ""
        v1 = dic(key) '赋值给初始取货数量        
        Do While v1 > 0
            If id = "" Then
                drs = dt.Select("产品条码 = '" & key & "'"  )
            Else
                drs = dt.Select(id & " And 产品条码 = '" & key & "'"  )
            End If
           
            If drs.Count > 0 Then
                v2 = Math.Abs(v1 - drs(0)("库存数量"))
                For i As Integer = 0 To drs.count - 1
                    v2 = Math.Min(v2,Math.Abs(v1 - drs(i)("库存数量")))
                Next
                dr = dt.Find("库存数量 =" & v1 + v2 & " And 产品条码 = '" & key & "'"  )
                If dr IsNot Nothing Then
                    lst0.Add(dr(sts(0)))
                    lst1.Add(dr(sts(1)))
                    lst2.Add(dr(sts(2)))
                    lst3.Add(dr(sts(3)))
                    lst4.Add(dr(sts(4)))
                    lst5.Add(dr(sts(5)))
                    lst6.Add(dr(sts(6)))
                    lst7.Add(dr(sts(7)))
                    lst8.Add(dr(sts(8)))
                    lst.add(v1)
                    'dic1.Add(dr,v1)                                 
                    v1 = 0
                Else
                    dr = dt.Find("库存数量 =" & v1 - v2 & " And 产品条码 = '" & key & "'"  )
                    If dr IsNot Nothing Then
                        lst0.Add(dr(sts(0)))
                        lst1.Add(dr(sts(1)))
                        lst2.Add(dr(sts(2)))
                        lst3.Add(dr(sts(3)))
                        lst4.Add(dr(sts(4)))
                        lst5.Add(dr(sts(5)))
                        lst6.Add(dr(sts(6)))
                        lst7.Add(dr(sts(7)))
                        lst8.Add(dr(sts(8)))
                        lst.add(v1)
                        'dic1.Add(dr,v1)
                         v1 = v1 - dr("库存数量")
                        If id = "" Then
                            id = "[_Identify] <>" & dr("_Identify")
                        Else
                            id = id & "And"
                            id = id & "[_Identify] <>" & dr("_Identify")
                        End If
                    End If
                End If
            End If
        Loop
    Next
   
    sppp = Date.now - ttt
    MessageBox.Show("第三段:" & sppp.TotalSeconds & "秒")
    ttt = Date.Now


If lst.count > 0 Then
DataTables("取货明细").StopRedraw()
For i As Integer = 0 To lst.count - 1
    dr = DataTables("取货明细").AddNew()   
    dr(sts(0)) = lst0(i)
    dr(sts(1)) = lst1(i)
    dr(sts(2)) = lst2(i)
    dr(sts(3)) = lst3(i)
    dr(sts(4)) = lst4(i)
    dr(sts(5)) = lst5(i)
    dr(sts(6)) = lst6(i)
    dr(sts(7)) = lst7(i)
    dr(sts(8)) = lst8(i)
    dr("计划取货数量") = lst(i)
    dr("单号") = e.Form.Controls("单号").text
Next
DataTables("取货明细").ResumeRedraw()
End If

   
    'If dic1.Keys.count > 0 Then
        'DataTables("取货明细").StopRedraw()
        'For Each dr In dic1.keys
            'Dim dr1 As DataRow = DataTables("取货明细").AddNew()    '此处可能有问题
            'For i As Integer = 0 To sts.length - 1
                'dr1(sts(i)) = dr(sts(i))
            'Next
            'dr1("单号") = e.Form.Controls("单号").text
            'dr1("计划取货数量") = dic1(dr)
        'Next
        'DataTables("取货明细").ResumeRedraw()
    'End If
End If

 

sppp = Date.now - ttt
MessageBox.Show("第四段:" & sppp.TotalSeconds & "秒")
ttt = Date.Now

 

DataTables("取货明细").Save()
Forms("出库明细及取货明细").Open()


 回到顶部
帅哥哟,离线,有人找我吗?
有点甜
  2楼 | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:版主 帖子:85326 积分:427815 威望:0 精华:5 注册:2012/10/18 22:13:00
  发帖心情 Post By:2013/10/24 21:15:00 [只看该作者]

 耗时是在addnew上。你直接 addnew(lst.count),然后再分别写入数据。

 这样会快很多。

 回到顶部
帅哥哟,离线,有人找我吗?
y2287958
  3楼 | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:狐神 帖子:4632 积分:33804 威望:0 精华:0 注册:2008/8/31 22:44:00
  发帖心情 Post By:2013/10/24 23:33:00 [只看该作者]

楼上好经验
学习

 回到顶部
帅哥哟,离线,有人找我吗?
东坡一剑
  4楼 | QQ | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:小狐 帖子:366 积分:2478 威望:0 精华:0 注册:2013/1/13 23:43:00
  发帖心情 Post By:2013/10/25 0:10:00 [只看该作者]

我试了,现在耗时124秒,好像还是不够高效。


 回到顶部
帅哥哟,离线,有人找我吗?
y2287958
  5楼 | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:狐神 帖子:4632 积分:33804 威望:0 精华:0 注册:2008/8/31 22:44:00
  发帖心情 Post By:2013/10/25 8:32:00 [只看该作者]

你就第四段代码做个例子传上来
大家看看
俺测试是效率明显提升的
[此贴子已经被作者于2013-10-25 8:33:05编辑过]

 回到顶部
帅哥哟,离线,有人找我吗?
东坡一剑
  6楼 | QQ | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:小狐 帖子:366 积分:2478 威望:0 精华:0 注册:2013/1/13 23:43:00
  发帖心情 Post By:2013/10/25 10:25:00 [只看该作者]

 下载信息  [文件大小:   下载次数: ]
图片点击可在新窗口打开查看点击浏览该文件:效率.zip

例子传上,请诸位大神费心!

在”货品出库计划“窗口点“确定提交”即可测试

打开项目时需要手动连接数据库,数据库一并压缩到包里了。

[此贴子已经被作者于2013-10-25 10:26:33编辑过]

 回到顶部
帅哥哟,离线,有人找我吗?
Bin
  7楼 | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:贵宾 帖子:35433 积分:178524 威望:0 精华:3 注册:2013/3/30 16:36:00
  发帖心情 Post By:2013/10/25 11:11:00 [只看该作者]

循环行赋值效率不高,这个没什么好办法.  可以的话换个方式处理这个问题吧.

 回到顶部
帅哥哟,离线,有人找我吗?
东坡一剑
  8楼 | QQ | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:小狐 帖子:366 积分:2478 威望:0 精华:0 注册:2013/1/13 23:43:00
  发帖心情 Post By:2013/10/25 11:51:00 [只看该作者]

能不能建议一下,像我这种需求,还有什么可行的处理方式吗?


 回到顶部
帅哥哟,离线,有人找我吗?
lihe60
  9楼 | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:狐神 帖子:6831 积分:43213 威望:0 精华:0 注册:2009/3/2 14:07:00
  发帖心情 Post By:2013/10/25 12:01:00 [只看该作者]

没有明白楼主的意图,但有点可以确定:库存表、出库明细、取货明细这三张表是多余的。


 回到顶部
帅哥哟,离线,有人找我吗?
东坡一剑
  10楼 | QQ | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:小狐 帖子:366 积分:2478 威望:0 精华:0 注册:2013/1/13 23:43:00
  发帖心情 Post By:2013/10/25 13:24:00 [只看该作者]

根据货品出库计划中每种货品的需求数量,在库存表中找到每种货品的最佳的取货位置及本位置的取货数量,生成取货明细以便照单取货

这三张表是必须的,而不是多余的。

不管用什么方法能达到这个目的,同时代码的运行要高效!

 

现在的代码,目的达到了,但只要数据量稍微偏大,效率就极低。

 


 回到顶部
总数 15 1 2 下一页