银行系统不工作

Question

由于某种原因，我的银行脚本无法正常工作。更具体地说，search() 不起作用。我有点理解为什么它没有，可能是因为if(obj.returnId() == n)，但我不知道如何解决它。当我搜索一个帐户时，它只会让我找到最后一个帐户，而不是以前的任何帐户。这是我的代码：

#include "stdafx.h"
#include <iostream>
#include <string>
#include <string.h>
#include <fstream>
#include <Windows.h>
#include <conio.h>
using namespace std;
bool loop = true;

class account
{
          int id;
         char name[40];
         char password[40];
public:
          void getData()
          {
                     cout << "\nEnter your name: ";
                     cin >> name;
                     cout << "\nEnter ID: ";
                     cin >> id;
                     cout << "\Enter pass: ";
                     cin >> password;
          }
          void showData()
          {
                     cout << "\nName: ";
                     puts(name);
                     cout << "\nID: " << id;
                     cout << "\n";
          }
          int returnId()
          {
              return id;
          }
};

void createAccount()
{
    account obj;
    ofstream fileCreate;
    fileCreate.open("accounts.dat", ios::binary|ios::app);
    obj.getData();
    fileCreate.write((char*)&obj,sizeof(obj));
    fileCreate.close();
}

void display()
{
    account obj;
    ifstream fileRead;
    fileRead.open("accounts.dat", ios::binary);
    while(fileRead.read((char*)&obj, sizeof(obj)))
    {
        obj.showData();
    }
    fileRead.close();
}

void search(int n)
{
    account obj;
    ifstream fileRead;
    fileRead.open("accounts.dat", ios::binary);
    while(fileRead.read((char *) &obj, sizeof(obj)) );
    {
        fileRead.seekg(0,ios::beg);
        if(obj.returnId() == n)
        {
            obj.showData();
        }
        else {
            cout << "\nUser not foud!\n";
        }
    }
    fileRead.close();
}








void main()
{
    cout << "Welcome to the Bank.\n\n";

    while (loop==true)
    {
        char choice[10];
        cout << "Please select an option:\n";
        cout << "------------------------------------------------\n";
        cout << "(a)Log into an account\n(b)Create an account\n(s)Search an account\n(e)Exit\n";
        cout << "------------------------------------------------\n";
        cout << "Choice: ";
        cin >> choice;
        choice[0] = tolower(choice[0]);
        cout << "\n------------------------------------------------\n\n";

        switch (choice[0])
        {
        case 'a':
            display();
            break;
        case 's':
            int n;
            cout << "Enter the ID of the account: ";
            cin >> n;
            search(n);
            break;
        case 'b':
            createAccount();
            break;
        case 'e':
            loop = false;
            break;
        default:
            system("CLS");
            cout << "The option \"" << choice[0] <<  "\" is invalid.\n\n\n\n";
            break;
        }

    };
    cout << "\n\n\n";
    cout << "Click anything to exit.";
    getch();
}

Answer 1

你的问题是这行末尾的分号：

while(fileRead.read((char *) &obj, sizeof(obj)) );

这使得这个循环有一个空的主体。因此，您基本上阅读了整个文件并丢弃了结果，除了最后一个条目。

也摆脱这个：

fileRead.seekg(0,ios::beg);

我不知道你为什么需要它，它只会让你一遍又一遍地阅读第一个条目。

Answer 2

另一个错误是，当您测试了所有帐户并且它们都失败时，您应该只说“找不到用户”。您的循环（当您删除分号时）在每次失败的测试后都说“找不到用户”。

Answer 3

您可能找不到您要查找的条目，因为每次您从文件中读取一个条目时，都会将位置重置为开头。这意味着您的循环将永远运行，一遍又一遍地读取相同的条目，而永远找不到您搜索的条目。

Answer 4

寻找可能是问题所在：

while(fileRead.read((char *) &obj, sizeof(obj)) ) //;
{
    // seek to start?
    //fileRead.seekg(0,ios::beg);
    ...
}

看看http://www.boost.org/doc/libs/1_51_0/libs/serialization/doc/index.html

除此之外，使用

cout << "text" << endl;

用于与平台无关的换行符。