最后一条数据读了两次？如何改哇

2013-07-11

最后一条数据读了两次？？？怎么改哇m_downFile fopen(DownloadData.dat,r)//打开文件下载列表的记录

最后一条数据读了两次？？？怎么改哇


m_downFile = fopen("DownloadData.dat","r");//打开文件下载列表的记录文件
if (m_downFile==NULL)
{
//fclose(m_downFile);
return 0;
}
else
{
while(!feof(m_downFile))
{
//if (!feof(m_downFile))
//{
DOWNLOAD_FILE_LIST downlist;
CString str;
fread(&downlist, 1, sizeof(DOWNLOAD_FILE_LIST), m_downFile);
int count = m_downloadlist.GetItemCount();
//count++;
m_downloadlist.InsertItem(count, str,20);

m_downloadlist.SetItemText(count,0, downlist.FileName);
str.Format("%lld",downlist.size);
m_downloadlist.SetItemText(count, 1, str);
str.Format("%lld",downlist.downsize);
m_downloadlist.SetItemText(count, 2, str);

m_downloadlist.SetItemText(count, 5, downlist.Uuid);
//}
//else
//{

}
fclose(m_downFile);
//break;
}

}

[解决办法]

不要使用
while (条件)
更不要使用
while (组合条件)
要使用
while (1) {
 if (条件1) break;
 //...
 if (条件2) continue;
 //...
 if (条件3) return;
 //...
}
因为前两种写法在语言表达意思的层面上有二义性，只有第三种才忠实反映了程序流的实际情况。
典型如：
下面两段的语义都是当文件未结束时读字符
whlie (!feof(f)) {
 a=fgetc(f);
 //...
 b=fgetc(f);//可能此时已经feof了！
 //...
}
而这样写就没有问题：
whlie (1) {
 a=fgetc(f);
 if (feof(f)) break;
 //...
 b=fgetc(f);
 if (feof(f)) break;
 //...
}
类似的例子还可以举很多。

[解决办法]
检查 fread 的返回值. 失败了直接跳出 while
[解决办法]
fread()在读到文件最后一个字节时，feof(fp)还是为0值，此时while(!feof(fp))的控制条件依然为真，所以会将最后一个数据单元再次读取一遍。然后feof(fp)才为非0值
[解决办法]
可以用fread的返回值是否为0来判断是否到末尾
------解决方案--------------------

最后一次读操作不会feof，再次读才会触发feof。

[解决办法]
方法1:先用fread读一次，然后再循环读（不知道为什么，但是以前这样做可以解决问题）。
fread(...);
while(!feof(fp)){
fread(...);
}

方法2:先求循环读的次数，再用for循环读。
fseek(fp, 0, SEEK_END);
int fileSize = ftell(fp);
fseek(fp, 0, SEEK_SET);
int cycleTimes = fileSize/一次循环中读的大小;

for(int i = 0; i < cycleTimes; i++){
fread(...);
}
[解决办法]
简单的说不是feof之后在 fread
应该是 fread 之后在feof 试试看

热点排行

C++

最后一条数据读了两次？如何改哇