分享
 
 
 

半天时间找到原因却仍未解决的Bug

王朝other·作者佚名  2006-12-25
窄屏简体版  字體: |||超大  

半天时间找到原因却仍未解决的Bug

在一个巨大的数据文件作为输入时, 出现了一个assert断言错误.

花了半天时间跟踪代码, 费尽心思找到了错误的来源,

可是一时间也没有解决的办法, 只能打个临时补丁.

类里面有个成员变量

std::ofstream m_ofs;

在某个方法里面的断言警告:

assert(!m_ofs);

流操作之后都有一个状态判断:

if (!m_ofs)

{

log...

m_ofs.close();

return;

}

为什么会有错误的流状态?

但是open()操作后判断失败没有关闭流.

m_ofs.open(sPath);

if (!m_of)

{

log...

return;

}

表面上看是ofstream::open()打开文件出错了.

open()出错时只写了一条日志, 没做处理.

这个ofstream对象m_ofs不是临时对象, 而是类内嵌对象.

下一次使用时, 发现m_ofs处于已打开状态, 而流状态是错误的,

以为open()失败后文件会处于关闭状态, 看来是错误的.

open()失败后加了一个close()试了试,还是有流错误。

open()为什么会出错? strerror(errno)显示"No error".

以为是errno没有正确指示错误, 再跟踪进入open()代码, 发现open()其实是成功了.

m_ofs是在open()之前就已经fail了(查看内部状态State=2, 即failbit位).

一定是上次close()后failbit并没有清除.

close()成功情况下, 会执行clear(), 但有可能close()失败.

添加clear()在所有close()之前.

可是failbit象是凭空出现, clear()并没有起作用.

回忆历史代码, 原来只有临时变量的流, 用完就自动关闭了.

而为了对付频繁的同一文件打开, 流对象改为了内部成员变量.

而流的宿主类间接地是另一个类的成员变量, 该类是某stl容器类的元素.

问题就在这里了, stl容器类会使用其元素类的拷贝构造函数来复制元素.

一定是ofstream通过这样的复制后产生了错误的状态.

而宿主类及其上层也没有提供拷贝构造函数, 只有默认的浅拷贝.

如果没有这个流, 这些类都是一些简单数据类, 完全可以使用默认的字节拷贝.

浅拷贝不行, 深拷贝不能, 结果是找到原因, 却不知如何修改.

只能暂时恢复为临时流变量.

不过最终还应该支持大量的文件打开关闭操作, 不知怎么办.

心得:

* 没有必要就不要保存一个流变量,使用临时变量可保证文件打开与关闭是成对的。

* stl容器最好只存放简单数据,可避免自定义拷贝构造函数。

如果元素类较复杂,应考虑使用指针,因为有可能今后的修改后使元素类更复杂。

* ofstream::close()可能不会清除错误状态,并可能置failbit,如未打开就关闭。

上面的clear()应加在close()之后。

问题:

* 如何添加单元测试,使得一个类保证为简单数据类,或保证stl容器可安全使用?

* 不能实现深拷贝时,有没有办法解决浅拷贝问题?

 
 
 
免责声明:本文为网络用户发布,其观点仅代表作者个人观点,与本站无关,本站仅提供信息存储服务。文中陈述内容未经本站证实,其真实性、完整性、及时性本站不作任何保证或承诺,请读者仅作参考,并请自行核实相关内容。
2023年上半年GDP全球前十五强
 百态   2023-10-24
美众议院议长启动对拜登的弹劾调查
 百态   2023-09-13
上海、济南、武汉等多地出现不明坠落物
 探索   2023-09-06
印度或要将国名改为“巴拉特”
 百态   2023-09-06
男子为女友送行,买票不登机被捕
 百态   2023-08-20
手机地震预警功能怎么开?
 干货   2023-08-06
女子4年卖2套房花700多万做美容:不但没变美脸,面部还出现变形
 百态   2023-08-04
住户一楼被水淹 还冲来8头猪
 百态   2023-07-31
女子体内爬出大量瓜子状活虫
 百态   2023-07-25
地球连续35年收到神秘规律性信号,网友:不要回答!
 探索   2023-07-21
全球镓价格本周大涨27%
 探索   2023-07-09
钱都流向了那些不缺钱的人,苦都留给了能吃苦的人
 探索   2023-07-02
倩女手游刀客魅者强控制(强混乱强眩晕强睡眠)和对应控制抗性的关系
 百态   2020-08-20
美国5月9日最新疫情:美国确诊人数突破131万
 百态   2020-05-09
荷兰政府宣布将集体辞职
 干货   2020-04-30
倩女幽魂手游师徒任务情义春秋猜成语答案逍遥观:鹏程万里
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案神机营:射石饮羽
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案昆仑山:拔刀相助
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案天工阁:鬼斧神工
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案丝路古道:单枪匹马
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:与虎谋皮
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:李代桃僵
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:指鹿为马
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案金陵:小鸟依人
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案金陵:千金买邻
 干货   2019-11-12
 
推荐阅读
 
 
 
>>返回首頁<<
 
靜靜地坐在廢墟上,四周的荒凉一望無際,忽然覺得,淒涼也很美
© 2005- 王朝網路 版權所有