Discussion :

[Hind4Dev]

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



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]

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]

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]

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]

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]

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