Operation not allowed after ResultSet closed

**Hind4Dev** · 16/08/2015, 16h37

j'appelle 4 méthodes d'insertion dans le bouton sauvegarde, je reçoit cette exception

Nom : cc.jpg
Affichages : 258
Taille : 82,3 Ko

quand je regarde ma base je trouve 2 tables bien remplies mais 2 non, voilà les 2 méthodes d'insertion de ces 2 tables:

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
 
@SuppressWarnings("deprecation")
    public void insertEtudiant() throws Exception{
        try{
            int codeLieu = getLieu();
            int codeNat = getNat();
            String codeF = getFiliere();
            int codeMotif = getMotif();
 
            String query = "INSERT INTO etudiant (bac,nom,prenom,genre,adresse,n_phone,mail,n_ccp,decision,bloque,"
                    + "n_ins,annee__encours,annee_univ,annee_courante,reste,"
                    + "an_1,an_2,an_3,an_4,an_5,an_6,an_7,an_8,an_9,an_10,"
                    + "cycle,nbr_annee,code_f,date_nais,code,code_nat,Observation,date_bloc,code_motif) "
                    + "VALUES (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)";
 
 
            stat = cnx.prepareStatement(query);
            stat.setString(1, txtBAC.getText());
            stat.setString(2, txtNom.getText());
            stat.setString(3, txtPrenom.getText());
            stat.setString(4, genre);
            stat.setString(5, txtAdr.getText());
            stat.setString(6, txtPhone.getText());
            stat.setString(7, txtMail.getText());
            stat.setString(8, (String)txtCCP.getValue());
            stat.setString(9, txtDec.getText());
            stat.setBoolean(10, bloc);
            stat.setString(11, txtIns.getText());
            stat.setString(12, txtAn.getText());
            stat.setString(13, txtUniv.getText());
            stat.setInt(14,new java.sql.Date(System.currentTimeMillis()).getYear());
            stat.setInt(15, reste);
            //stat.setInt(6, rds);
            stat.setInt(16, 0);
            stat.setInt(17, 0);
            stat.setInt(18, 0);
            stat.setInt(19, 0);
            stat.setInt(20, 0);
            stat.setInt(21, 0);
            stat.setInt(22, 0);
            stat.setInt(23, 0);
            stat.setInt(24, 0);
            stat.setInt(25, 0);
            stat.setString(26,txtBAC.getText());
            stat.setString(27, codeF);
            stat.setString(28, cycle);
            stat.setString(29, ((JTextField) dateChooser.getDateEditor().getUiComponent()).getText());
            stat.setInt(30, codeLieu);
            stat.setInt(31, codeNat);
            //stat.setInt(14, codePhoto);
            stat.setString(32, txtObs.getText());
            stat.setDate(33,new java.sql.Date(System.currentTimeMillis()));
            stat.setInt(34, codeMotif);
 
            stat.executeQuery();
 
 
        }catch(SQLException ex){
            JOptionPane.showMessageDialog(null,"Problème insertEtudiant");
        }
        finally{
                stat.close();
        }
    }
 
    public void insertFils() throws SQLException{
 
        try{
            int codeP = insertPere();
            int codeM = insertMere();
 
            String query = "INSERT INTO fils_de (bac,code_p,code_m) VALUES (?,?,?)";
            stat = cnx.prepareStatement(query);
            stat.setString(1,txtBAC.getText());
            stat.setInt(2, codeP);
            stat.setInt(3, codeM);
 
            stat.executeQuery();
 
        }catch(Exception ex){
            JOptionPane.showMessageDialog(null, "Problème insertFils");
        }
        finally{
            stat.close();
        }
    }

**joel.drigo** · 16/08/2015, 17h15

Le message d'erreur indique que tu cherches à lire des enregistrements dans un ResultSet alors qu'il a été fermé.

Comment sont codées ces méthodes :

Code :

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

1
2
3
4
int codeLieu = getLieu();
int codeNat = getNat();
String codeF = getFiliere();
int codeMotif = getMotif();

Ne sont-ce pas ces méthodes qui liraient des données depuis un ResultSet fermé ?

Même question pour ces deux-là :

Code :

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

1
2
int codeP = insertPere();
int codeM = insertMere();

**Hind4Dev** · 16/08/2015, 18h26

elle sont codées de cette façon toutes juste la table et le champ retourné qui diffère:

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
 
