linq - 使用 Linq-to-XML 插入，带线程

Question

在编写时确保线程安全的 Linq-to-XML 的最佳方法是什么？

最近我们的 Web 集群重载了，我不得不从我的屁股中取出一个快速的 overloaded.aspx 来捕获电子邮件，以便稍后当网站变得更加响应时可以与人们联系。在我短短 5 分钟的匆忙中，我写下了这个:

private static object LockHandle = new object();

protected override void OnLoad(EventArgs e)
{
  SubmitButton.ServerClick += new EventHandler(SubmitButton_ServerClick);
  base.OnLoad(e);
}

void SubmitButton_ServerClick(object sender, EventArgs e)
{
  string email = Email.Value.Trim();
  if (email.Length > 0)
  {
    Regex regex = new Regex(@"^([\w\-\.]+)@((\[([0-9]{1,3}\.){3}[0-9]{1,3}\])|(([\w\-]+\.)+)([a-zA-Z]{2,4}))$");
    if (regex.IsMatch(email))
    {
      lock (LockHandle)
      {
        try
        {
          string fileName = Server.MapPath("emails.xml");
          XDocument xdoc = XDocument.Load(fileName);
          xdoc.Element("emails").Add(new XElement("email", email));
          xdoc.Save(fileName);
        }
        catch {}
      }

      ResponseText.Text = "Thanks! We'll get back to you.";


    }
  }
}

我无法查询 Linq-to-XML 是否线程安全；所以，理论是“让我只锁定一个静态对象，这将防止多次写入。”这是最好的方法吗？甚至需要它(Linq-to-Xml 线程安全吗？)。遗憾的是，我的两本 Pro LINQ 和 Linq in Action 书籍没有涉及该主题。

这工作正常，我们在重载的 20m 范围内捕获了大量电子邮件。只是想知道是否有更好的方法；或者，如果一开始就锁定它就太过分了。

Answer 1

以这种方式保存文件本质上不是线程安全的，因此使用静态锁句柄是明智的。