Excel精英培训网

 找回密码
 注册
数据透视表40+个常用小技巧,让你一次学会!
查看: 2936|回复: 8

[已解决]简化代码

[复制链接]
发表于 2014-12-22 23:10 | 显示全部楼层 |阅读模式
本帖最后由 张雄友 于 2014-12-23 19:03 编辑

简化代码 ,请帮用简化代码实现F1:I1。
  1. Sub 执行()
  2. [F1:I1].ClearContents
  3. Call 交飞人数
  4. Call 上班时间小于8小时人数
  5. Call 达标人数
  6. Call 不达标人数
  7. End Sub

  8. Sub 交飞人数()
  9. Dim arr, i&, m&
  10. arr = Range("A1:D" & Cells(Rows.Count, 1).End(3).Row)
  11.     For i = 2 To UBound(arr)
  12.         If Len(arr(i, 3)) Then
  13.         m = m + 1
  14.         End If
  15.    Next
  16. [F1] = "交飞人数有" & m & "人"
  17. End Sub

  18. Sub 上班时间小于8小时人数()
  19. Dim arr, i&, k&
  20. arr = Range("A1:D" & Cells(Rows.Count, 1).End(3).Row)
  21.     For i = 2 To UBound(arr)
  22.         If arr(i, 3) < 8 Then
  23.         k = k + 1
  24.         End If
  25.    Next
  26. [G1] = "上班时间小于8小时有" & k & "人"
  27. End Sub

  28. Sub 达标人数() '平均时薪11以上而且上班时间大于等于8,就是达标!
  29. Dim arr, i&, x&
  30. arr = Range("A1:D" & Cells(Rows.Count, 1).End(3).Row)
  31.     For i = 2 To UBound(arr)
  32.         If arr(i, 3) >= 8 And arr(i, 4) >= 11 Then
  33.         x = x + 1
  34.         End If
  35.    Next
  36. [H1] = "达标人数有" & x & "人"
  37. End Sub

  38. Sub 不达标人数() '平均时薪11以下而且上班时间大于等于8,就是不达标!
  39. Dim arr, i&, y&
  40. arr = Range("A1:D" & Cells(Rows.Count, 1).End(3).Row)
  41.     For i = 2 To UBound(arr)
  42.         If arr(i, 3) >= 8 And arr(i, 4) < 11 Then
  43.         y = y + 1
  44.         End If
  45.    Next
  46. [I1] = "不达标人数有" & y & "人"
  47. End Sub

  48. '2+2+16=20
复制代码
最佳答案
2014-12-23 14:48
楼上的逻辑关系有点乱,结果有误,应该是这样:
  1. Sub 执行()
  2.     [F1:I1].ClearContents
  3.     Dim arr, i&, m&, K&, x&, y&
  4.     arr = Range("A1:D" & Cells(Rows.Count, 1).End(3).Row)
  5.       For i = 2 To UBound(arr)
  6.           If Len(arr(i, 3)) Then
  7.               m = m + 1
  8.               If arr(i, 3) < 8 Then
  9.                   K = K + 1
  10.               Else
  11.                   If arr(i, 4) >= 11 Then x = x + 1 Else y = y + 1
  12.               End If
  13.           End If
  14.       Next
  15.     [F1] = "交飞人数有" & m & "人"
  16.     [G1] = "上班时间小于8小时有" & K & "人"
  17.     [H1] = "达标人数有" & x & "人"
  18.     [I1] = "不达标人数有" & y & "人"
  19. End Sub
复制代码

简化代码.rar

11.86 KB, 下载次数: 11

excel精英培训的微信平台,每天都会发送excel学习教程和资料。扫一扫明天就可以收到新教程
 楼主| 发表于 2014-12-23 06:38 | 显示全部楼层
回复

使用道具 举报

发表于 2014-12-23 08:40 | 显示全部楼层
  1. =COUNTA(C2:C21)
  2. =COUNTIF(C2:C21,"<8")
  3. =SUMPRODUCT((C2:C21>=8)*(D2:D21>=11))
  4. =SUMPRODUCT((C2:C21>=8)*(C2:C21<11))
复制代码
回复

使用道具 举报