public int getNat() throws Exception{
		String query = "SELECT code_nat FROM nationalite WHERE nationalite=?";
		   try ( PreparedStatement stat = cnx.prepareStatement(query) ) {
 
			  stat.setString(1, (String) cmbNat.getSelectedItem());
			  rslt = stat.executeQuery();
		      rslt.next();
		      return rslt.getInt(1);
 
		   }catch(SQLException ex){
		          ex.printStackTrace();
		          return 0; 
		   }
	}

**joel.drigo** · 16/08/2015, 18h33

Elles sont bien toutes comme ça ?

Parce que si par exemple, cette méthode était comme suit, tu obtiendrais l'exception en question :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
String query = "SELECT code_nat FROM nationalite WHERE nationalite=?";
try (PreparedStatement stat = cnx.prepareStatement(query)) {
 
    stat.setString(1, (String) cmbNat.getSelectedItem());
    rslt = stat.executeQuery();
    rslt.next();
 
} catch (SQLException ex) {
   ex.printStackTrace();
   return 0;
}
return rslt.getInt(1);

Avec le try-with-resource, le PreparedStatement est fermé en dehors du bloc du try, et tous les ResultSets créés à partir de statement le sont également.

Pourquoi cette variable rslt est-elle déclarée en dehors du bloc du try-with-resource ? Le meilleur moyen d'empêcher le risque d'y accèder ailleurs est de la déclarer dans ce bloc.

**Hind4Dev** · 16/08/2015, 20h22

elles sont toutes de cette façon , non le return est dans le bloc non pas ailleur , mais la déclaration est juste après les déclarations des attributs en haut avant toute méthode comme suit:

Code :

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

1
2
3
4
 
Connection cnx = Connexion.ConnectDB();
	PreparedStatement stat = null;
	ResultSet rslt = null;

**joel.drigo** · 16/08/2015, 20h45

Et tu n'appelles nulle part close() sur le ResultSet ou le PreparedStatement ?

**tchize_** · 16/08/2015, 20h56

Le resultset (rslt) n'a rien à faire comme champ de classe. Retire les des champs et déclare le dans les blocs de tes méthodes. Ca te permettra d'éviter l'erreur de le réutiliser alors qu'il a été fermé.

**Hind4Dev** · 16/08/2015, 21h02

alors dans chaque try je ferme le statement mais après l'insertion avant le catch dans le try lui même comme dans le code affiché, mais le resultset j'ai cherché dans tout mon code pas de cloture

mais j'ai deux fonctions d'insertion du père et mère où il y'a 2 statements mais les données s'insèrent normal dans ces deux tables dont voici le code

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 int insertPere() throws Exception{        try{
            int codeProfP = getProfP();
 
            String query = "INSERT INTO pere (nom_pere , revenu_pere,code_prof) VALUES (?,?,?)";
            stat = cnx.prepareStatement(query);
            stat.setString(1, txtPere.getText());
            stat.setString(2, txtRevP.getText());
            stat.setInt(3, codeProfP);
 
            stat.executeUpdate();
            stat.close();
 
            String codeQuery = "SELECT MAX(code_p) FROM pere";
            stat = cnx.prepareStatement(codeQuery);
            rslt = stat.executeQuery();
            rslt.next();
 
            stat.executeQuery();
            stat.close();
            return rslt.getInt(1);
 
        }catch(Exception ex){
            JOptionPane.showMessageDialog(null, ex);
            return 0;
        }finally{
 
        }
    }

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 int insertMere() throws Exception{        try{
            int codeProfM = getProfM();  
 
            String query = "INSERT INTO mere (nom_mere , revenu_mere,code_prof) VALUES (?,?,?)"; 
            stat = cnx.prepareStatement(query);
            stat.setString(1, txtMere.getText());
            stat.setString(2, txtRevM.getText());
            stat.setInt(3, codeProfM);
 
            stat.executeUpdate();
            stat.close();
 
            String codeQuery = "SELECT MAX(code_m) FROM mere";
            stat = cnx.prepareStatement(codeQuery);
            rslt = stat.executeQuery();
            rslt.next();
 
            stat.executeQuery();
            stat.close();
            return rslt.getInt(1);
 
        }catch(Exception ex){
            JOptionPane.showMessageDialog(null, ex);
            return 0;
        }
    }

**joel.drigo** · 16/08/2015, 21h29

