2

我有一个文本文件，我想通过将其重写为临时文件然后覆盖原始文件来进行编辑。这段代码没有这样做，因为它被简化了，但它确实包含了我遇到的问题。在 Windows 上，当重命名功能失败时，EXAMPLE.TXT 文件将在看似随机的运行次数后消失。我不知道为什么，但到目前为止它在 Linux 上运行良好。为什么会发生这种情况，我该如何以完全不同的方向解决它，例如从程序中覆盖原始文件而不重命名？

此外，还有什么其他更好的方法存在？此方法在 Windows 上还有其他缺陷，例如程序在调用 remove 之后重命名之前被用户关闭，这在 Linux 上不会出现问题（摆脱 remove 之后）？

#include <stdio.h>
#include <assert.h>

int main(int argc, char *argv[]) {
  unsigned int i=0;
  FILE *fileStream, *tempStream;
  char fileName[] = "EXAMPLE.TXT";
  char *tempName = tmpnam(NULL);

  while(1) {
     printf("%u\n",i++);
     assert(fileStream = fopen(fileName, "r+"));
     assert(tempStream = fopen(tempName, "w"));

     fprintf(tempStream,"LINE\n");
     fflush(tempStream); /* fclose alone is enough on linux, but windows will sometimes not fully flush when closing! */

     assert(fclose(tempStream) == 0);
     assert(fclose(fileStream) == 0);
     assert(remove(fileName) == 0); /* windows fails if the file already exists, linux overwrites */
     assert(rename(tempName,fileName) == 0);
  }
}
4

2 回答 2

5

这样做确实很可能会引起麻烦。在 Windows 上，您的代码有四种可能的结果：

  • 删除很好，重命名工作，没问题
  • 删除很好，但另一个进程通过删除共享打开了文件。常见于恶意软件扫描程序和文件内容索引器。这可确保在文件的最后一个句柄关闭时文件实际被删除。问题是，重命名失败，因为文件仍然存在
  • 不会因为文件被锁定而删除，您的断言会触发
  • 什么都没有发生，因为 assert() 在您构建发布版本时是无操作的。

顺便说一句，最后一个子弹的可能性很大，它肯定解释了可重复的失败。你需要一个更具防御性的策略来处理第二个子弹：

  • 删除filename.bak，如果失败则报告错误
  • 将fileName重命名为filename.bak，如果失败则报告错误
  • 将 tempName 重命名为文件名，如果失败则报告错误并重命名 filename.back
  • 删除filename.bak，不报错

这是一种常见的场景，winapi 有一个函数ReplaceFile()。请务必使用备份文件选项以获得最大的收益。-

于 2012-06-13T20:49:38.163 回答
1

有时，防病毒软件会在不方便的时候扫描文件，从而导致此类问题。

如果remove失败，请尝试休眠一小段时间，然后重试。

于 2012-06-13T19:32:06.797 回答