删除文件异常，删除一行文字

Question

我正在尝试在一个类中实现一些基本的配置文件读/写/编辑功能。 配置文件存储在文件 [...]\config.txt 中，该文件存储在“路径”变量中。

我的配置文件的语法如下：

param0 value_of_it
param1 value_of_it
something_else you_get_it

第一个空格的所有内容都是要找到参数的名称，该行的其余部分被认为是该参数的值。

上下文就这么多。 现在，当我尝试使用 file.delete 删除该文件时，我得到：

“System.IO.IOException: 无法访问文件，因为它正被另一个进程使用”。

这里是有问题的代码：

    internal void alter(string param, bool replace = false, string value = null) {
    //here
        string buf;

        System.IO.StreamReader reader = new System.IO.StreamReader(path);

        string bufPath = path.Substring(0, path.Length-4)+"_buf.txt";

        System.IO.StreamWriter writer = new StreamWriter(bufPath);

        while((buf = reader.ReadLine()) != null) {
            if(buf.Substring(0, param.Length) != param)
                writer.WriteLine(buf);
            else if(replace)
                writer.WriteLine(param+" "+value);
        }
        writer.Dispose();
        reader.Dispose();
        writer.Close();
        reader.Close();
        //there



        File.Delete(path);
        File.Move(bufPath, path);
    }

该函数将原始文件逐行复制到缓冲区中，调用中指定的参数除外。该行要么被忽略，因此被删除，或者在调用时被替换。 然后删除原始文件并将_buf版本“移动”到那里（重命名）。

这些步骤被正确执行（_buf 文件被创建并正确填充）直到抛出异常的“File.Delete(path)”。

我已经尝试注释掉 //here 和 //there 之间的整个部分，只删除文件，这会导致相同的异常，因此问题必须具有更基本的性质。

按照here on ServerFault 的建议，我尝试使用 Sysinternals Process Explorer 查找一个有问题句柄的进程，但找不到任何东西。我在自己的进程中只找到了该文件的三个句柄。 我在杀死 ​​Windows 资源管理器后也运行了该应用程序，因为我读过有时它是违规者；这也没有帮助。

我还检查了整个程序的其余部分并确认了以下内容：

1. 这是第一次使用此文件，因此不可能忘记过去的句柄。

2. 它是一个单线程应用程序（到目前为止），因此也排除了死锁和竞争条件。

所以这是我的问题：我怎样才能找到锁定我的文件的内容？ 或者：有没有更简单的方法来删除或编辑文件中的一行？

Answer 1

以 xml 格式存储配置会更方便，因为如果您以后需要存储更复杂的值，您可以选择存储。您还可以依靠XDocument 来加载和保存文件，就像XDocument.Load(path) 和doc.Save(path) 一样简单。例如，您当前的配置文件将以 xml 格式显示如下：

<?xml version="1.0" encoding="utf-8"?>
<params>
  <param name="param0">value_of_it</param>
  <param name="param1">value_of_it</param>
  <param name="something_else">you_get_it</param>
</params>

您更改现有参数值或向配置文件添加新参数的功能将如下所示：

internal void alterXml(string param, bool replace = false, string value = null)
{
    //load xml configuration file to XDocument object
    var doc = XDocument.Load(path);
    //search for <param> having attribute "name" = param
    var existingParam = doc.Descendants("param").FirstOrDefault(o => o.Attribute("name").Value == param);
    //if such a param element doesn't exist, add new element
    if (existingParam == null)
    {
        var newParam = new XElement("param");
        newParam.SetAttributeValue("name", param);
        newParam.Value = "" + value;
        doc.Root.Add(newParam);
    }
    //else update element's value
    else if (replace) existingParam.Value = "" + value;
    //save modified object back to xml file
    doc.Save(path);
}

Answer 2

如果您的进程中有该文件的三个句柄，那么您将多次打开该文件，或者在句柄保持打开状态的情况下多次调用该函数，或者您的处理存在幽灵版本。您应该仔细检查您没有在代码的其他地方打开此文件。当我在测试中运行您的代码时，它运行良好（如果我没有导致与下面的 Substring 函数相关的错误）。

因为“路径”处的文件是以读取模式打开的，所以其他阅读器可以毫无问题地打开它。但是，只要其他代码尝试写入操作（包括文件元数据，例如 File.Delete），您就会看到此错误。

您的代码不是异常安全的，因此当您在该循环中从流中读取数据时，由此引发的异常或类似函数将导致句柄保持打开状态，从而导致稍后调用打开文件的函数失败，您现在在 File.Delete 遇到的异常。为避免异常安全问题，请使用 try/finally 或 using（我将在下面提供示例）。

一个可能发生的异常是在循环中调用 Substring 的地方，因为行的长度有可能小于完整参数的长度。这里还有另一个问题，如果您传递的参数没有尾随空格，那么您可能会匹配包含第一个作为前缀的另一个参数（即“hello”将匹配“hello_world”）。

这是一个稍微修正过的代码版本，它是异常安全的，并且修复了 Substring 问题。

internal void alter( string param, bool replace = false, string value = null )
{
    //here
    string buf;
    string bufPath = path.Substring( 0, path.Length - Path.GetExtension( path ).Length ) + "_buf.txt";

    using ( System.IO.StreamReader reader = new System.IO.StreamReader( path ) )
    {
        string paramWithSpace = param + " ";

        using ( System.IO.StreamWriter writer = new StreamWriter( bufPath ) )
        {
            while ( ( buf = reader.ReadLine() ) != null )
            {
                if ( !buf.StartsWith( paramWithSpace ) )
                    writer.WriteLine( buf );
                else if ( replace )
                    writer.WriteLine( paramWithSpace + value );
            }
        }
    }
        //there

    File.Delete( path );
    File.Move( bufPath, path );
}

但是，您可能希望考虑将您的配置完全加载到内存中，在内存中对其进行多次更改，然后批量写入一次。这将为您提供更高的性能读/写配置（通常您必须一次编写多个配置更改）并且它还将简化您的代码。对于简单的实现，请使用通用 Dictionary 类。