开启辅助访问 切换到宽版

精易论坛

 找回密码
 注册

QQ登录

只需一步,快速开始

用微信号发送消息登录论坛

新人指南 邀请好友注册 - 我关注人的新帖 教你赚取精币 - 每日签到


求职/招聘- 论坛接单- 开发者大厅

论坛版规 总版规 - 建议/投诉 - 应聘版主 - 精华帖总集 积分说明 - 禁言标准 - 有奖举报

查看: 5881|回复: 20
收起左侧

[已解决] 精易模块5.7版本的一个会革盘的BUG

[复制链接]
结帖率:100% (11/11)
发表于 2017-5-24 20:40:46 | 显示全部楼层 |阅读模式   广东省广州市
本帖最后由 2576605471 于 2017-5-24 23:54 编辑

5.6里的 目录_删除()
1.bmp

5.7里的 目录_删除()
2.bmp


3.bmp

反馈5.7模块的bug:
"目录_删除()" 此命令,在意外情况下,传了个空字符串参数,导致的结果和格盘无异!
我就因为代码测试中,错误导致传了个空字符串,结果命令直接删了我一百G的资料,删了我整个盘的文件,我用了一天一夜来做数据恢复,简直变态.就这样吧,不改正,就是在害人

5.6的命令都没什么问题的,5.7就手贱改了

@zhifu3158
@你凌哥i


评分

参与人数 1精币 +5 收起 理由
项目部002 + 5 感谢反馈,请等待新版更新

查看全部评分

发表于 2023-10-31 04:31:05 | 显示全部楼层   浙江省台州市
111111111111111111111111111111111111
回复 支持 反对

使用道具 举报

发表于 2018-1-14 21:30:21 | 显示全部楼层   北京市北京市
不要呼吁啊
回复 支持 反对

使用道具 举报

发表于 2017-9-22 00:12:34 | 显示全部楼层   浙江省湖州市
学习了,支持支持
回复 支持 反对

使用道具 举报

发表于 2017-8-31 19:37:56 | 显示全部楼层   吉林省长春市
学习了,支持支持
回复 支持 反对

使用道具 举报

结帖率:62% (21/34)
发表于 2017-5-28 12:58:21 | 显示全部楼层   广东省湛江市
2576605471 发表于 2017-5-26 19:15
我是不知道你们改了哪里,但从我能拿到的源码来看,5.6和5.7的改法,根本就是不应该出的错误,

5.6的代码, ...

这是个很严重的bug啊。。。应该在最前面再加两个判断    参数=“” 或者参数=“\” 就返回。这样是不是就可以避免了?
回复 支持 反对

使用道具 举报

结帖率:100% (11/11)

签到天数: 28 天

 楼主| 发表于 2017-5-26 19:15:43 | 显示全部楼层   广东省广州市
zhifu3158 发表于 2017-5-26 16:09
好吧,我明天就回老家村里 去种地去。。。

最后给楼主一个建议,你这不是在反馈问题,而 ...

我是不知道你们改了哪里,但从我能拿到的源码来看,5.6和5.7的改法,根本就是不应该出的错误,

5.6的代码,是先判断是否存在,再+"\",根本没问题

而5.7的,什么也不改,就单单改了,先+"\",再判断是否存在

这样一看,搞得就像故意的一样,我才气的,

另外,关于开源的模块,我表示致敬,我是一直用精易模块,别的都少用,只是这样的更新,让人太寒心了.

我只是一个使用者,我没啥贡献,也不应该去质疑你们无私的工作,但期望的是尽可能严谨点,没其他意思.

另外,回家种田只是胡话,我删了,你别介意
回复 支持 反对

使用道具 举报

结帖率:100% (2/2)

签到天数: 3 天

发表于 2017-5-26 16:09:21 | 显示全部楼层   山东省东营市
2576605471 发表于 2017-5-26 14:39
我反馈快两天了吧,依旧还有人在质疑我的反馈,没一个人能看出问题

我说个不好听的话,要是让这样疏忽的人 ...

  好吧,我明天就回老家村里 去种地去。。。

最后给楼主一个建议,你这不是在反馈问题,而是在抱怨!而且抱怨的前提还是你自己的源码出现不正确操作后导致的问题的抱怨!
我给你个正确反馈BUG 的例子:

1、提出在什么情况下会出现BUG!
2、提出修改意见,最好是给出修改方法!

人家写模块式为了方便大家“偷懒”调用的,为什么要把模块开源?把源码都直接给你了??难道不懂其中的寓意吗?
  
