Discussion :

[phoebe]

Tout est dans le titre, et ça vient de ce code ci-dessous mais je ne vois pas où exactement... quelqu'un a une idée ?

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
 
public static MyResultSet execute(Connection c,String sql,Object[] args) throws SQLException {
	ArrayList<HashMap> result = new ArrayList<HashMap>();
	try{
		PreparedStatement ps = perform(c, sql, args);
		ResultSet rs = ps.getResultSet();
		ResultSetMetaData rsmd = rs.getMetaData();
		HashMap<String,String> line;
		while (rs.next()) {
			line = new HashMap<String, String>();
			for (int i=1;i<=rsmd.getColumnCount();i++) line.put(rsmd.getColumnName(i),rs.getString(i));
			result.add(line);
		}
		rsmd = null;
		rs.close();
		ps.close();
	} catch (Throwable t) {}
 
	MyResultSet mrs = new MyResultSet();
	mrs.setListe(result);
	return mrs;
}

c'est la méthode que j'appele pour toutes mes requêtes, et lors d' "inserts" massifs mon nombre de curseurs ouverts déborde à priori...

je précise que le problème ne vient pas de la ligne "PreparedStatement ps = perform(c, sql, args);" j'ai procédé à quelques tests déjà

ça ne viendrait pas de l'utilisation de ResultSetMetaData ?

[fr1man]

Ce n'est peut-être pas directement la cause de ton problème, en tout cas, la fermeture de tes objets rs et ps serait mieux dans un bloc finally pour être exécutée même en cas d'exception arrivant avant.

[phoebe]

c'est exactement ça, je venais juste de trouver, vu que j'exécute toutes mes requêtes avec cette méthode, même les "inserts"... le rs était null et donc une exception était levée, j'ai donc ajouté un rs.close() et ps.close dans mon catch

merci fr1man !

[DevTeam_]

Comme l'a dit fr1man, préfère le finally : ça t'évite d'avoir 2 fois le code de fermeture des rs/pst. De plus, pense à tester le not null avant de faire le .close !

[phoebe]

pour le not null je l'ai fait c'était pour faire court, Ok pour le finally

[fr1man]

Tant mieux.
En tout cas, je tiens au bloc finally ce qui t'évite de dupliquer les *.close().

Code :

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

1
2
3
4
5
6
7
8
 
finally {
 if(rs != null)
  rs.close();
 
 if(ps != null)
  ps.close();
}

[phoebe]

oui oui, merci c'est fait

[adiGuba]

Salut,

Tant mieux.
En tout cas, je tiens au bloc finally ce qui t'évite de dupliquer les *.close().

Code :

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

1
2
3
4
5
6
7
8
 
finally {
 if(rs != null)
  rs.close();
 
 if(ps != null)
  ps.close();
}

Perso, je te conseillerais plutôt d'utiliser un finally par élément, ce qui donnerait une structure 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
	PreparedStatement ps = ...;
	try {
 
		ResultSet rs = ps.executeQuery();
		try {
 
			while (rs.next()) {
				...
			}
 
		} finally {
			rs.close();
		}
 
	} finally {
		ps.close();
	}

Car il pourrait très bien y avoir un exception dans le premier close(), et dans ce cas là avec ton code le finally ne se termine pas correctement...



Enfin phoebe, dans ton code il y a ceci qui est une abomination sans nom :

Code :

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

catch (Throwable t) {}

On ne doit jamais ignorer une exception !!!!!!!!!


a++