Discussion :

[hbellahc]

Bonjour à tous

J'ai un petit souci JDBC avec ORACLE.

En fait dans un projet JAVA, j'ai un package qui contient que des classes pour les accès à la base de données, toutes ses classes se ressemblent avec celle-ci :

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
public class ContentAreaData {

    private Statement stmt;
    private ResultSet rset;
    private Connection conn;
    

    public Vector <Folders> listeFolders(String caid)  {
        Vector<Folders> resultat = new Vector<Folders>();
        StringBuffer requete ="MaRequete";
        try {
            rset = stmt.executeQuery(requete.toString());
            while (rset.next()) {
                Folders folderTmp = new Folders(rset);
                resultat.add(folderTmp);
            }
            rset.close();
        } catch (SQLException e) {
           System.out.println("Erreur de selection "+ e.getMessage());
        }        
        return resultat;
    }
}

Elle a trois attributs : ResultSet, Statement et Connexion,
Le Statement et le Connection sont initialisés dans le constructeur, quand au ResultSet il est initialisé dans les méthodes qui permettent de récupérer des données de la base de données, une fois que j'ai récupéré mes résultats je ferme le ResultSet.

Mais le problème c'est que j'ai l'impression que le resultSet n'est pas fermé, car quand j'ai beaucoup de requêtes à faire j'ai cette erreur qui sort :

Code :

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

1
2
 
ORA-01000: Nombre maximum de curseurs ouverts atteint

Maintenant je ne sais si c'est le statement que je dois fermer aussi, ou bien il n’y'a pas d'autres solutions que d'augmenter le nombre de curseurs dans la config d'Oracle, Alors que si les ResultSets sont bien fermé, je ne doit pas avoir plus de 100 ouverts au même temps.

Merci de votre aide.
Cordialement hbellahc

[adiGuba]

Salut,


Est-ce que tu passes souvent dans le bloc catch ? Car dans ce cas là le ResultSet n'est pas fermé

