ExcelHome技术论坛

 找回密码
 免费注册

QQ登录

只需一步,快速开始

快捷登录

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

请教 我这段代码哪儿有问题?

[复制链接]

TA的精华主题

TA的得分主题

发表于 2013-1-22 19:55 | 显示全部楼层 |阅读模式
sub test
    arr=array("AU","CN","CH","HK")

    for each cell in selection

        flag=true

        for i=0 to 4 step 1

            if cell.text = arr(i) then

                flag = false

                exit for

          end if

    next
    if flag  then columns(cell.column).delete

    next

end sub

为什么要运行好几次才能筛选出结果,而且不知道结果对不对

就是不含数组中数据的列,都删除

TA的精华主题

TA的得分主题

发表于 2013-1-26 12:07 | 显示全部楼层
这段代码运行的结果是只要列中的某单元格不是数组中的数据,那么该列就会被删掉。
我理解你的意思应该是:如果某列所有单元格都不是数组中的数据,那么删掉该列。如果是这个意思,最好按照column来控制循环了,而不是按照cell来控制循环了。
不知道说得对不对。

TA的精华主题

TA的得分主题

发表于 2013-1-26 22:01 | 显示全部楼层
本帖最后由 hehex 于 2013-1-26 22:17 编辑

你的代码二处有问题,
arr=array("AU","CN","CH","HK")
说明数组共4个元素,下标号是从0-3
   for i=0 to 4 step 1
当内层循环执行到最后一定会出现下标越界错误,你没发现多半是selection 的最后一列有符合的记录
执行exit for 退出了。但是这个问题还是客观存在的,而且是语法错误。
这是你程序的第一处问题。应该改成 ifor =0 to 3, 或者直接用for i=0 to ubound(arr)。

第二个问题,有逻辑错误,当你找到某单元格包含数组数据时,就应跳过该列。而你目前的代码没有实现这点,因为是单元格遍历整个区域,有可能会循环回该列,而该列下一行的单元格数据如果不在数组内,则该列被误删。因为你的代码不是固定区域只是Selection, 如果手动选取一行或者整个选定区域的列内容都一样,则这逻辑勉强可以通过。
再有,你的选中区域是被不停的进行删除,你的删除列语句放在循环内,有可能造成遍历代码的逻辑错误,因为你的区域有不确定性。不停的删一列,后面左移一列这种情况。
我想到的算法是另外做一个和列数等长的布尔型数组,当循环到某列时,如果该删除,就在该数组位置设定为true,不该删就设false。在遍历过程中不进行删除,确保遍历区域的稳定。
最后针对该数组的记录一次性删除所需要删的列,这样就不会出现多次运行的问题了。

最后,一般删除单元格所在的列最常用的方法是 cell.EntireColumn.Delete。




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

本版积分规则

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

GMT+8, 2024-11-23 05:37 , Processed in 0.035340 second(s), 9 queries , Gzip On, MemCache On.

Powered by Discuz! X3.4

© 1999-2023 Wooffice Inc.

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

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

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