[Oracle] nombre maximum de curseurs ouverts dépassé

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:

---

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 ?

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.

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 !

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 !

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

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

Code:

---

1 2 3 4 5 6 7 8

finally { if(rs != null) rs.close();   if(ps != null) ps.close(); }

---

oui oui, merci c'est fait

Salut,

Citation:

---

Envoyé par fr1man

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

Code:

---

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:

---

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:

---

catch (Throwable t) {}

---

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


a++