Comme je te l'ai dit précedemment, la fermeture du PreparedStatement ferme aussi le ResultSet. Donc, comme tu fermes le PreparedStatement avant de relever la valeur qui t'intéresse dans le ResultSet, le ResultSet est fermé à ce moment.

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
public int insertPere() throws SQLException{        
   int codeProfP = getProfP();
 
   String query = "INSERT INTO pere (nom_pere , revenu_pere,code_prof) VALUES (?,?,?)";
   try(PreparerStatement stat = cnx.prepareStatement(query)) { // le try-with-resource dispense de faire le close() explicitement
       stat.setString(1, txtPere.getText());
       stat.setString(2, txtRevP.getText());
       stat.setInt(3, codeProfP);
 
       stat.executeUpdate();
    }
 
    String codeQuery = "SELECT MAX(code_p) FROM pere";
    try(PreparedStatement stat = cnx.preparedStatement(codeQuery)){ // le try-with-resource dispense de faire le close() explicitement
        ResultSet rslt = stat.executeQuery();
        rslt.next(); // attention au cas où cette méthode retourne false...
 
        // pourquoi répéter la requête ici ? stat.executeQuery();
        return rslt.getInt(1);
    }
    catch(SQLException ex){
            JOptionPane.showMessageDialog(null, ex);
            return 0;
    }
}

