2 sinat 31675145 sinat_31675145 于 2016.03.16 15:01 提问

MFC 文件写入 WriteString 引起中断。

MFC写文件时,打开写入位置文件夹,或者刷新文件夹里的内容,就会引起中断。
我的代码如下:

CString savepath("E:\测试文件夹");
CreateDirectory(savepath,NULL);

CString filename("\test.txt");
filename = savepath + filename;

CString str;
str =_T("test2016test2016test2016test2016test2016\r\n");

CStdioFile myFile;
CFileException fileException;

while(1)
{
Sleep(1);
if(!myFile.Open(filename,CFile::modeCreate|CFile::modeNoTruncate|CFile::modeReadWrite|CFile::shareCompat|CFile::shareDenyNone),&fileException;)
{
myFile.SeekToEnd();
myFile.WriteString(str);
myFile.Close();
}
else
{
MessageBox(_T("打开文件失败"));
}
}

文件循环写入的图中,以下情况会偶尔引起中断
1,打开“测试文件夹”
2,在“测试文件夹”里按F5刷新,
3,复制test.txt文件。
4,打开test.txt文件。
当文件越来越大的时候,这样的操作引起中断更容易。
中断位置指向WriteString(str);

请问各位大神,怎么处理?哪里操作不够安全?

3个回答

baijiaheizhiganmao
baijiaheizhiganmao   2016.03.16 16:33

你while(1)的时候,在内部打开了文件,但是你if判断逻辑错误,然后文件就一直开着!因为文件打开成功了,到了else分支,while运行第二次后,你打开失败,因为前一次打开了,然后到了if的!open分支,你往里面写数据,然后close
接着while继续……现在你明白吧

a274767172
a274767172 嗯嗯嗯!知道了!omg!这个得留心了!
接近 2 年之前 回复
baijiaheizhiganmao
baijiaheizhiganmao   2016.03.16 15:30

你是不是逻辑错了? !myFile.Open的意思是:打开失败!此时你失败了还writeString肯定会引起中断!

 if(!myFile.Open(filename,CFile::modeCreate|CFile::modeNoTruncate|CFile::modeReadWrite|CFile::shareCompat|CFile::shareDenyNone),&fileException;)
{
myFile.SeekToEnd();
myFile.WriteString(str);
myFile.Close();
}
else
{
MessageBox(_T("打开文件失败"));
}
a274767172
a274767172 哦哦哦! 是的 我明白了! 我的语句有问题!之前那个语句是抄别人用的。修改见楼下,就可以了。多谢提醒!
接近 2 年之前 回复
a274767172
a274767172 可是,我那样写,为什么还可以写入呢?只是写入到后边会引起中断。
接近 2 年之前 回复
a274767172
a274767172   2016.03.16 16:32

代码中的,&fileException(没有分号,上面多打了。)位置放错了!导致if语句一直执行。这时候打开失败也能进入写分支。else分支一直没有用到。

原因:代码是抄别人的,没留意if里的判断。
修改方案:删除&fileException(我不太清楚它的作用)。调整if语句的判断如下,并更改写入逻辑。

if(!myFile.Open(filename,CFile::modeCreate|CFile::modeNoTruncate|CFile::modeReadWrite|CFile::shareCompat|CFile::shareDenyNone))
{
MessageBox(_T("打开文件失败,文件被占用"));
}
else
{
myFile.SeekToEnd();
myFile.WriteString(str);
myFile.Close();
}

a274767172
a274767172 回复忘世麒麟: 好的! 详细的内容我再慢慢了解。多谢~
接近 2 年之前 回复
a274767172
a274767172 回复忘世麒麟: 嗯!我测试过了,错误的代码里,一直走的是if分支,else没进去。这样当有一次文件被占用,无法打开时,还是进入if分支,导致writestring中断。
接近 2 年之前 回复
baijiaheizhiganmao
baijiaheizhiganmao 回复阿理阿why: 其实你打开失败不一定是因为文件被占用了,你可以使用GetLastError 获取错误的值,然后去搜索这个值:http://baike.baidu.com/link?url=mo3dqjjzyNSe0Tg2NEeRn0ZbBy9UZqTRR6A3shFQcddegKcz_xYhrm1Sp0C7Gz3qcHpH5_vyhH1hyCSIzk_Nuq
接近 2 年之前 回复
a274767172
a274767172 回复忘世麒麟: 灰常感谢了~
接近 2 年之前 回复
a274767172
a274767172 回复忘世麒麟: 我上面这样改动没有问题了。文件被占用也是正常通知,程序不会崩溃了。
接近 2 年之前 回复
baijiaheizhiganmao
baijiaheizhiganmao 每次打开前,要判断是否文件已经打开过。或者你的open和close要配对使用,不然有你疯狂的时候。
接近 2 年之前 回复
Csdn user default icon
上传中...
上传图片
插入图片
准确详细的回答,更有利于被提问者采纳,从而获得C币。复制、灌水、广告等回答会被删除,是时候展现真正的技术了!