Discussion :

[Goupo]

Salut à tous,

Pourriez-vous me dire à quoi est due cette erreur?

Merci.

Code :

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

1
2
3
4
5
6
java.sql.SQLException: Instruction fermée: getMetaData
        at oracle.jdbc.driver.DatabaseError.throwSqlException(DatabaseError.java:112)
        at oracle.jdbc.driver.DatabaseError.throwSqlException(DatabaseError.java:146)
        at oracle.jdbc.driver.OracleResultSetImpl.getMetaData(OracleResultSetImpl.java:128)
        at cerbere.dao.pojo.DAO_Applications.getListApplication(DAO_Applications.java:44)
        at cerbere.bo.pojo.BO_Applications.getListApplications(BO_Applications.java:18)

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
 
Vector listeApplications = new Vector();
        String requete = 
                "select AP#***, AP#***, AP#*** " +
                "from AP#*** " +
                "order by AP#***";
try {
            resultSet = bdd.executeRequeteAndGetResults(requete);
 
            resultSetMetaData = resultSet.getMetaData(); /*<==Ligne de l'erreur*/
            int columnCount = resultSetMetaData.getColumnCount();
 
            while (resultSet.next()) {
                String[] ligne = new String[columnCount];
                for (int i = 0; i < columnCount; i++) {
                    ligne[i] = resultSet.getString(i + 1);
                }
                listeApplications.add(ligne);
            }
        } catch (SQLException e) {
            JOptionPane.showMessageDialog(null, "N'a pas pu récupérer la liste des applications\n" + e.getMessage(), "Erreur" , JOptionPane.OK_OPTION);
            e.printStackTrace();
        }

[natha]

Tu fais sûrement un close sur le ResultSet dans executeRequeteAndGetResults.

[Goupo]

A vrai dire, non et c'est ça qui me surprends.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
/**
     * Execute la requête et renvoie le resulSet
     * @param requete
     * @return
     */
    public ResultSet executeRequeteAndGetResults(String requete) throws SQLException {
        this.ouvre();
        Statement stmt = connexion.createStatement();           
        ResultSet resultSet = stmt.executeQuery(requete);
        this.ferme();
        System.out.println("Requête executée avec succès : \n" + requete);
        return resultSet;
    }

Code :

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

private Connection connexion;

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
private void ouvre() throws SQLException {
       if (this.properties == null) System.out.println("Properties null");
       String paramAdress = this.properties.getParametres("parametreOracle.connexion.adresse");
       String paramUser = properties.getParametres("parametreOracle.connexion.utilisateur");
       String paramPass = properties.getParametres("parametreOracle.connexion.passe");
 
       DriverManager.registerDriver(new oracle.jdbc.driver.OracleDriver());
       connexion = DriverManager.getConnection(paramAdress, paramUser, paramPass);
       System.out.println("Connecté à la base avec succès.");
    }
 
    private void ferme() throws SQLException {
        connexion.close();
        System.out.println("Fermeture de la base avec succès.");
    }

[natha]

Ah ben ouais tu fermes la connection, c'est pire !

[Goupo]

Ah! Je pensais que le ResultSet était indépendant de la connexion. Donc il faudrait que je fasse ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
/**
     * Execute la requête et renvoie le resulSet
     * @param requete
     * @return
     */
    public ResultSet executeRequeteAndGetResults(String requete) throws SQLException {
        this.ouvre();
        Statement stmt = connexion.createStatement();           
        ResultSet resultSet = stmt.executeQuery(requete);
        System.out.println("Requête executée avec succès : \n" + requete);
        return resultSet;
    }

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
 Vector listeApplications = new Vector();
        String requete = 
                "select AP#***, AP#***, AP#*** " +
                "from AP#*** " +
                "order by AP#***";
try {
            resultSet = bdd.executeRequeteAndGetResults(requete);
 
            resultSetMetaData = resultSet.getMetaData(); /*<==Ligne de l'erreur*/
            int columnCount = resultSetMetaData.getColumnCount();
 
            while (resultSet.next()) {
                String[] ligne = new String[columnCount];
                for (int i = 0; i < columnCount; i++) {
                    ligne[i] = resultSet.getString(i + 1);
                }
                listeApplications.add(ligne);
            }
        } catch (SQLException e) {
            JOptionPane.showMessageDialog(null, "N'a pas pu récupérer la liste des applications\n" + e.getMessage(), "Erreur" , JOptionPane.OK_OPTION);
            e.printStackTrace();
        } finally {
bdd.ferme();
}

Il n'y aurait pas un truc à lire sur le sujet? Pour en savoir plus.

Merci.

[natha]

Mouais, je sortirais aussi le "ouvre". Par exemple tu pourrais vouloir exécuter plusieurs requêtes sur la même connection (gain de perfs et possibilité de faire un rollback/commit global par exemple).

[Goupo]

Mouais, je sortirais aussi le "ouvre".

C'est déjà fait. (Mais certes pas posté).

Par exemple tu pourrais vouloir exécuter plusieurs requêtes sur la même connection (gain de perfs et possibilité de faire un rollback/commit global par exemple).

Ah super, ça me donne une idée! Merci!