Je te conseille fortement d'utiliser des try/finally pour toujours libérer tes ressources au plus tôt...

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
 
    public Vector <Folders> listeFolders(String caid)  {
        Vector<Folders> resultat = new Vector<Folders>();
        StringBuffer requete ="MaRequete";
        try {
            ResultSet rset = stmt.executeQuery(requete.toString());
            try {
                  while (rset.next()) {
                      Folders folderTmp = new Folders(rset);
                      resultat.add(folderTmp);
            } finally {
                  rset.close();
            }
        } catch (SQLException e) {
           System.out.println("Erreur de selection "+ e.getMessage());
        }        
        return resultat;
    }

Plus d'info dans la FAQ : Comment libérer proprement les ressources (ou comment utiliser proprement les bloc try/finally) ?

Plus généralement je te conseille fortement d'utiliser cela pour tous les types de ressources à libérer...



De plus je te conseille fortement d'éviter d'utiliser ces éléments en tant qu'attribut de classe mais de limiter au maximum leurs scopes. Dans cet exemple le ResultSet n'a pas à être utilisé en dehors de la méthode, alors pourquoi est-il déclaré en dehors...

Et il y a de forte chance que le problème soit également présent pour les Statement/Connection...

a++

[in]

Je ne sais pas si ton erreur vient de là mais il est fortement conseillé de fermer ton ResultSet dans un bloc finally. [EDIT] Ha je sentais bien qu'adiGuba allait me griller, je le sentais ...


Sinon, 2ème point, Vector est une vieille classe. si tu n'es pas en environnement multithread et que tu veux être au gout du jour utilise plutôt une ArrayList

Enfin, ceci me paraît (très) bizarre :

Code :

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

1
2
3
4
 rset = stmt.executeQuery(requete.toString());
            while (rset.next()) {
                Folders folderTmp = new Folders(rset);  // pourquoi parcourir le ResultSet si tu t'en sers pour construire un objet ???
                ...

Soit ut parcours le resultSet, soit tu t'en sers pour construire un autre objet, mais là ce que tu fais ne sers à rien et est sûrement la cause de ton problème ...

[hbellahc]

Soit ut parcours le resultSet, soit tu t'en sers pour construire un autre objet, mais là ce que tu fais ne sers à rien et est sûrement la cause de ton problème ...

En fait je crée un vecteur d'objets, car le constructeur de Folders permet de créer un objet Folders en utilisant le resultat que je trouve dans le rset.

Sinon j'ai essayé la solution du try{..} finally{} mais ça ne resoud pas le problème.

[in]

En fait je crée un vecteur d'objets, car le constructeur de Folders permet de créer un objet Folders en utilisant le resultat que je trouve dans le rset.

Peux tu nous montrer ce constructeur.

En tous cas ça ne sert à rien de faire

Code :

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

1
2
while(rs.next){ // <-- ça ne sert à rien de parcourir
    new MonObject(rs)

puisque je suppose que tu parcours le resultset dans le constructeur de folder. ton erreur est dûe au fait que tu essaie de parcourir ton resultset à plusieurs endroits de ton code (dans deux classes en l'occurrence) ... enfin je suppose

[adiGuba]

Sinon j'ai essayé la solution du try{..} finally{} mais ça ne resoud pas le problème.

Est-ce que tu l'a bien généralisé à toutes tes ressources ? Car si le problèmes survient ici il peut être posé par d'autres ressources mal fermées...

Et tu ne m'a pas répondu : est-ce que tu rentres dans des bloc catch pendant ton application ???

puisque je suppose que tu parcours le resultset dans le constructeur de folder.

Je pense plutôt qu'il utilise le constructeur pour récupérer les données de la ligne courante... Quelque chose du style :

Code :

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

1
2
3
4
public Folder(ResultSet rs) {
	this.name = rs.getString("name");
	this.id = rs.getInt("id");
}

a++

[in]

Je pense plutôt qu'il utilise le constructeur pour récupérer les données de la ligne courante... Quelque chose du style :

Code :

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

1
2
3
4
public Folder(ResultSet rs) {
	this.name = rs.getString("name");
	this.id = rs.getInt("id");
}

Ha ok ... ben ça me semblait bizarre mais présenté comme ça effectivement ça à plus de sens ...

mea culpa

[hbellahc]

Est-ce que tu l'a bien généralisé à toutes tes ressources ? Car si le problèmes survient ici il peut être posé par d'autres ressources mal fermées...

Oui je l'ai fait sur toutes mes méthodes ou j'effectue des requêtes, c'est à dire partout ou j'ai des ResultSet.

Et tu ne m'a pas répondu : est-ce que tu rentres dans des bloc catch pendant ton application ???

Il rentre dans le catch si jamais y'a une erreur, mais ça arrive justement dans le cas ou le nombre de curseurs ouverts est atteint.

Je pense plutôt qu'il utilise le constructeur pour récupérer les données de la ligne courante... Quelque chose du style :

Code :

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

1
2
3
4
public Folder(ResultSet rs) {
    this.name = rs.getString("name");
    this.id = rs.getInt("id");
}

a++

Exact : c'est ce que je fais dans mes constructeur.



Merci en tous cas, je vais essayer de suivre le conseil sur le ResultSet, je le déclare que dans mes fonctions et je vous tiens au courant si ça marche.

Merci encore

[adiGuba]

Oui je l'ai fait sur toutes mes méthodes ou j'effectue des requêtes, c'est à dire partout ou j'ai des ResultSet.

Fais le également pour les Statement et Connection... Pour être sûr

a++

[hbellahc]

Fais le également pour les Statement et Connection... Pour être sûr

a++

Bonjour à tous

Milles merci les amis (adiGuba et in) ça marche super bien, en fait j'ai fait ce que tu m'as proposé adiGuba, mes classes ressemblent à celle-ci maintenant

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
 
public class ContentAreaData {
 
 
    private Connection conn;
 
 
    public Vector <Folders> listeFolders(String caid)  {
    Statement stmt=null;
    ResultSet rset=null;
    Vector<Folders> resultat = new Vector<Folders>();
    StringBuffer requete = new StringBuffer("SELECT * "
            + "FROM KWEB_FOLDERS " + "WHERE PARENT_ID=0 " + "AND CAID="
            + caid);
    try {
        stmt = conn.createStatement();
        rset = stmt.executeQuery(requete.toString());
        try {
 
            while (rset.next()) {
                Folders folderTmp = new Folders(rset);
                resultat.add(folderTmp);
            }
        }finally{
            if (rset!=null) rset.close();
            if (stmt!=null) stmt.close();
        }            
    } catch (SQLException e) {
        logger.warn("Erreur de selection "+ e.getMessage());
        System.out.println("Erreur de selection "+ e.getMessage());
    }        
    return resultat;
    }
}

Et ça marche super bien, le nombre de curseurs ouverts reste le même pendant toute l'exécution de mes actions sur l'application.

Pour ceux ça intéresse, j'ai trouvé une fonction qui permet d'afficher justement le nombre de curseurs ouverts :

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
public static int getCurrentOpenCursors(Connection conn) {
            PreparedStatement psQuery = null;
            ResultSet rs = null;
            
            int cursors = -1;
            try {
              String sqlQuery = "select count(*) AS COUNT from v$open_cursor where user_name like 'NOM DE L'utilsateur '";
              psQuery = conn.prepareStatement(sqlQuery);
              rs = psQuery.executeQuery();
              
              if (rs.next()) {
                cursors = rs.getInt("COUNT");
              }
            } catch (SQLException e) {
              System.out.println("SQLException in getCurrentOpenCursors(Connection conn): "+e);
            } finally {
              try {
                if (rs != null) {rs.close();}
                if (psQuery != null) {psQuery.close();}
              }
              catch (SQLException ex)  {
                System.out.println("A SQLException error has occured in getCurrentOpenCursors(Connection conn): " + ex.getMessage());
                ex.printStackTrace();
              }
            }
            return cursors;
          }

et aussi comme j'avais beaucoup de classes à modifier, j'ai crée un template pour aller plus vite (pour les gens qui utilisent eclipse ou netbeans):

Code :

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

1
2
3
4
5
6
7
8
9
 
try {
    ${line_selection}${cursor}
} finally {
    if (rset!=null)
        rset.close();
    if (stmt!=null)
       stmt.close();
}

Pour l'utiliser il faut la crée comme indiqué sur ce lien http://java.developpez.com/faq/eclip...creerTemplates

Ensuite Alt+Ctrl+z et choisir le nom que vous avez donné à votre template.

Voila merci à tous en tous cas.
Cordialement hbellahc

[adiGuba]

Juste une remarque : il est possible d'englober les bloc try/catch afin de bien séparer chaque ressource et d'éviter les vérifications de la nullité :

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
	public Vector<Folders> listeFolders(String caid) {
		Vector<Folders> resultat = new Vector<Folders>();
		StringBuffer requete = new StringBuffer("SELECT * "
				+ "FROM KWEB_FOLDERS " + "WHERE PARENT_ID=0 " + "AND CAID="
				+ caid);
		try {
			Statement stmt = conn.createStatement();
			try {
				ResultSet rset = stmt.executeQuery(requete.toString());
				try {
					while (rset.next()) {
						Folders folderTmp = new Folders(rset);
						resultat.add(folderTmp);
					}
				} finally {
					rset.close();
				}
			} finally {
				stmt.close();
			}
		} catch (SQLException e) {
			// logger.warn("Erreur de selection "+ e.getMessage());
			System.out.println("Erreur de selection " + e.getMessage());
		}
		return resultat;
	}

Perso je préfère (mais bon on est d'accord cela n'est plus qu'une question de goût )

a++