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

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 正确指出的那样，在您的第二个循环中，您正在从第一个循环中遇到 feof 时开始读取“GEM.txt”。因此，您的第二个循环永远不会进入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