1

下面的代码有什么问题。对于某些输入，它运行得非常好，而对于某些特殊输入，它会崩溃？

#include<iostream>
#include<string>
#include<fstream>

using namespace std;


struct event { 

string date,time,content;
bool is_high_priority;

};


int main() {

event one,two;
one.is_high_priority=false;
char tmp;

ofstream out_file("events" , ios::binary );


    cout<<"\nEnter Date(dd.mm) ";
    cin>>one.date;
    cout<<"\nEnter Time(hh:mm:ss) ";
    cin>>one.time;
    cout<<"\nenter content";
    cin>>one.content;

    if(tmp == 't') 
        one.is_high_priority = true;
    else
        one.is_high_priority = false;


    out_file.write((char*) &one, sizeof(one) );

    out_file.close();


    ifstream in_file("events" , ios::binary );
    in_file.read((char*)&two,sizeof(two));

    cout<<two.date<<" "<<two.time<<" "<<two.content<<" "<<two.is_high_priority;

    in_file.close();

}

它因这些输入而崩溃： Enter Date(dd.mm) ankmjjdn md

输入时间(hh:mm:ss) 输入内容njs sjnsn

4

3 回答 3

6

您不能只将std::string对象的字节保存到文件中，然后再次加载它们。包含指向动态分配内存的std::string指针，您的保存/加载只会复制指针本身，而不是指向数据。

于 2012-08-11T12:24:19.533 回答
1 
if(tmp == 't')

tmp是一个本地未初始化的变量，您正在使用它，如上所示。

因此，您的代码会调用未定义的行为。没有什么可以说的了。

于 2012-08-11T12:23:11.740 回答
0

正如@sth 所指出的，将指针复制到文件并重新解释它们，最终会出现无重复的情况。

但是，我认为还有一些更微妙的东西，如果我错了，请纠正我。该程序并不总是在所有输入上崩溃。它甚至不依赖于字符串的长度。我尝试用一​​些测试用例运行程序。

请注意，字符“空格”被视为分隔符。即使您提供输入，空格后面的字符也会被下一个字符串占用。

因此，第三个字符串占用该值直到 'space' 剩余的输入（直到回车）仍在输入缓冲区中。

我怀疑当从 ​​istream 构造第二个对象时，“空格”之后的字符串会覆盖指针，这会导致损坏检测。

于 2012-08-11T16:04:00.150 回答