Discussion :

[fripette]

Bonjour !
J'ai un problème avec mes fondamentaux décidément.
Je bloque encore sur un truc tout bête.

Je crée au début de mon script des lignes dans ma table et si elles ne sont pas remplies apres mon traitement alors je vais les effacer.
Voici le coté servlet:

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
void doActualisationFormDonnee(HttpServletRequest request,
			HttpServletResponse response) throws ServletException, IOException {
		Connexion updatePrlvmt= new Connexion();

		if (listIdPrelevement.isEmpty()){
			getServletContext().getRequestDispatcher((String) params.get("urlFormulaire")).forward(request,
					response);       
			return;
		}

		else{
//serveur,login,pwd,database
				updatePrlvmt.loadDriverAndConnect("127.0.0.1","3306","root","root","");

				//

				// ETAPE 1 :recuperation de la signification de chacun des indices
				for (int j=0; j<listIndice.length; j++) {
					String query="insert into CorrespIndiceEBM values ('"+NEtude +"','"+listIndice[j] +"','null','null','null','null','null','null'," +
					"'null','null','null','null','null','null')";
					updatePrlvmt.execute(query);

					for (int k=1; k<=listIndice.length; k++) {
			try{	[TRAITEMENT: INSERTION DES VALEURS DE L'UTILISATEUR DANS LES LIGNES ADEQUATES]
	// si apres la création de la ligne concernant la lettre, il n'y a pas eu d'information rentré (donc status='null') 
				// alors on supprime la ligne
				for (int m=0; m<listIndice.length; m++) {	
					String queryCheck="SELECT Corresp1,Corresp2,Corresp3,Corresp4" +
							" FROM CorrespIndiceEBM where Netude='"+NEtude+"' AND lettre='"+listIndice[m]+"'";
					
					ResultSet rsCheck=updatePrlvmt.ConnectAndQuestion(queryCheck);
					
					while (rsCheck.next()){
						String Corresp1= rsCheck.getString("Corresp1");
						String Corresp2= rsCheck.getString("Corresp2");
						String Corresp3= rsCheck.getString("Corresp3");
						String Corresp4= rsCheck.getString("Corresp4");
											
						if (Corresp1.equals("null") && Corresp2.equals("null") && Corresp3.equals("null") && Corresp4.equals("null")){
							updatePrlvmt.execute("delete FROM CorrespIndiceEBM " +
									" where Netude='"+NEtude+"' AND lettre='"+listIndice[m]+"'");
						}
						
					}
					
				}

Voilà le message d'erreur que j'ai :

Code :

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

1
2
3
4
5
6
7
8
9
 
Connected to :	jdbc:mysql://127.0.0.1:3306/BIOTECH
 
*** SQLException caught in main()
java.sql.SQLException: Operation not allowed after ResultSet closed
	at com.mysql.jdbc.ResultSet.checkClosed(ResultSet.java:652)
	at com.mysql.jdbc.ResultSet.next(ResultSet.java:5942)
	at com.mysql.jdbc.UpdatableResultSet.next(UpdatableResultSet.java:975)
	at germande.EBM.ServletDonneeResultEBM.doActualisationFormDonnee(ServletDonneeResultEBM.java:249)

Donc je ne comprend absolument pas ce qu'il se passe !! J'y comprend rien !
Je n'ai pas fermé la connection.
Par exemple quand seule les correspondances de la lettre "a" sont remplies : j'ai bien la ligne concernant la lettre "a "qui est gardé dans la table mais aussi "c" et "d" mais pas "b" !!
Une idée s'il vous plait ??
Merci d'avance !

[djsnipe]

Code :

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

ResultSet rsCheck=updatePrlvmt.ConnectAndQuestion(queryCheck);

Je suppose que ta méthode s'occupe d'exécuter la requête et de renvoyer le ResultSet. Le problème, c'est qu'elle doit également fermer le Statement utilisé pour la requête, et, foi de javadoc, si le Statement est fermé, le ResultSet l'est également. Du coup, impossible d'en extraire le contenu.

Donc soit tu changes la signature de ta méthode pour qu'elle te renvoi une liste d'objets complètement décorelée des couches JDBC, soit tu ne ferme pas le Statement, mais tu l'ajoute à une liste de ressources à libérer et tu ajoute une méthode close() générale à ton objet Connexion.
Pour éviter de bricoler un utilitaire, utilise des libs qui facilitent ce genre d'opérations, comme DbUtils

[fripette]

Il semble que ma méthode ne ferme pas le Rs:

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
 
public ResultSet ConnectAndQuestion(String query) throws Exception
		{
			try {
				String user="root";
				String pwd="root";
 
				Class.forName("com.mysql.jdbc.Driver").newInstance();
				String url=("jdbc:mysql://127.0.0.1:3306/BIOTECH");
 
				conn = DriverManager.getConnection(url, user,pwd);
 
				statement = conn.createStatement(
						ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_UPDATABLE );
				resultSet=statement.executeQuery(query);
				 // Print stdout warning messages if necessary
		         checkForSQLWarnings(conn.getWarnings());
 
			    // Print stdout info messages
		     }
			catch(SQLException e) {
			    System.err.println("\n*** SQLException caught in LoadDriver()");
			    printSQLErrors(e);
		         throw(SQLException) e;
		     }
		     catch(Exception e) {
			    // This exception is caught if JDBC driver used cannot be loaded
			    System.err.println("\n*** Exception caught in LoadDriver()"+e);
		         throw(Exception) e;
		     }
			return resultSet;
		}

[djsnipe]

C'est encore pire que ce que je pensais ....
Qui ferme les Statement, ResultSet et Connection alors Le GC , cf javadoc
Petit conseil, jette vite ta classe Connexion.

[fripette]

J'ai aussi une méthode dans ma classe connexion qui s'appelle close():

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
 
 /**
	  *	Close the connection to the data base source
	  */
	 public void close() throws Exception
	 {
		try {
		    resultSet.close();
		    statement.close();
		    conn.close();
 
 
		    System.out.println("Deconnexion...");
 
	     }
	     catch(Exception e) {
		    System.err.println("\n*** Exception caught in close()");
	        throw e; 
 
	     }
	 }

qui va donc être appelé apres tous mes traitements en fin de script ... rassurant ?

En tout cas, je ne vois pas pourquoi il y a ce message d'erreur

[djsnipe]

J'ai aussi une méthode dans ma classe connexion qui s'appelle close(): qui va donc être appelé apres tous mes traitements en fin de script ... rassurant ?

Oui !, j'ai fait mon , j'ai pas lu assez attentivement.
Toutefois, si ta méthode close() s'occupe uniquement d'un seul statement, ta classe n'aurait-elle pas un pb lors de la séquence suivante :

Code :

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

1
2
3
4
5
ResultSet rsCheck=updatePrlvmt.ConnectAndQuestion(requete1);
while (rsCheck.next()){
   [...]
   updatePrlvmt.execute(requete2);
}

On créer le statement pour la requete 1, puis on l'écrase pour créer celui de la requete 2 non ? CQFD ?