为什么我的程序不从文件中读取数据

Question

当我按“2”读取文件时，虽然我的语法正确，但无法读取文件中的数据。为什么我的程序无法读取数据？顺序访问文件和随机访问文件有什么区别，为什么首选随机访问文件？

void EnterRecord();
void ShowRecord();
using namespace std;
class student
{
    int rollno;
    string name;
    int marks;
    public:
        void SetData();
        void DisplayData();
        void MainMenu();
};
   void student:: MainMenu()
   {
      cout<<"                             "<<endl;
      cout<<"press 1 to enter record"<<endl;
      cout<<"press 2 to show record"<<endl;
   }
   void student::SetData()
   {
      cout<<"enter roll no of student"<<endl;
      cin>>rollno;
      cout<<"enter name of student"<<endl;
      cin>>name;
      cout<<"enter marks of student"<<endl;
      cin>>marks;
   }
   void student::DisplayData()
   {
      cout<<rollno<<setw(10)<<setw(10)<<marks<<endl;
   }

     int main()
     {
            student s1;
        int choice;
        repeat:
        s1.MainMenu();
        cout<<"enter ur choice ::"<<endl;
        cin>>choice;

        switch(choice)
        {
            case 1:
                EnterRecord();
                break;
                case 2:
                    ShowRecord();
                    break;
         }
           return 0;
     }

      void  EnterRecord()
   {
      ofstream myfile;
      myfile.open("student3.txt",ios::app);
      int choice=1;
      while(choice==1)
      {
        student s1;
          s1.SetData();

          myfile.write((char*)&s1,sizeof(student));
          cout<<"press 1 to enter record again"<<endl;
          cin>>choice;
          if(choice!=1)
          {
            system("cls");
           s1.MainMenu();
      }
         }
           myfile.close();
   }
     void ShowRecord()
     {
        student s2;
         ifstream myfile;
         myfile.open("student3.txt");
         myfile.seekg(0);
         while(myfile.eof())
         {
            myfile.read((char*)&s2,sizeof(student));
          }
          s2.DisplayData();
     }

Answer 1

最大的问题是您将student 的实例视为单个普通旧数据（POD）块，而实际上它不是。它包含一个std::string 的实例，其中包含您不知道的内部数据。第一个也是最明显的是它包含一个指向字符串数据的指针。当你把它写出来时，你保存的是指针值而不是实际的字符串。当你重新加载它时，它会读取 old 指针值，该值只有在你是一个非常幸运的小妖精时才有效。

您的函数EnterRecord 和ShowRecord 将需要更新，以便它们序列化student 的每个成员，而不是将其写成一个普通的旧数据块。下面的内容应该可以帮助您入门。

保存数据：

myfile << st.rollno;
myfile << st.name;
myfile << st.marks;

加载数据

myfile >> st.rollno;
myfile >> st.name;
myfile >> st.marks;

由于您没有提供有关文件布局方式的任何信息，因此您可以自行决定如何布局。传统上你会超载operator>> 和operator<<，但因为这看起来像是家庭作业，可能有点多。

您的代码还有其他几个问题...

1. 您的 case 表达式不正确。您正在使用 values 1 和 2，它们与 1 和 2 键的 ASCII 值不同。您应该使用 '1' 和 '2' 而不是简单的 1 和 2。

2. 您还应该开始检查您调用的函数返回的错误，例如open。

3. 在你的 while 循环中检查 eof 肯定是行不通的。见问答@Why is iostream::eof inside a loop condition considered wrong?

Answer 2

应该是：

switch(choice)
{
    case '1':
        EnterRecord();
        break;
    case '2':
        ShowRecord();
        break;
}