¿Está bien usar la instancia de Gson como un campo estático en un bean modelo (reutilizar)?

Question 1

Aquí está el modelo que implementé:

public class LoginSession {
    private static final Gson gson = new Gson();

    private String id;
    private String name;
    private long timestamp;

    public LoginSession(String id, String name) {
        this.id = id;
        this.name = name;
        this.timestamp = System.currentTimeMillis();
    }

    public String toJson() {
        return gson.toJson(this);
    }

    public static LoginSession fromJson(String json) {
        checkArgument(!isNullOrEmpty(json));
        return gson.fromJson(json, LoginSession.class);
    }
}

Pensé que era inútil crear una nueva instancia de Gson para cada instancia de LoginSession.

Pero lo que me preocupa son los problemas de seguridad de subprocesos. Se crearán aproximadamente más de 1000 instancias por segundo.

¿Está bien usar la instancia de Gson como campo estático?

Gracias por cualquier consejo/correcciones.

Question 2

Me parece bien. No hay nada en la instancia de GSON que la relacione con una instancia específica de LoginSessionpor lo que debe ser estático.

Instancias GSON debe ser seguro para subprocesosy había un error con respecto a lo que estaba arreglado.

Question 3

El núcleo Gson la clase es segura para subprocesos. Acabo de encontrar un problema de seguridad de subprocesos que supuestamente estaba relacionado con GSON. El problema ocurrió al usar un personalizado JsonDeserializer y JsonSerializer por Date análisis y formateo. Al final resultó que, el problema de seguridad de subprocesos estaba relacionado con el uso de mi método de una estática SimpleDateFormat instancia que no es segura para subprocesos. Una vez que envolví la estática SimpleDateFormat en un ThreadLocal ejemplo, todo salió bien.

Question 4

Según los comentarios, la prueba de unidad existente realmente no prueba mucho, tenga cuidado con todo lo relacionado con la seguridad de subprocesos…

Hay un prueba de unidad comprobando la seguridad del subproceso:

/**
 * Tests for ensuring Gson thread-safety.
 *
 * @author Inderjeet Singh
 * @author Joel Leitch
 */
public class ConcurrencyTest extends TestCase {
  private Gson gson;
  ...

Quizás se pregunte si esta prueba unitaria es suficiente para encontrar todos los problemas posibles en todas las configuraciones posibles de la máquina. ¿Algún comentario sobre esto?

También está esta frase en el documentos:

La instancia de Gson no mantiene ningún estado mientras invoca las operaciones de Json. Por lo tanto, puede reutilizar el mismo objeto para múltiples operaciones de serialización y deserialización de Json.

Question 5

Tuvimos problemas con la seguridad de subprocesos hace un tiempo y los resolvimos usando FastDateFormat en apache commons.

Acabo de crear una esencia Enlace para Gist alrededor de esto para ayudar a las personas que se preguntan si las instancias de Gson se pueden reutilizar. No tienen setters y todas las vars son privadas.

Entonces, aparte del problema de SimpleDateFormat, no los veo manteniendo el estado en ningún otro lugar.

verifique eso afuera. Esta es la primera vez que respondo a uno de estos. Feliz de devolver por una vez. 🙂

¿Ha sido útil esta solución?