如果总是返回新实例，静态方法是线程安全的吗？

Question

我有以下课程：

public class SesionPasoProxy
{
    private static EntitySerializer _serializer = EntitySerializer.Create();

    public static string Obtener(Guid idSesion, int orden)
    {
        SesionPaso item = new SesionPaso();
        item.Orden = orden;
        item.IdSesion = idSesion;
        return _serializer.ToXml(item);

    }        
}


 public class EntitySerializer
 {
      private EntitySerializer(){}  
      public static EntitySerializer Create()
      {
         return EntitySerializer.Create("Test");
      }

       public static EntitySerializer Create(string serializationContextName)
       {
            EntitySerializer  instance = new EntitySerializer();
            instance.Name = serializationContextName;
            return instance;
       }
       public ToXml(SesionPaso x){ return x.toString();}
}

该代码是线程安全的吗？ SessionPasoProxy 在 ASP.NET WebForms aspx 页面中使用。所以它可以被同时调用。 如您所见，EntitySerializer 是静态的，但会返回新实例（不是单例） 你怎么看？ 我的单元测试显示是 thread.safe，但我完全不确定。

非常感谢

Answer 1

一般来说，静态方法与实例方法相比并不多或少是线程安全的。 但是如果它们只使用局部变量而不是字段，那么它们很可能是线程安全的（当然这取决于所使用的方法）。

在您的情况下，Create 是线程安全的，因为它总是返回一个新实例，并且没有使用其他可以同时从不同线程访问的（静态）字段。

但有一件事：你也应该将EntitySerializer.ToXml 设为静态，而不是在Obtener 中使用相同的实例。它不需要是实例方法：

public static string ToXml(SesionPaso x){ return x.toString();}

那么你就不需要static字段_serializer（至少不需要这个）：

public static string Obtener(Guid idSesion, int orden)
{
    SesionPaso item = new SesionPaso();
    item.Orden = orden;
    item.IdSesion = idSesion;
    return EntitySerializer.ToXml(item);
}