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).