Каков наилучший способ синхронизации доступа XmlWriter к файлу для предотвращения исключений IOException?

Question

В приложении есть несколько мест, которые вызывают XmlWriter.Create для одного и того же файла, и все они доступны через следующую функцию. Когда один звонит, а другой все еще пишет, я получаю IOException. Какой лучший способ заблокировать или синхронизировать доступ?

Вот функция, которая используется:

        public void SaveToDisk()
    {
        try
        {
            XmlWriterSettings settings = new XmlWriterSettings();
            settings.Indent = true;
            using (XmlWriter writer = XmlWriter.Create(SaveFileAbsolutePath, settings))
            {
                XamlWriter.Save(this, writer);
                writer.Close();
            }

        }
        catch (Exception ex)
        {
            // Log the error
            System.Diagnostics.Debug.WriteLine(ex.Message);

            // Rethrow so we know about the error
            throw;
        }
    }

ОБНОВЛЕНИЕ: похоже, что проблема не только в вызовах этой функции, а в том, что другой поток читает файл, пока эта функция пишет в is. Какой лучший способ блокировки, чтобы мы не пытались записать файл во время его чтения?

Answer 1

Использование блокировки может решить вашу проблему параллелизма и, таким образом, избежать исключения IOException, но вы должны помнить об использовании одного и того же объекта в SaveToDisk и ReadFromDisk (я предполагаю, что это функция чтения), в противном случае блокировку полностью бесполезно, только когда вы читаете.

private static readonly object syncLock = new object();

public void SaveToDisk()
{
     lock(syncLock)
     {
          ... write code ...
     }
}

public void ReadFromDisk()
{
     lock(syncLock)
     {
          ... read code ...
     }
}

Answer 2

Статическая блокировка должна выполнять работу быстро и просто:

private static readonly object syncLock = new object();

затем...

public void SaveToDisk()
{
    lock(syncLock)
    {
        ...your code...
    }
}

Вы также можете использовать [MethodImpl(MethodImplOptions.Synchronized)] (для статического метода, который принимает экземпляр в качестве аргумента - например, метод расширения), но явная блокировка более универсальна.

Answer 3

Я бы на самом деле использовал ReaderWriterLock для максимизации параллелизма. Вы можете разрешить несколько читателей, но только один писатель одновременно.

private ReaderWriterLock myLock = new ReaderWriterLock();

public void SaveToDisk()
{
     myLock.AcquireWriterLock();
     try
     {
          ... write code ...
     } 
     finally
     {
          myLock.ReleaseWriterLock();
     }
}

public void ReadFromDisk()
{
     myLock.AcquireReaderLock();
     try
     {
          ... read code ...
     } 
     finally
     {
          myLock.ReleaseReaderLock();
     }
}

Просто убедитесь, что файл открыт с помощью FileShare.Read, чтобы последующее чтение не завершилось неудачей.