发表于 2014-12-23 08:46 | 显示全部楼层
  1. Sub 执行()
  2. [F1:I1].ClearContents
  3.   Dim arr, i&, m&, K&, x&, y&
  4.   arr = Range("A1:D" & Cells(Rows.Count, 1).End(3).Row)
  5.     For i = 2 To UBound(arr)
  6.         If Len(arr(i, 3)) Then
  7.         m = m + 1
  8.         ElseIf arr(i, 3) < 8 Then K = K + 1
  9.         Else
  10.         If arr(i, 3) >= 8 And arr(i, 4) >= 11 Then
  11.         x = x + 1
  12.         Else
  13.         If arr(i, 3) >= 8 And arr(i, 4) < 11 Then y = y + 1
  14.         End If
  15.         End If
  16.     Next
  17. [F1] = "交飞人数有" & m & "人"
  18. [G1] = "上班时间小于8小时有" & K & "人"
  19. [H1] = "达标人数有" & x & "人"
  20. [I1] = "不达标人数有" & y & "人"
  21. End Sub
复制代码
回复

使用道具 举报

发表于 2014-12-23 14:48 | 显示全部楼层    本楼为最佳答案   
楼上的逻辑关系有点乱,结果有误,应该是这样:
  1. Sub 执行()
  2.     [F1:I1].ClearContents
  3.     Dim arr, i&, m&, K&, x&, y&
  4.     arr = Range("A1:D" & Cells(Rows.Count, 1).End(3).Row)
  5.       For i = 2 To UBound(arr)
  6.           If Len(arr(i, 3)) Then
  7.               m = m + 1
  8.               If arr(i, 3) < 8 Then
  9.                   K = K + 1
  10.               Else
  11.                   If arr(i, 4) >= 11 Then x = x + 1 Else y = y + 1
  12.               End If
  13.           End If
  14.       Next
  15.     [F1] = "交飞人数有" & m & "人"
  16.     [G1] = "上班时间小于8小时有" & K & "人"
  17.     [H1] = "达标人数有" & x & "人"
  18.     [I1] = "不达标人数有" & y & "人"
  19. End Sub
复制代码
回复

使用道具 举报

发表于 2014-12-23 15:08 | 显示全部楼层
或者直接套用3楼公式:
  1. Sub tt()
  2.     Dim Rng1 As Range, Rng2 As Range, m&, K&, x&, y&
  3.     [F1:I1].ClearContents
  4.     Set Rng1 = Range("c2:c" & [c65536].End(3).Row)
  5.     Set Rng2 = Range("d2:d" & [d65536].End(3).Row)
  6.     m = Application.WorksheetFunction.CountA(Rng1)
  7.     K = Application.WorksheetFunction.CountIf(Rng1, "<8")
  8.     x = Application.WorksheetFunction.CountIfs(Rng1, ">= 8", Rng2, ">= 11")
  9.     y = Application.WorksheetFunction.CountIfs(Rng1, ">= 8", Rng2, "< 11")
  10.     [F1] = "交飞人数有" & m & "人"
  11.     [G1] = "上班时间小于8小时有" & K & "人"
  12.     [H1] = "达标人数有" & x & "人"
  13.     [I1] = "不达标人数有" & y & "人"
  14. End Sub
复制代码
回复

使用道具 举报

发表于 2014-12-23 15:12 | 显示全部楼层
用公式多方便啊
回复

使用道具 举报

 楼主| 发表于 2014-12-23 18:50 | 显示全部楼层
grf1973 发表于 2014-12-23 14:48
楼上的逻辑关系有点乱,结果有误,应该是这样:

If arr(i, 4) >= 11 And arr(i, 3) >= 8 Then x = x + 1 Else y = y + 1

第 11 句怎么不用红色代码???
回复

使用道具 举报

发表于 2014-12-23 19:11 | 显示全部楼层
因为在if arr(i,3)<8.....else....的大循环里面,else后面的语句就包含了arr(i,3)>=8的条件
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册

本版积分规则

小黑屋|手机版|Archiver|Excel精英培训 ( 豫ICP备11015029号 )

GMT+8, 2024-4-27 06:43 , Processed in 0.388693 second(s), 10 queries , Gzip On, Yac On.

Powered by Discuz! X3.4

Copyright © 2001-2020, Tencent Cloud.

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