Невозможно записать в файл XML с помощью потоков и XDocument

string filepath = Environment.CurrentDirectory + @"Expense.xml";

public void  WriteToXML(object param)
{
Expense exp = (Expense)param;
   if (File.Exists(filepath)) {
    XDocument xDocument = XDocument.Load(filepath);

    XElement root = xDocument.Element("Expenses");
    IEnumerable<XElement> rows = root.Descendants("Expense");
    XElement firstRow = rows.First();

    firstRow.AddBeforeSelf(new XElement("Expense",
           new XElement("Id", exp.Id.ToString()),
           new XElement("Amount", exp.Amount.ToString()),
           new XElement("Contact", exp.Contact),
           new XElement("Description", exp.Description),
           new XElement("Datetime", exp.Datetime)));
    xDocument.Save(filepath);
 }
}

Expense exp = new Expense();
exp.Id = new Random().Next(1, 10000);
exp.Amount = float.Parse(text1[count].Text);
exp.Contact = combo1[count].SelectedItem.ToString();
exp.Description = rtext1[count].Text.ToString();
exp.Datetime = DateTime.Now.ToString("MM-dd-yyyy");

workerThread = new Thread(newParameterizedThreadStart(WriteToXML));
workerThread.Start(exp); // throws System.IO.IOException

Я не могу записать в файл XML с рабочими гусеницами - я получаю эту ошибку:

System.IO.IOException: «Процесс не может получить доступ к файлу «C:\work\FinanceManagement\FinanceManagement\bin\DebugExpense.xml», поскольку он используется другим процессом.

но если я использую его как WriteToXML(exp);, он работает. Я думаю, что XDocument.Load(filepath) не является потокобезопасным. Как я могу решить эту проблему?


xml c# worker-thread linq-to-xml
person TRomesh    schedule 10.05.2019    source источник
comment
Разве первая строка не должна быть: string filepath = Path.Combine(Environment.CurrentDirectory, Expense.xml)? Кажется, вы имели в виду путь bin\Debug\Expense.xml   -  person Metheny    schedule 10.05.2019
comment
Есть ли более одного потока одновременно?   -  person Crowcoder    schedule 10.05.2019
comment
@Crowcoder я использовал File.Exists(filepath), чтобы проверить, существует ли файл. у меня есть обновление кода.   -  person TRomesh    schedule 10.05.2019
comment
File.Exists уязвим для условий гонки. Если у вас есть потоки, работающие очень близко друг к другу, вы можете получить несколько потоков в этом блоке кода одновременно.   -  person Crowcoder    schedule 10.05.2019
comment
@Metheny мой плохой . спасибо, что указали.   -  person TRomesh    schedule 10.05.2019
comment
@Crowcoder, я немного новичок в C#, можете ли вы помочь мне решить эту проблему?   -  person TRomesh    schedule 10.05.2019
comment
@TRomesh, если это действительно причина вашей проблемы, самый простой способ избежать этого — используйте lock. Или просто не переносить эту работу в новую тему. Какова причина этого?   -  person Crowcoder    schedule 10.05.2019
comment
@Crowcoder мне нужно прочитать/записать/обновить xml, используя threads. Вот почему я использовал этот метод.   -  person TRomesh    schedule 10.05.2019

Ответы (1)


arrow_upward
3
arrow_downward

Попробуйте ввести lock, посмотрите, решит ли это проблему:

// Declare this somewhere in your project, can be in same class as WriteToXML
static object XmlLocker;

Затем оберните логику lock:

public void WriteToXML(object param)
{
    Expense exp = (Expense)param;

    lock (XmlLocker) // <-- this limits one thread at a time
    {
        if (File.Exists(filepath))
        {
            XDocument xDocument = XDocument.Load(filepath);

            XElement root = xDocument.Element("Expenses");
            IEnumerable<XElement> rows = root.Descendants("Expense");
            XElement firstRow = rows.First();

            firstRow.AddBeforeSelf(new XElement("Expense",
                   new XElement("Id", exp.Id.ToString()),
                   new XElement("Amount", exp.Amount.ToString()),
                   new XElement("Contact", exp.Contact),
                   new XElement("Description", exp.Description),
                   new XElement("Datetime", exp.Datetime)));
            xDocument.Save(filepath);
        }
    }
}
person Crowcoder    schedule 10.05.2019