Foxtable(狐表)用户栏目专家坐堂 → 执行效率测试


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

主题:执行效率测试

帅哥哟,离线,有人找我吗?
狐狸爸爸
  1楼 | 信息 | 搜索 | 邮箱 | 主页 | UC


加好友 发短信
等级:管理员 帖子:47497 积分:251403 威望:0 精华:91 注册:2008/6/17 17:14:00
  发帖心情 Post By:2014/4/10 15:59:00 [显示全部帖子]

DataColChanged可不止几十秒,除非之前用其他方法算过一次,表中已经有算好的值。

 

在我的i5笔记本,12秒完成:

 

Dim dt As Date =Date.now()
Dim dic As new Dictionary(of String,Double)
Dim nms As List(of String) = DataTables("药品收发记录").GetValues("单据号")
For Each nm As String In nms
    dic.Add(nm,DataTables("药品收发记录").Compute("Sum(成本金额)","单据号 = '" & nm & "'"))
Next
Dim drs As List(of DataRow) = DataTables("药品收发记录").Select("单据号 is not null","单据号")
drs(0)("零售金额") = dic(drs(0)("单据号"))
For i As Integer = 1 To drs.count -1
    If drs(i)("单据号") =  drs(i-1)("单据号") Then
        drs(i)("零售金额") = drs(i-1)("零售金额")
    Else
        drs(i)("零售金额") = dic(drs(i)("单据号"))
    End If
Next
messagebox.show((Date.now - dt).TotalSeconds)

[此贴子已经被作者于2014-4-10 16:01:23编辑过]

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


加好友 发短信
等级:管理员 帖子:47497 积分:251403 威望:0 精华:91 注册:2008/6/17 17:14:00
  发帖心情 Post By:2014/4/10 16:49:00 [显示全部帖子]

哈哈,这样好,1秒不到就可以了。

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


加好友 发短信
等级:管理员 帖子:47497 积分:251403 威望:0 精华:91 注册:2008/6/17 17:14:00
  发帖心情 Post By:2014/4/10 17:08:00 [显示全部帖子]

lsy兄,这个也是赋值到数据表的,而且直接保存了。

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


加好友 发短信
等级:管理员 帖子:47497 积分:251403 威望:0 精华:91 注册:2008/6/17 17:14:00
  发帖心情 Post By:2014/4/10 17:34:00 [显示全部帖子]

呵呵,没错,Replacefor是高效的,但是这里的情况不同,Select只执行一次,用RepalceFor执行的话,是8000多次,自然有极大的差别。

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


加好友 发短信
等级:管理员 帖子:47497 积分:251403 威望:0 精华:91 注册:2008/6/17 17:14:00
  发帖心情 Post By:2014/4/10 17:54:00 [显示全部帖子]

以下是引用don在2014-4-10 17:48:00的发言:
你循环的是表的行数,我循环的是不重复的单据号!

 

不重复的单号有8000多个,所以循环单号,其实执行了8000多次RepalceFor。

我虽然遍历所有行,有5万多次循环,但其实只是执行5万多次的简单赋值而已,所以快很多。

 

往深处考虑,每次RepalceFor都会遍历5万行,所以你的是遍历了8000 * 50000行,远远超过我的5万行。

[此贴子已经被作者于2014-4-10 18:35:12编辑过]

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


加好友 发短信
等级:管理员 帖子:47497 积分:251403 威望:0 精华:91 注册:2008/6/17 17:14:00
  发帖心情 Post By:2014/4/10 18:07:00 [显示全部帖子]

这个我就不懂了,我对于sql不太熟悉。

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


加好友 发短信
等级:管理员 帖子:47497 积分:251403 威望:0 精华:91 注册:2008/6/17 17:14:00
  发帖心情 Post By:2014/4/11 8:39:00 [显示全部帖子]

以下是引用hunanwl在2014-4-10 22:40:00的发言:
狐爸  你是怎么修改的 用1S就搞定,  赐教啊

 

我没有执行,正常执行这种sql语句,就是不到一秒的。


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


加好友 发短信
等级:管理员 帖子:47497 积分:251403 威望:0 精华:91 注册:2008/6/17 17:14:00
  发帖心情 Post By:2014/4/11 8:43:00 [显示全部帖子]

以下是引用don在2014-4-10 20:42:00的发言:
有些不明白:ReplaceFor不是加了筛选条件吗?难道加了筛选也是遍历50000行后,筛选出符合的记录,再统一替换?如是,难怪效率低了。

 

 

其实ReplaceFor就是下面的代码的简写:

 

For Each dr As DataRow In DataTables("xxx").Select("条件")

    dr("列名") =值

next

 

所以你的方案,等于执行了8000多次Select和遍历,而我的方案只执行了一次Select和遍历。

 

我第一次写的代码和你一模一样,效率不行,分析之后,才有现在的代码。

[此贴子已经被作者于2014-4-11 8:45:28编辑过]

 回到顶部