[SOLUCIONADO] Aplicacion encima procesos

[lucy87]

Buen día.
Espero alguien pueda orientarme un poquitin para resolver mi problema.

Tengo una aplicación web desarrollada con java (jsp, servlets) + oracle y hago reportes con iText.
Sucede que si se ejecuta un reporte en una sesión, y en otra sesión el mismo reporte, los resultados que se muestran se enciman :(

Ejemplo.
Si ejecuto el reporte de asistencia en una sesión del 01 al 31 de octubre, y en otra del 01 al 30 de enero, el reporte sale con las fechas así:
01/10/2014
02/10/2014
01/01/2014
03/10/2014
02/01/2014
03/01/2014

siendo que en un reporte solo debería venir de octubre, y otro de enero.
Muchas gracias y espero q alguien me pueda ayudar un poco

[Profesor_Falken]

Buenas,

Parece que tienes una condición de carrera.

¿No estarás utilizando por casualidad variables de instancia en los servlets?

Ten en cuenta que los servlets funcionan con una sola instancia, que se ejecuta en un pool de threads, por lo que las variables de instancia son compartidas por todas las peticiones/sesiones.

Un saludo

[lucy87]

:o Muchas gracias!

Asi declaro variables al inicio:

Código Java:

@WebServlet("/Reporte")
public class Reporte extends HttpServlet {
 
    private static final long serialVersionUID = 1L;
    private static Connection con = null;
    int idusuario;
    int numeroDias;
    int nopag = 0;
    int serie = 0;
    int vacac = 0;
.......
 

Ya después en el metodo doPost vuelvo a declarar otras variables donde convierto datos y asi... cual es la forma y el lugar correcto para declararlas y que no comparta las instancias entre sesiones??

Mil gracias.

[Profesor_Falken]

Buenas,

Deberias declarar esas variables como variables locales dentro del doPost (o del metodo en el que generas el informe, imposible de decir sin ver el codigo) y pasarlas como parametros a los metodos que las necesiten.

Respecto al objecto connection static, entiendo que abres la conexion en el init() del servlet y mantienes la conexion abierta siempre. Eso no es para nada eficiente, ya que todas la sesiones van a reusar la misma conexion y puede haber bloqueos. Ademas, si por algun motivo la conexion se corta, tendrias que reinciar el servidor para recuperar la aplicacion.
Lo mejor seria que lo implementes usando un pool de conexiones (en Tomcat, por ejemplo: http://tomcat.apache.org/tomcat-7.0-doc/jdbc-pool.html), aunque si tienes muy pocos usuarios supongo que como lo tienes te servira...


Un saludo

[lucy87]

Muchisiimas gracias por tu explicación!! efectivamente, al meter todas las variables en el doPost, y enviarlas como parámetro en los otros métodos que las necesitaban ya no me hacia eso a la hora de generar reportes al mismo tiempo.

Gracias por el dato del pool de conexiones, ya lo estoy implementando pero con glassfish.

Gracias de nuevo.
Saludos :)