-1

我正在尝试从文件中读取我的数据库。

这是我的 save_base 方法：

void data_base::save_base()
{
    fstream file;

    file.open("base.dat", ios::in | ios::out | ios::trunc);

    if(file.good()==true) {
        node *p = new node();
        p=first;

        while(p) {
            file << p->content->connect() << ";" << "\n";
            p=p->next;
        }
        file.close();
    }else{
    cout << "Err - opening file." << endl;
    }
}

连接方法：

string product::connect() {

    ostringstream do_string;
    do_string << lp;
    string new_lp = do_string.str();

    ostringstream do_string1;
    do_string1 << count;
    string new_count = do_string1.str();

    ostringstream do_string2;
    do_string2 << prize;
    string new_prize = do_string2.str();

    ostringstream do_string3;
    do_string3 << vat;
    string new_vat = do_string3.str();

    string connected = type + ";" + new_lp + ";" + name + ";" + new_count + ";" + unit + ";" + new_prize + ";" + new_vat;
    return connected;
}

和 read_base 方法：

void data_base::read_base()
{
    fstream file;

    file.open("base.dat", ios::in);
    if(file.good()==true)
    {
        char data_row[50];
        int i=1;
        while(!file.eof()) {
            file.getline(data_row,100);

        string data_content[50];
        int j = 0;

        char *buff;
        buff = strtok (data_row,";");
        while (buff != NULL)    {
            data_content[j] = buff;
            buff = strtok (NULL, ";");
            j++;
        }
        string type = data_content[0];
        int lp;
        istringstream iss1(data_content[1]);
        iss1 >> lp;
        double count;
        istringstream iss2(data_content[3]);
        iss2 >> count;
        double prize;
        istringstream iss3(data_content[5]);
        iss3 >> prize;
        double vat;
        istringstream iss4(data_content[5]);
        iss4 >> vat;

        // Sprawdzamy typ obiektu zapisanego w danym wierszu pliku
        if(type == "product")
        {

            product new_prod(lp, data_content[2], count, data_content[4], prize, vat);
            product *new_product = new product(new_prod);
            this->add(new_product);
        }
        i++;
        }
    file.close();
    }else{
        cout << "Err opening file." << endl;
    }
}

我正在向数据库中添加一些产品，它工作正常。即使保存到文件也很好。但主要问题是当我试图从文件中读取数据库时。从文件中读取数据库工作正常，但最后，应用程序不会自行结束。我认为仍有一些缓冲区需要关闭。但我不知道它们是哪个或多近。

4

2 回答 2

1

老实说 - 您在这段代码中遇到了很多问题，我建议您的主要问题是您可能仍在努力解决封装问题。您的代码混合使用了 C 和 C++ 样式，这保证使代码难以阅读并因此难以维护，这就是为什么您在这里并且人们正在努力给您答案的原因。

也就是说，问题看起来是在保存中你调用“new node()”，看起来你会保存......什么都没有。您不应该查询产品中的一些现有值吗？编辑：啊 - 不，你现在将它重新分配为“p = first”，我明白了。那么你在上一行分配的节点会发生什么？

当您的任何行超过 50 个字节时，您还有一个潜在的堆栈破坏器：

char data_row[50];
int i=1;
while(!file.eof()) {
    file.getline(data_row,100);

如果必须使用 char 数组，请养成使用 sizeof 的习惯：

file.getline(data_row, sizeof(data_row));

我花了一些时间才弄清楚 istringstreams 的用途——你试图将字符串转换为数字？可以更简单、更高效地写成：

unsigned int lp = atoi(data_content[1].c_str());
or
unsigned int lp = strtoul(data_content[1].c_str(), NULL, 10);

您的 read_base 函数是数据库的一部分，对数据记录了解太多了。您需要将记录群体封装起来，以便 read_base 看起来更像：

void data_base::read_base()
{
    fstream file("base.dat", ios::in);
    if(file.good() == false) {
        cout << "Error opening file." << endl;
        return;
    }

    for(size_t rowNo = 0; !file.eof(); ++rowNo) {
        row* row = data_type::make_object_from_row(file);
        if(row != nullptr)
            this->add(row);
        }
    }
}

您可能需要查看“工厂模式”以了解如何实现允许您执行此操作的 data_row 基本类型。但简而言之，您使产品继承 data_type 以便上述工作。data_type::make_object_from_row 将读取行的第一部分并枚举类型，以便它可以执行以下操作：

data_type* data_type::make_object_row_row(istream& file) {
    switch(get_type(file)) {
        case DATA_TYPE_PRODUCT: return new product(file);
        case DATA_TYPE_PERSON: return new person(file);
        default:
          clog << "invalid type in database: " << type << endl;
          return nullptr;
    }
}

（这只是一种可能的方法）。

于 2013-05-20T00:19:49.153 回答
0

我看到至少一个明显的错误可能会产生未定义的行为。您正在分配 50 个字节，但最多读取 100 个。

    char data_row[50];
    int i=1;
    while(!file.eof()) {
        file.getline(data_row,100);

我建议在此代码中将 50 更改为 100。

于 2013-05-19T22:32:52.897 回答