¿Debo declarar el ObjectMapper de Jackson como un campo estático?

Question

La biblioteca de Jackson ObjectMapper clase seems to be thread safe.

¿Quiere decir esto que debo declarar mi ObjectMapper como un campo estático como sigue

class Me { 
    private static final ObjectMapper mapper = new ObjectMapper(); 
} 

en lugar de como un campo de instancia como esta?

class Me { 
    private final ObjectMapper mapper = new ObjectMapper(); 
} 

Answer 1

Sí, es seguro y recomendable.

La única advertencia de la página que ha recomendado es que no puede modificar la configuración del asignador una vez que se ha compartido; pero no estás cambiando la configuración, así que está bien. Si necesitas cambiar la configuración, lo harías desde el bloque estático y también estaría bien.

EDITAR: (2013/10)

Con 2,0 y por encima, por encima se puede aumentar por señalar que hay una manera aún mejor: utilizar ObjectWriter y ObjectReader objetos, que puede ser construido por ObjectMapper. Son totalmente inmutables, seguros para subprocesos, lo que significa que ni siquiera es teóricamente posible provocar problemas de seguridad de subprocesos (que pueden ocurrir con ObjectMapper si el código intenta reconfigurar la instancia).

Answer 2

Aunque es seguro declarar un ObjectMapper estático en términos de seguridad de subprocesos, debe tener en cuenta que la construcción de variables de Objeto estáticas en Java se considera una mala práctica. Para obtener más información, consulte Why are static variables considered evil? (y, si lo desea, my answer)

En resumen, debe evitar las estáticas porque dificultan la realización de pruebas unitarias concisas. Por ejemplo, con un ObjectMapper estático final, no puede cambiar la serialización de JSON por un código ficticio o un no-op.

Además, un final estático le impide volver a configurar ObjectMapper en tiempo de ejecución. Puede que no visualice una razón para eso ahora, pero si se bloquea en un patrón final estático, nada menos que derribar el cargador de clases le permitirá volver a inicializarlo.

En el caso de ObjectMapper está bien, pero en general es una mala práctica y no hay ninguna ventaja sobre el uso de un patrón singleton o inversión de control para administrar sus objetos de larga vida.

Answer 3

Aunque ObjectMapper es seguro para subprocesos, desaconsejaría declararlo como una variable estática, especialmente en una aplicación multiproceso. Ni siquiera porque sea una mala práctica, sino porque corre un alto riesgo de bloqueo. Lo estoy contando desde mi propia experiencia. Creé una aplicación con 4 hilos idénticos que obtenían y procesaban datos JSON de los servicios web. Mi solicitud fue estancando con frecuencia en el siguiente orden, de acuerdo con el volcado de hilo:

Map aPage = mapper.readValue(reader, Map.class); 

Además de eso, el rendimiento no era bueno. Cuando reemplacé la variable estática con la variable basada en la instancia, el estancamiento desapareció y el rendimiento se cuadruplicó. Es decir. 2.4 millones de documentos JSON fueron procesados ​​en 40min.56sec., En lugar de 2.5 horas antes.

Answer 4

com.fasterxml.jackson.databind.type.TypeFactory._hashMapSuperInterfaceChain (HierarchicType)

com.fasterxml.jackson.databind.type.TypeFactory._findSuperInterfaceChain(Type, Class) 
    com.fasterxml.jackson.databind.type.TypeFactory._findSuperTypeChain(Class, Class) 
    com.fasterxml.jackson.databind.type.TypeFactory.findTypeParameters(Class, Class, TypeBindings) 
     com.fasterxml.jackson.databind.type.TypeFactory.findTypeParameters(JavaType, Class) 
      com.fasterxml.jackson.databind.type.TypeFactory._fromParamType(ParameterizedType, TypeBindings) 
       com.fasterxml.jackson.databind.type.TypeFactory._constructType(Type, TypeBindings) 
       com.fasterxml.jackson.databind.type.TypeFactory.constructType(TypeReference) 
        com.fasterxml.jackson.databind.ObjectMapper.convertValue(Object, TypeReference) 

El _hashMapSuperInterfaceChain método en la clase com.fasterxml.jackson.databind.type.TypeFactory está sincronizado. Estoy viendo contención en el mismo a altas cargas.

Puede ser otra razón para evitar un ObjectMapper estático