Devo rendere statico questo XmlSerializer?

Question

Ho una classe che utilizza uno XmlSerializer nei suoi metodi Read/WriteXml. Il serializzatore è attualmente private readonly.

public class Foo : IXmlSerializable 
{ 
    private Bar _bar = new Bar(); 
    private readonly XmlSerializer serBar = new XmlSerializer (typeof (Bar)); 

    public void WriteXml (XmlWriter writer) 
    { 
     serBar.Serialize (writer, Bar); 
    } 
    // ... 
} 

Sto pensando di rendere il Serializzatore private static invece, in modo un caso è condivisa tra tutti i Foos. È una buona idea o ci sono problemi?

Answer 1

Sì, è una buona idea. No, non ci sono problemi con esso. In particolare, la sicurezza thread è non un problema - da MSDN documentation for XmlSerializer class:

Thread Safety

questo tipo è thread-safe.

Answer 2

Sì. In generale, vorrai farlo per tutte le tue classi di serializzazione. Si può drasticamente accelerare l'applicazione

Il modo più semplice per farlo è:

public static class MySerializers { 
    public static XmlSerializer MyType = new XmlSerializer(typeof(MyType));  
} 

Poi, quando hai bisogno di un serializzatore si può semplicemente chiamare:

MySerializers.MyType 

Si noti inoltre che in base al C# semantica, le classi statiche sono inizializzate al primo utilizzo, non al momento del caricamento. Se vuoi anticipare tutti i costi di caricamento, devi accedere esplicitamente alla classe.

Answer 3

Un modo potrebbe essere quello di creare una fabbrica XmlSerializers e fare riferimento a esso in modo statico (o come un riferimento IOC), qualcosa come:

public class XmlSerializerFactory 
{ 
    public XmlSerializer GetSerializerFor<T>() 
    { 
     lock (this) 
     { 
      Type typeOfT = typeof(T); 
      if (false == serializers.ContainsKey(typeOfT)) 
      { 
       XmlSerializer newSerializer = new XmlSerializer(typeOfT); 
       serializers.Add(typeOfT, newSerializer); 
      } 

      return serializers[typeOfT]; 
     } 
    } 

    private Dictionary<Type, XmlSerializer> serializers = new Dictionary<Type, XmlSerializer>(); 
} 

Answer 4

Secondo Neal - ancora più universale e sicuro attraverso Generics e in sola lettura:

public static class Helper<T> 
{ 
    public static readonly XmlSerializer Serializer = new XmlSerializer(typeof(T)); 
} 

Usa come:

Helper<My>.Serializer