java - total - Contador de sesión con HttpSessionListener y acceso variable de conteo de sesión

total-=-- contador java

Question

Vi un ejemplo con el contador de sesiones en "Servlets principales y JavaServer Pages vol 2" de Sun.
Counter simplemente se basa en HttpSessionListener y aumenta / disminuye el número de sesiones con sessionCreated / sessionDestroyed :

public class SessionCounter implements HttpSessionListener {
   private int currentSessionCount = 0;

   public void sessionCreated(HttpSessionEvent event) {
       currentSessionCount++;
   }
   ...
   public int getTotalSessionCount() {
       return(totalSessionCount);
   }
   ... // counter decrement, self registering in context attribute etc.

El oyente se registra en el contexto, por lo que los servlets pueden acceder a él y obtener el valor del contador.
No hay un bloque sincronizado.
¿Es seguro que currentSessionCount no sea volatile ?

¿Puede currentSessionCount almacenarse en caché en el registro de la CPU y no ser visible con el valor exacto para otros subprocesos que sirven la solicitud con servlets ?

Answer 1

La especificación de Servlet 3.0 dice (§ 11.5):

Instancias de Escucha y Roscado

[...]

El contenedor no es necesario para sincronizar las notificaciones resultantes para atribuir clases de escucha. Las clases de escucha que mantienen el estado son responsables de la integridad de los datos y deben manejar este caso de manera explícita.

Entonces, no, el código no es seguro. Usar un AtomicCounter o sincronizar el acceso al contador lo corrige.

Hacerlo volátil no lo hace más seguro, porque ++ no es una operación atómica. Por lo tanto, cada otro subproceso verá el nuevo valor gracias a la volatilidad, pero es posible que aún pierda incrementos debido a una lectura de condición de carrera, y luego incremente el contador en paralelo.

Answer 2

Esto sería seguro.

public class HttpSessionListenerTest implements HttpSessionListener {
    final private AtomicInteger sessionCount = new AtomicInteger(0);

    @Override
    public void sessionCreated(HttpSessionEvent event) {
        sessionCount.incrementAndGet();
    }

    @Override
    public void sessionDestroyed(HttpSessionEvent event) {
        sessionCount.decrementAndGet();
    }

    public int getTotalSessionCount() {
        return sessionCount.get();
    }
}