Удаление файлового исключения, удаление строки текста

Будет удобнее хранить конфигурацию в формате 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);
}

Если в вашем процессе есть три дескриптора этого файла, то вы открываете этот файл несколько раз или вызываете функцию несколько раз, оставляя дескриптор открытым, или у вас есть призрачные версии вашей обработки. Вы должны дважды проверить, что вы не открываете этот файл в другом месте вашего кода. Когда я запускаю ваш код в тесте, он работает нормально (если я не вызываю ошибку, связанную с функцией Substring ниже).

Поскольку файл по пути открыт в режиме чтения, его без проблем смогут открыть другие читатели. Однако, как только другой код попытается выполнить операцию записи (включая метаданные файла, например File.Delete), вы увидите эту ошибку.

Ваш код не является безопасным для исключений, поэтому исключение, вызванное этой или подобной функцией, когда вы читаете из потока в этом цикле, приведет к тому, что дескриптор останется открытым, что приведет к сбою более позднего вызова функции, которая открывает файл с исключение, которое вы сейчас испытываете в File.Delete. Чтобы избежать проблем с безопасностью исключений, используйте try/finally или using (ниже я приведу пример).

Одним из таких исключений, которое может возникнуть, является вызов Substring внутри цикла, поскольку есть вероятность, что длина строки меньше длины полного параметра. Здесь также есть еще одна проблема, заключающаяся в том, что если вы передаете параметр без завершающего пробела, то возможно, что вы совпадете с другим параметром, который содержит первый в качестве префикса (т. е. «hello» будет соответствовать «hello_world»).

Вот слегка исправленная версия кода, которая безопасна для исключений и устраняет проблему с подстрокой.

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.