[求助]C++关于fstream的eof

[colebin]

我的代码如下:
int _tmain(int argc, _TCHAR* argv[])
{
fstream file;

file.open( "basic_istream_tellg.dat", ios::out|ios::binary);
if (!file.is_open()) {
printf("error\n");
exit(-1);
}

char* buf = "Students";
int len = strlen(buf)+1;
file.write((char*)&len,sizeof(int));
file.write(buf, len);

file.close();

file.open( "basic_istream_tellg.dat", ios::in|ios::binary);
if (!file.is_open()) {
printf("error\n");
exit(-1);
}

file.seekg(streampos(0));
file.unsetf(ios::skipws);
bool end=false;
while(!end) {
if(file.eof())
{
end=true;
break;
}
char* buffer;
file.read((char*)&len,sizeof(int));
buffer = new char[len];
file.read(buffer, len);

printf(buffer);
}

file.close();

return 0;
}

但输出为：Students屯屯屯屯妄铪

为什么?

[newbiestar]

根本问题在read上面，你需要在read后面判断是不是eof了，而不是在前面……

写文件么，只不过是不合规范而已，那个'\0'的确写进去了，一般每人这么干的……

在你这个例子里面都是写8进去，然后写Students进去就结束了，而不是9和"Students\0"

[colebin]

可我如果用while(file.peek() != EOF) {

char* buffer;
file.read((char*)&len,sizeof(int));
buffer = new char[len];
file.read(buffer, len);

printf(buffer,len);
}
读出来是正确的

[newbiestar]

peek带了读取操作测试的……

[newbiestar]

读文件的时候有错误，还有就是printf用的极其不规范……

第一个参数是给你写格式用的，而不是让你作为字符串使用的……

具体原因如下：
你读文件的时候仅仅读到最后一个字节，文件状态并为被置为eof，只有当你的读入操作由于eof失败的时候，才会把文件状态置eof，所以你的循环执行了两次，这样的话虽然第一次读写成功了，但是第二次的读失败冲毁了你先前的内容（因为内存泄漏，new了确没有delete，估计你认为while循环只能执行一次……），所以打印的内容就不对了……

另外，printf按照这个用法的话，你如果文件里面存在"%s%p"这样的printf格式的话，得出的结果将是极其古怪的……

[newbiestar]

你应该在每一个read以后检查eof的情况，这样的话就可以避免你现在的情况了

[naokii]

不小心就会这样了。是你写文件时的错误，没有文件的结束符‘\0’。这样写了：
file.open( "basic_istream_tellg.dat", ios::out|ios::binary);
if (!file.is_open()) {
printf("error\n");
exit(-1);
}

char* buf = "Students";
int len = strlen(buf)+1;
file.write((char*)&len,sizeof(int));
file.write(buf, len);
file.write("\0",1);

file.close();

读文件时，也有不妥的地方。那个while循环设计的不够专业，尽管不会影响结果。

[colebin]

谢谢楼上几位！
to naokii(西莫)和 newbiestar：读文件中有什么地方不妥，怎么设计读文件比较有效?