Sache, par ailleurs, que tu peux aussi récupérer le code_pere sans faire de requête SELECT (c'est à priori un identifiant autoincrémental), par ResultSet keys = stat.getGeneratedKeys(), dans le bloc de l'update, bien sûr.

**Hind4Dev** · 16/08/2015, 21h48

alors là je ne comprends pas!!

l'exception ne se génère plus mais il n'enregistre toujours pas dans ces deux tables, je me désespère

**joel.drigo** · 16/08/2015, 21h55

Code :

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

1
2
3
4
5
6
7
String query = "INSERT INTO fils_de (bac,code_p,code_m) VALUES (?,?,?)";
            stat = cnx.prepareStatement(query);
            stat.setString(1,txtBAC.getText());
            stat.setInt(2, codeP);
            stat.setInt(3, codeM);
 
            stat.executeQuery();

Au lieu de executeQuery(), met executeUpdate(). Etonnant que tu n'obtiennes pas d'exception sur ce executeQuery().

**Hind4Dev** · 16/08/2015, 22h13

j'ai changé à stat.executeUpdate(); mais toujours pas de résultat, voici les deux fonctions d'insertion, et il ne génère aucune exception tous parait normal

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
@SuppressWarnings("deprecation")
	public void insertEtudiant() throws Exception{
		try{
			int codeLieu = getLieu();
			int codeNat = getNat();
			String codeF = getFiliere();
			int codeMotif = getMotif();
 
			String query = "INSERT INTO etudiant (bac,nom,prenom,genre,adresse,n_phone,mail,n_ccp,decision,bloque,"
					+ "n_ins,annee__encours,annee_univ,annee_courante,reste,"
					+ "an_1,an_2,an_3,an_4,an_5,an_6,an_7,an_8,an_9,an_10,"
					+ "cycle,nbr_annee,code_f,date_nais,code,code_nat,Observation,date_bloc,code_motif) "
					+ "VALUES (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)";
 
 
			stat = cnx.prepareStatement(query);
			stat.setString(1, txtBAC.getText());
			stat.setString(2, txtNom.getText());
			stat.setString(3, txtPrenom.getText());
			stat.setString(4, genre);
			stat.setString(5, txtAdr.getText());
			stat.setString(6, txtPhone.getText());
			stat.setString(7, txtMail.getText());
			stat.setString(8, (String)txtCCP.getValue());
			stat.setString(9, txtDec.getText());
			stat.setBoolean(10, bloc);
			stat.setString(11, txtIns.getText());
			stat.setString(12, txtAn.getText());
			stat.setString(13, txtUniv.getText());
			stat.setInt(14,new java.sql.Date(System.currentTimeMillis()).getYear());
			stat.setInt(15, reste);
			//stat.setInt(6, rds);
			stat.setInt(16, 0);
			stat.setInt(17, 0);
			stat.setInt(18, 0);
			stat.setInt(19, 0);
			stat.setInt(20, 0);
			stat.setInt(21, 0);
			stat.setInt(22, 0);
			stat.setInt(23, 0);
			stat.setInt(24, 0);
			stat.setInt(25, 0);
			stat.setString(26,txtBAC.getText());
			stat.setString(27, codeF);
			stat.setString(28, cycle);
			stat.setString(29, ((JTextField) dateChooser.getDateEditor().getUiComponent()).getText());
			stat.setInt(30, codeLieu);
			stat.setInt(31, codeNat);
			//stat.setInt(14, codePhoto);
			stat.setString(32, txtObs.getText());
			stat.setDate(33,new java.sql.Date(System.currentTimeMillis()));
			stat.setInt(34, codeMotif);
 
			stat.executeUpdate();
 
 
		}catch(SQLException ex){
			JOptionPane.showMessageDialog(null,"Problème insertEtudiant");
		}
		finally{
				stat.close();
		}
	}

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
 
	public void insertFils() throws Exception{
		int codeP = insertPere();
		int codeM = insertMere();
 
		try{
			String query = "INSERT INTO fils_de (bac,code_p,code_m) VALUES (?,?,?)";
			stat = cnx.prepareStatement(query);
			stat.setString(1,txtBAC.getText());
			stat.setInt(2, codeP);
			stat.setInt(3, codeM);
 
			stat.executeUpdate();
 
		}catch(Exception ex){
			JOptionPane.showMessageDialog(null, "Problème insertFils");
		}
		finally{
			stat.close();
		}
	}

**joel.drigo** · 16/08/2015, 22h24

Tu as d'autres INSERT qui fonctionnent et pas ceux-là : sur la même instance de Connection ? Tu n'as fait aucune modification de l'autocommit ? Si tu affiches System.out.println("autocommit="+cnx.getAutoCommit());, tu obtiens bien "autocommit=true" ?
Par ailleurs, qu'est-ce qui te fais dire que les 2 insert ne fonctionnent pas ? Si tu as bien mis un affichage de l'exception dans tous les catch et que tu n'obtiens pas d'exception, il va falloir utiliser le mode debug, ou mettre des traces, pour voir exactement ce qui est exécuté.

**Hind4Dev** · 16/08/2015, 22h31

oui j'ai l'insertion de la mère et du père qui fonctionnent à merveille
mais la table étudiant et fils_de , ces deux là tjrs vide , je vérifie dans ma base de données sur xamp que je travaille, toujours vides juste ces deux là

**tchize_** · 16/08/2015, 22h52

Et tu est sur d'avoir appelé les méthodes? Si tu met n'importe quoi à la place de ta query, t'as bien un exception qui remonte?

**Hind4Dev** · 16/08/2015, 23h05

effectivement, je l'ai appelle dans l’événement du bouton de sauvegarde, si je remplace query j'ai une erreur sur le code

**joel.drigo** · 16/08/2015, 23h09

Si tu modifies ta méthode comme ça :

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
public void insertFils() throws Exception{
		int codeP = insertPere();
		int codeM = insertMere();
 
		try{
			String query = "INSERT INTO fils_de (bac,code_p,code_m) VALUES (?,?,?)";
			stat = cnx.prepareStatement(query);
			stat.setString(1,txtBAC.getText());
			stat.setInt(2, codeP);
			stat.setInt(3, codeM);
 
			int resultat = stat.executeUpdate();
                        System.out.println("INSERT INTO fils_de=" + resultat + " autocommit="+cnx.getAutoCommit());
 
		}catch(Exception ex){
			JOptionPane.showMessageDialog(null, "Problème insertFils");
		}
		finally{
			stat.close();
		}
	}

Tu vois quoi s'afficher dans la console ?

**Hind4Dev** · 16/08/2015, 23h15

bizar je n'ai rien ni dans la console ni en message

mais il n'a rien sauvegardé même dans les deux tables qui fonctionnaient avant

**joel.drigo** · 16/08/2015, 23h23

Dans ce cas, soit executeUpdate n'est pas exécuté, soit il y a une exception, mais le message devrait être affiché par le JOptionPane.showMessageDialog()...

Et en remplaçant par :

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
public void insertFils() throws Exception{
                        System.out.println("appel insertFils");
		int codeP = insertPere();
		int codeM = insertMere();
                        System.out.println("appel insertFils pere="+codeP + " mere="+codeM);
 
		try{
			String query = "INSERT INTO fils_de (bac,code_p,code_m) VALUES (?,?,?)";
			stat = cnx.prepareStatement(query);
			stat.setString(1,txtBAC.getText());
			stat.setInt(2, codeP);
			stat.setInt(3, codeM);
 
                        System.out.println("avant INSERT INTO fils_de");
			int resultat = stat.executeUpdate();
                        System.out.println("INSERT INTO fils_de=" + resultat + " autocommit="+cnx.getAutoCommit());
 
		}catch(Exception ex){
                        ex.printStackTrace();
			JOptionPane.showMessageDialog(null, "Problème insertFils");
		}
		finally{
			stat.close();
		}
	}

**Hind4Dev** · 16/08/2015, 23h30

alors maintenant ni exception ni message même les print ne s'affichent pas mais l'enregistrement sur les deux fonctions père et mère est effectué
je ne comprends pas , au moins les print devraient s'afficher non?!!!!

Operation not allowed after ResultSet closed

JDBC Java

Discussions similaires

Partager

Partager