Question

我正在尝试编写一个函数，在调用时，将搜索我的文本文件以匹配参数，然后通过覆盖文件来“更新”它。

这是我目前的职能：

编辑：这是新的更新代码，仍然无效：/

以下是它发送给UpdateGem的行:(示例sTempTxt：“AB：5.55”）

ostringstream stream; 
stream << std::setprecision(3)  << fEindgem;
string sTempTxt = sVak + ":" + stream.str() + "\n";
UpdateGem(sTempTxt);

void UpdateGem(string text)
{
     ifstream f;
     f.open("GEM.txt");
     string sGEMS[100];
     string temp[3];
     splitstring(text, ":", temp[0], temp[1]);
     bool OverWrite;
     int count = 0;
     string Delete, line;

     while(true)
     {
         getline(f, sGEMS[count], '\n');
         if(f.eof()) break;
         splitstring(sGEMS[count], ":", temp[1], temp[2]);
         if (temp[0] == temp[1])
         {
              OverWrite = 1;
              Delete = sGEMS[count];
         }
         count++;
     }

// Don't set count to 0, since we need it
// count = 0;
   ofstream f2;
   f2.open("GEM2.txt"/*,std::ios_base::app*/);
   if (OverWrite) 
   {
      f.seekg(0, std::ios::beg);
      for (int i = 0; i < count; ++i) 
      {
         if (sGEMS[i] != Delete) f2 << sGEMS[i];
      }
   }
     f.close();
     f2.close();
     remove("GEM.txt");
     rename("GEM2.txt", "GEM.txt");

     ofstream file;
     file.open("GEM.txt",std::ios_base::app);
     file << text;
     file.close();     


}

它必须替换的行采用NAME：NUMBER的形式，其中数字可以不同，所以我使用splitstring函数将名称与找到的行的名称进行比较，然后完全删除该行通过稍后重新添加来“更新”它。但是，我当前的代码只将更新的行写入文件，而不是旧文件......

Answer 1

从您的程序看起来，您假设“GEM.txt”中的文本行数不超过100行。但是，在初始扫描文件时，您不记得读入了多少行。因此，为了弥补这一点，您尝试通过在原始“GEM.txt”文件中重新读取多少行来记住，并将保存的行写入“GEM2.txt”。

正如jrok正确指出的那样，在你的第二个循环中，你从“GEM.txt”中读取，从你在第一个循环中遇到feof开始。所以，你的第二个循环永远不会进入while循环，因为getline调用认为没有任何东西可以获取（因为f位于文件的末尾）。因此，jrok为您提供了在第二个while循环之前添加的代码行：

f.seekg(0, std::ios::beg);

但是，由于您已经将所有行保存在数组中，因此您可以直接将sGEMS数组写入新文件中。

// Don't set count to 0, since we need it
// count = 0;
ofstream f2;
f2.open("GEM2.txt"/*,std::ios_base::app*/);
if (Overwrite) {
    for (int i = 0; i < count; ++i) {
        if (sGEMS[i] != Delete) f2 << sGEMS[i] << std::endl;
    }
}
//...

如果文件长度超过100行，则使用矢量会更安全，而不是使用数组。而且，您不再需要count，因为它会被vector的大小跟踪。

std::vector<std::string> sGEMS;
//...
while (true) {
    getline(f, line);
    if (f.eof()) break;
    sGEMS.push_back(line);
    //...
}

如果您希望“GEM.txt”文件非常大，则可能需要重新考虑如何处理文件。特别是，您应该只记住要跳过的行号，并将“GEM.txt”中的行复制到原来的“GEM2.txt”。

最后，在代码的最后一次写入中，打开要追加的文件，但不打开输出。 C ++认为这意味着您要丢弃文件的内容。而是将ios_base::out标记添加到您的公开呼叫中。

ios_base::openmode mode = ios_base::out|ios_base::app;
file.open("GEM.txt",mode);
file << text << std::endl;
file.close();

但是，在关闭它之前，您可以将其添加到f2的末尾，而不是这样做。

根据我的建议，UpdateGem的最终形式应为：

void UpdateGem(string text) {
    ifstream f;
    ofstream f2;
    vector<string> sGEMS;
    string temp[3];
    bool OverWrite;
    string Delete, line;

    splitstring(text, ":", temp[0], temp[1]);
    sGEMS.reserve(100);
    f.open("GEM.txt");
    while (true) {
        getline(f, line);
        if (f.eof()) break;
        sGEMS.push_back(line);
        splitstring(line, ":", temp[1], temp[2]);
        if (temp[0] == temp[1]) {
            OverWrite = 1;
            Delete = line;
        }
    }
    f.close();
    f2.open("GEM2.txt"/*,std::ios_base::app*/);
    if (OverWrite) {
        for (int i = 0; i < sGEMS.size(); ++i) {
            if (sGEMS[i] != Delete) f2 << sGEMS[i] << std::endl;
        }
    }
    f2 << text << std::endl;
    f2.close();
    remove("GEM.txt");
    rename("GEM2.txt", "GEM.txt");
}

我使用以下main程序测试了代码：

int main () { UpdateGem("Test:0001"); }

并像这样实施splitstring：

bool splitstring (string s, string match, string &a, string &b)
{
    int x = s.find(match);
    if (x != string::npos) {
        a = s.substr(0, x);
        b = s.substr(x+match.size());
        return true;
    }
    return false;
}

当输入以下输入“GEM.txt”时：

a:x
b:x
Test:1234
c:x

程序将“GEM.txt”的内容修改为：

a:x
b:x
c:x
Test:0001

通过覆盖来“更新”文件中的一行

1 个答案: