在C#中确保静态方法的线程安全性

我有一些我目前在静态类/方法中的代码,但我想检查它是否是线程安全的。 从我读过的内容来看,我认为这应该没问题,但我脑海中的一些事情却说它可能不是。 我的网页的数据处理阶段使用外部Web服务来创建订单记录,这可能非常慢:可能是30-40秒,可能是5或10分钟(这不在我的手中)所以我要开火返回页面返回给用户,然后启动新线程,然后在处理完成后通过电子邮件发送给用户。 这是一个静态类/方法。 如果我的所有对象都是在特定方法中创建的(除了系统默认值,这是常见的)该方法应该是线程安全的,不应该。 所以,例如,如果我有

public static class ProcessOrder() { public static int GetOrderMaxSize() { return (....gets and parses ConfigurationManager.AppSettings["MaxOrderSize"]...); } public static bool CreateOrder(Order order) { XmlDocument xmlDoc = GetOrderXML(order); bool check = false; using (CreateOrderXML.Create xmlCo = new CreateOrderXML.Create()) { xmlCo.Timeout = 60000; System.Text.UTF8Encoding encoding = new System.Text.UTF8Encoding(); string xmlString = ""; using (StringWriter stringWriter = new StringWriter()) { using (XmlWriter xmlWriter = XmlWriter.Create(stringWriter)) { xmlDoc.WriteTo(xmlWriter); xmlWriter.Flush(); xmlString = stringWriter.GetStringBuilder().ToString(); } } byte[] bXMLOrder = encoding.GetBytes(xmlString); byte[] breturnMessage; check = xmlCo.Create(bXMLOrder, out breturnMessage); .... do something with return message } return check; } private static XmlDocument GetOrderXML(Order order) { ... creates an XML object for the order } } 

(CreateOrderXML是对Web服务URL /方法的服务引用)是否是线程安全的,特别是对于长时间运行(主要在xmlCo.Create(….)阶段)并发线程? 我明白,如果我开始使用类成员然后在方法中使用它们,这肯定会引入一个问题,不同的线程会覆盖这些值,但只要在方法中创建对象,它们应该没问题,应该是’他们?

它看起来不像你在那里访问任何共享数据; 您正在请求远程资源,并在每次执行此方法时构建一组唯一的数据。 那里不需要同步。

这里方法的每次执行都是创建局部变量 – 它是自己的副本。 所以什么都没有被分享。

如果静态方法不访问任何静态(类)数据,则它应该是线程安全的。 唯一可能的争用点是它可能使用的外部资源(例如文件或其他系统资源),以及传入的数据。只有您知道这种用法的上下文。

可能在争用中的事物的使用可以用lock或其他灵长类动物来序列化。 不要忘记以相同的顺序序列化资源,以免死锁。 如果您有一个使用资源A和B的方法:

 lock( latch_a ) { process(object_a) ; lock ( latch_b ) { process(object_a,object_b) ; } } 

和另一种做反过来的方法:

 lock( latch_b ) { process(object_b) ; lock ( latch_a ) { process(object_a,object_b) ; } } 

在某些时候,你的两个线程将会死锁,当每个线程需要一个资源时,另一个需要bofore,它可以放弃对所述资源的访问。

编辑注意:有关详细信息,请参阅lock语句的C#文档。 通常,锁定的对象表示(并且可能是)正在序列化的共享资源访问。 一种常见的模式是做类似的事情:

 class Widget { private static readonly object X = new object() ; public void Foo() { lock(X) { // Do work using shared resource } return ; } }