Discussion :

[PsychoZ]

Bonjour à tous, voici mon problème insoluble ...

Je l'ai testé sur différentes configuration et j'obtient toujours la même chose ... (Tomcat5 et 5.5 sur Ubuntu, Fedora, Windows et un serveur MySQL dernier modèle et le connecteur JDBC qui va avec ... ).


Pour faire simple mes connexion à la base sont gérées dans un Pool.
J'ai une servlet nommée CreateTempo qui se connecte et affiche le contenu d'une table et "close" la connexion.
J'ai enfin une page jsp qui forward vers la servlet.

Je démarre mon Tomcat, je lance ma page tempo.jsp
Le premier coup, j'ai ma liste ...

Je relance la page une seconde fois et là ... rien,
java.sql.SQLException: Connection is closed.

J'ai largement assez de connexion au niveau du pool, les timeout sont suffisamment longs etc etc etc...

Voici les codes si ça peut aider ...
CreateTempo.java

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
import java.io.IOException;
import java.io.PrintWriter;
import java.sql.ResultSet;
import java.sql.SQLException;
 
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import java.sql.Statement;
 
import frx.JdbcServletPA;
 
public class CreateTempo extends JdbcServletPA{
 
	private static final long serialVersionUID = 1L;
	private PrintWriter out;
 
	public void doGet(HttpServletRequest req,HttpServletResponse res) throws IOException,ServletException {
		out = res.getWriter();
		Statement stmt = null;
		try {
			stmt = conPA.createStatement();
			String query="SELECT id_,c1,c2 FROM p0_0";
			ResultSet rs=stmt.executeQuery(query);
			while(rs.next()){
				out.println(rs.getString("id_")+" - "+rs.getString("c1")+" - "+rs.getString("c2"));
			}
			stmt.close();
			conPA.close();
			//System.out.println("Close Connexion Base PA Tempo");
		} catch (SQLException e) {
			e.printStackTrace();
			if (stmt != null) {
				try {
					stmt.close();
				} catch (SQLException e1) {
					e1.printStackTrace();
				}
				stmt = null;
			}
			if (conPA != null) {
				try {
					conPA.close();
				} catch (SQLException e1) {
					e1.printStackTrace();
				}
				conPA = null;
			}
		}
	}
}

JdbcServletPA.java

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
package fx;
import java.sql.SQLException;
 
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
import javax.servlet.ServletConfig;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.sql.DataSource;
 
import java.sql.Connection;
 
/* Classe contenant les methodes utilises par les pages jsp */
 
public class JdbcServletPA extends HttpServlet{
	private static final long serialVersionUID = 1L;
	protected Connection conPA;
	protected DataSource dsA;
	public void init(ServletConfig conf) throws ServletException {
		super.init(conf);
		Context initCtx = null;
		try {
			initCtx = new InitialContext();
			dsA = (DataSource) initCtx.lookup("java:comp/env/jdbc/PoolPA");
		} catch (NamingException e) {
			e.printStackTrace();
		}
		try {
			Class.forName("com.mysql.jdbc.Driver");
		} catch (ClassNotFoundException e) {
			e.printStackTrace();
		}
		try {
			conPA =  dsA.getConnection();
			System.out.println("Open Connexion Base PA Servlet");
		} catch (SQLException e) {
			e.printStackTrace();
			if (conPA != null) {
				try {
					conPA.close();
				} catch (SQLException e1) {
					//
					e1.printStackTrace();
				}
				conPA = null;
			}
		}
	}
 
	public void destroy() {
		try {
			if(conPA!=null){
				if(!conPA.isClosed()){
					conPA.close();
					System.out.println("Close Connexion Base PA Servlet Destroy");
				}
			}
		} catch (SQLException e) {
			e.printStackTrace();
			if (conPA != null) {
				try {
					conPA.close();
				} catch (SQLException e1) {
					//
					e1.printStackTrace();
				}
				conPA = null;
			}
		}
	}
}

Si vous avez besoin d'autres infos ...

Merci.

[Duc Lebowski]

C'est ton DataSource qui va te donner une connexion du pool quand tu lui en demande une :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
Connection conPA = null;
Statement stmt = null;
try
{
    conPA = dsA.getConnection();
    stmt = conPA.createStatement();
    String query="SELECT id_,c1,c2 FROM p0_0";
    ResultSet rs=stmt.executeQuery(query);
    while(rs.next())
        out.println(rs.getString("id_")+" - "+rs.getString("c1")+" - "+rs.getString("c2"));
}
catch (SQLException sqlex)
{
     sqlex.prinStackTrace();
}
finally 
{
    if (stmt != null)
        try {stmt.close();}
        catch (SQLException sqlex) {ex.printStackTrace();}
    if (conPA != null)
        try {conPA.close();}
        catch (SQLException sqlex) {ex.printStackTrace();}
}
...

[OButterlin]

Il y a un problème dans ta manière de procéder :

Tu ouvres une connexion par servlet, si tu as 50 servlets, tu as 50 connexions ouvertes... pas top !

Tu n'as pas à faire un Class.forName(...), si tu n'as pas de connexion par la DataSource, c'est qu'elle n'est pas bien configurée...

N'oublie pas que les servlet sont multi-thread, si tu as 2 clients qui utilisent la même servlet, tu partages la connexion (pourvu qu'il n'y ait pas de transaction parce que le premier qui "tire" valide les insertions/modifications/suppressions en attentes à cet instant )

Bref, ça ne va pas...


A la limite, tu peux laisser la définition de la DataSource en l'état.
Par contre, il faut supprimer la Connection connA de la servlet et la mettre dans la méthode doGet (ou doPost ou service...)
Un truc du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public void doGet(...)
{
   Connection conPA = null;
   try
   {
      conPA = dsA.getConnection();
      ...
   }
   catch (...)
   {
      ...
   }
   finally
   {
      if ( conPA != null ) conPA.close();
   }
}
}

Voilà... A+

[PsychoZ]

Je vais me repencher là dessus ! Merci

[PsychoZ]

Pour ce qui est d'une ouverture de connexion par servlet, j'ai bien conscience que c'est pas top, mais comment veux tu que je m'y prenne là ...

Parce que je vois pas comment la servlet peut faire autrement ?

[OButterlin]

Le plus simple (à mon avis), c'est de passer par une classe Utils et une méthode getConnection() static (c'est très répendu comme usage), quelque chose comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
 
package a.preciser.utils;
 
import java.sql.Connection;
import java.sql.SQLException;
import javax.naming.InitialContext;
import javax.naming.Context;
import javax.sql.DataSource;
 
public class Utils
{
   private static DataSource dsA;
   private static Context initCtx;
 
   static
   {
       try
       {
           initCtx = new InitialContext();
           dsA = (DataSource) initCtx.lookup("java:comp/env/jdbc/PoolPA");
       }
       catch (Exception e)
       {
           // A FAIRE
       }
   }
 
   public static Connection getConnection() throws SQLException
   {
      return dsA.getConnection();
   }
}

Au niveau de l'usage dans les servlets (ou autres), il suffit de faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
...
public void doGet(...) throws Exception
{
   Connection conPA = null;
   try
   {
      conPA = Utils.getConnection();
...
   }
   catch (Exception e)
   {
...
   }
   finally
   {
      if ( conPA != null ) conPA.close();
   }
}

A toi de tester...
A+