不得不承认开发人员在写这个命令时出现了两个疏忽之处:
第一:在命令说明(备注)处没有警示说明。易语言自带支持库中的“删除目录”中特别备注了“该目录应实际存在,如果目录中存在文件或子目录,将被一并删除,因此使用本命令请千万慎重”,这是疏忽之处,望加上对应备注。。。

第二:没有考虑到参数可能为空的情况,建议将 判断(目录_是否存在)命令放在 判断尾部“\”  的前面!!
(其实这一点谁都没有想到,毕竟无论是使用易语言自身支持库 还是调用精易模块,凡是重要的操作尤其是文本、目录的 删除、清空等操作时,使用者都会非常小心并在使用前判断目录(或者是变量)是否为空,长度是否大于2.。。。这是最基本的源码编写的注意事项。)

我在5.6中反馈的问题是 模块中针对目录地址尾部“\” 的判断方式不完整(其实就是将参数赋值给局部变量不完整)。如果像你这样,参数传递“” ,在5.6中是什么都不执行的,

我也不知道该说什么了。总之建议在调用模块命令之前先看一下源码,最后是将源码直接复制过来并根据自身的需求稍作修改,还有注意自己软件的开发逻辑,如果出现问题,请正确的方式来反馈问题,人家写模块给你, 还是开源的,有没有强迫你用,也没有收你的钱,自己操作失误导致的后果却将责任全部推卸到别人那里,就如同开车时睡着了,出了车祸,就去找汽车厂家理论,要求对方加上“如果司机开车中睡着了防止出车祸的”安全措施。。。。。。。

不说了,再说就是吵架了,总之模块是要再完善下的,还有:我说到做到,明天回老家村里去。。。
回复 支持 反对

使用道具 举报

结帖率:100% (11/11)

签到天数: 28 天

 楼主| 发表于 2017-5-26 14:39:29 | 显示全部楼层   广东省广州市
本帖最后由 2576605471 于 2017-5-26 19:03 编辑

我反馈快两天了吧,依旧还有人在质疑我的反馈,没一个人能看出问题



回复 支持 反对

使用道具 举报

结帖率:100% (11/11)

签到天数: 28 天

 楼主| 发表于 2017-5-26 14:24:55 | 显示全部楼层   广东省广州市
本帖最后由 2576605471 于 2017-5-26 14:30 编辑
zhifu3158 发表于 2017-5-26 11:24
rd /s /q "\"  是删除你的源码所在盘的跟盘,也就是如果你的源码在F 盘 就删除整个F盘,等同格式化,。。 ...


参数 = ""       而不是   空

一开始我语误

空传进来的时候,他不是判断,而是直接 + "\"         对着5.7的源码去看吧

然后接着   目录_是否存在("\")   的结果是真

所以,不会返回,于是革盘悲剧来了,,,非要我解释这么透...让我一个不想解释的人,情何以堪

我本思想


回复 支持 反对

使用道具 举报

结帖率:100% (2/2)

签到天数: 3 天

发表于 2017-5-26 11:24:15 | 显示全部楼层   山东省东营市
2576605471 发表于 2017-5-26 00:21
我本来还想让大家自己找出错误,结果大失所望,没人能看出来,可能是字节集的问题掩盖了,难直接从图片上看 ...

rd /s /q "\"  是删除你的源码所在盘的跟盘,也就是如果你的源码在F 盘 就删除整个F盘,等同格式化,。。
  但是我不明白的是,你怎样才能传进去 空参数??能否将当初你那出现问题的源码发上来,我测试下,帮你看看,毕竟精易模块中 目录_删除() 命令是不允许为空的,也就是说,除非你调用命令的时候给他传一个 “\” 参数。否则命令是不生效的 无法运行的,
       难道你调用命令时传递的 欲删除目录参数是“\”???
回复 支持 反对

使用道具 举报

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

本版积分规则 致发广告者

发布主题 收藏帖子 返回列表

sitemap| 易语言源码| 易语言教程| 易语言论坛| 诚聘英才| 易语言模块| 手机版| 广告投放| 精易论坛
拒绝任何人以任何形式在本论坛发表与中华人民共和国法律相抵触的言论,本站内容均为会员发表,并不代表精易立场!
论坛帖子内容仅用于技术交流学习和研究的目的,严禁用于非法目的,否则造成一切后果自负!如帖子内容侵害到你的权益,请联系我们!
防范网络诈骗,远离网络犯罪 违法和不良信息举报电话0663-3422125,QQ: 800073686,邮箱:800073686@b.qq.com
Powered by Discuz! X3.4 揭阳市揭东区精易科技有限公司 ( 粤ICP备12094385号-1) 粤公网安备 44522102000125 增值电信业务经营许可证 粤B2-20192173

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