erreur lorsque je valide des champs dans une base mysql

Bonjour,
j'ai une Bdd Bibliothèque, et je rentre 4 valeurs : nomLivre, auteur, année livre, prix livre. et un bouton enregistrer pour remplir mes champs dans la base MySql.
mon code :

Code:

---

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 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139

/*  * Click nbfs://nbhost/SystemFileSystem/Templates/Licenses/license-default.txt to change this license  * Click nbfs://nbhost/SystemFileSystem/Templates/GUIForms/JFrame.java to edit this template  */ package gestionbibliotheque;   import java.sql.Statement; //import java.sql.Connection; import java.sql.SQLException; //import java.sql.DriverManager; import javax.swing.JOptionPane;   public class FenetreAjoutLivre extends javax.swing.JFrame {   Statement stmt; Connexion maConnexion = new Connexion();   public FenetreAjoutLivre() { initComponents(); }   @SuppressWarnings("unchecked") // <editor-fold defaultstate="collapsed" desc="Generated Code"> private void initComponents() {   jLabel1 = new javax.swing.JLabel(); jLabel2 = new javax.swing.JLabel(); jLabel3 = new javax.swing.JLabel(); jLabel4 = new javax.swing.JLabel(); jLabel5 = new javax.swing.JLabel(); mNomLivre = new javax.swing.JTextField(); mAuteur = new javax.swing.JTextField(); mAnnee = new javax.swing.JTextField(); mPrix = new javax.swing.JTextField(); mBtnEnregistrer = new javax.swing.JButton();   setDefaultCloseOperation(javax.swing.WindowConstants.DISPOSE_ON_CLOSE);   jLabel1.setFont(new java.awt.Font("Segoe UI", 1, 14)); // NOI18N jLabel1.setText("Enregistrement des ouvrages");   jLabel2.setText("Nom du livre");   jLabel3.setText("Auteur");   jLabel4.setText("Année");   jLabel5.setText("Prix");   mBtnEnregistrer.setText("Enregistrer"); mBtnEnregistrer.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { mBtnEnregistrerActionPerformed(evt); } });   javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane()); getContentPane().setLayout(layout); layout.setHorizontalGroup( layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING) .addGroup(layout.createSequentialGroup() .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING) .addGroup(layout.createSequentialGroup() .addGap(77, 77, 77) .addComponent(jLabel1, javax.swing.GroupLayout.PREFERRED_SIZE, 228, javax.swing.GroupLayout.PREFERRED_SIZE)) .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.TRAILING) .addGroup(layout.createSequentialGroup() .addContainerGap() .addComponent(mBtnEnregistrer, javax.swing.GroupLayout.PREFERRED_SIZE, 99, javax.swing.GroupLayout.PREFERRED_SIZE)) .addGroup(javax.swing.GroupLayout.Alignment.LEADING, layout.createSequentialGroup() .addGap(34, 34, 34) .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false) .addGroup(javax.swing.GroupLayout.Alignment.TRAILING, layout.createSequentialGroup() .addComponent(jLabel4) .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE) .addComponent(mAnnee, javax.swing.GroupLayout.PREFERRED_SIZE, 87, javax.swing.GroupLayout.PREFERRED_SIZE)) .addGroup(layout.createSequentialGroup() .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING) .addComponent(jLabel3) .addComponent(jLabel2)) .addGap(55, 55, 55) .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING) .addComponent(mNomLivre, javax.swing.GroupLayout.PREFERRED_SIZE, 87, javax.swing.GroupLayout.PREFERRED_SIZE) .addComponent(mAuteur, javax.swing.GroupLayout.PREFERRED_SIZE, 87, javax.swing.GroupLayout.PREFERRED_SIZE))) .addGroup(layout.createSequentialGroup() .addComponent(jLabel5) .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE) .addComponent(mPrix, javax.swing.GroupLayout.PREFERRED_SIZE, 87, javax.swing.GroupLayout.PREFERRED_SIZE)))))) .addContainerGap(95, Short.MAX_VALUE)) ); layout.setVerticalGroup( layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING) .addGroup(layout.createSequentialGroup() .addGap(19, 19, 19) .addComponent(jLabel1) .addGap(34, 34, 34) .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE) .addComponent(jLabel2) .addComponent(mNomLivre, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)) .addGap(18, 18, 18) .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE) .addComponent(jLabel3) .addComponent(mAuteur, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)) .addGap(20, 20, 20) .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE) .addComponent(jLabel4) .addComponent(mAnnee, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)) .addGap(25, 25, 25) .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE) .addComponent(jLabel5) .addComponent(mPrix, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)) .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, 34, Short.MAX_VALUE) .addComponent(mBtnEnregistrer) .addGap(19, 19, 19)) );   pack(); }// </editor-fold>   private void mBtnEnregistrerActionPerformed(java.awt.event.ActionEvent evt) { String nomLivre = mNomLivre.getText(); String nomAuteur = mAuteur.getText(); String numAnnee = mAnnee.getText(); String numPrix = mPrix.getText();   String requete="INSERT INTO livre (nom_livre,nom_auteur,annee-edition,prix_livre) VALUES ('"+nomLivre+"','"+nomAuteur+"','"+numAnnee+"','"+numPrix+"')";   try { stmt = maConnexion.ObtenirConnexion().createStatement(); stmt.executeQuery(requete); JOptionPane.showMessageDialog(null,"requete executé avec succès "); } catch(SQLException ex){ System.err.println(ex);   }     }

---

lorsque je rentre les différentes valeurs dans les champs et je clique sur le bouton pour enregistrer cela met une erreur :

Code:

---

1 2 3 4

run: Chargement du pilote Reussie Chargement Connection Reussie java.sql.SQLException: Statement.executeQuery() cannot issue statements that do not produce result sets.

---

Bonjour,

Pour commencer, il faut décomposer le code selon les rôles.
Ça évitera d'avoir un code imbitable!

C'est le principe SOLID (S ici):
https://fr.wikipedia.org/wiki/SOLID_(informatique)

Deuxième point général: toujours fermer les "Autoclosable":
https://docs.oracle.com/javase/8/doc...Closeable.html

Tout ce qui est proche de la BDD l'est.
Pour cela, on utilise le try-with-ressource:

Code:

---

1 2 3 4

try(MonAutoClosable monAutoClosable = ...){ //Traitement }

---

Ainsi, avec cette façon de faire, on fermera automatiquement la ressource car on appellera à la fin la méthode close().

Troisième point: toujours utiliser des PreparedStatement
https://docs.oracle.com/javase/8/doc...Statement.html.

Si par exemple tu as:

Code:

---

1 2	String monSQL = "SELECT * FROM Personne WHERE id='"+id+"'";

---

Et que:

Code:

---

1 2	id="DELETE * FROM PERSONNE";

---

Tu fais de l'injection de SQL, et tu peux dire au revoir à ta BDD.

Sur le fond du problème, il faut utiliser executeUpdate.

Soit:

Code:

---

1 2 3 4 5 6 7 8 9

String requete="INSERT INTO livre (nom_livre,nom_auteur,annee-edition,prix_livre) VALUES(?,?,?,?)"; try(PreparedStatment ps = maConnexion.ObtenirConnexion().prepareStatement(requete){ ps.setString(1,nomLivre); ps.setString(2,nomAuteur); ps.setString(3,numAnnee); ps.setString(4,numPrix); ps.executeUpdate(); }

---

Notez que la connexion est aussi AutoClosable:
https://docs.oracle.com/javase/8/doc...onnection.html
Cordialement.