java JDBC ArrayIndexOutOfBoundsException lors de la suppression, mettre à jour

Question

Donc, j'essaie de créer un JTable qui se connecte à la base de données du serveur SQL par JBDC, et avoir une fonction comme insérer, supprimer et éditer les données. Cela fonctionne bien avec l'insertion mais la mise à jour, la suppression. Pouvez-vous me montrer pourquoi j'ai obtenu ArrayIndexOutOfBoundsException: -1 et comment y remédier. Voici mon code. Livre est une classe étend ici Jframe

public Book() { 
    initComponents(); 

    model.addColumn("ID"); 
    model.addColumn("Name"); 
    model.addColumn("Type"); 

    jTable1.setModel(model); 
    displayTable(); 
    jTable1.getSelectionModel().addListSelectionListener(new ListSelectionListener() { 
     @Override 
     public void valueChanged(ListSelectionEvent e) { 
      int row = jTable1.getSelectedRow(); 

      txtName.setText(jTable1.getValueAt(row, 1).toString()); 
      txtType.setText(jTable1.getValueAt(row, 2).toString()); 

     } 
    }); 

} 

public void displayTable() { 
    try { 
     model.setRowCount(0); 
     ConnectToSQL sql = new ConnectToSQL(); 
     connection = sql.getConnection(); 
     st = connection.createStatement(); 
     ResultSet result = st.executeQuery("SELECT * FROM BOOK"); 
     while (result.next()) { 
      model.addRow(new Object[]{result.getInt("id"), result.getString("name"), result.getString("type")}); 
     } 

    } catch (SQLException ex) { 
     Logger.getLogger(Book.class.getName()).log(Level.SEVERE, null, ex); 
    } 
} 

private void btnUpdateActionPerformed(java.awt.event.ActionEvent evt) { 
    // TODO add your handling code here: 
    int row = jTable1.getSelectedRow(); 
    String id = jTable1.getValueAt(row, 0).toString(); 
    try { 
     // TODO add your handling code here: 
     st.executeUpdate("Update Book set name ='" + txtName.getText() + "',type='" + txtType.getText() + "' where id =" + id); 
     displayTable(); 

    } catch (SQLException ex) { 
     Logger.getLogger(Book.class.getName()).log(Level.SEVERE, null, ex); 
    } 

} 

private void btnDeleteActionPerformed(java.awt.event.ActionEvent evt) { 
    // TODO add your handling code here: 
    int row = jTable1.getSelectedRow(); 
    String id = jTable1.getValueAt(row, 0).toString(); 
    try { 
     // TODO add your handling code here: 
     st.executeUpdate("Delete from book where id =" + id); 
     displayTable(); 

    } catch (SQLException ex) { 
     Logger.getLogger(Book.class.getName()).log(Level.SEVERE, null, ex); 
    } 

} 

Answer 1

Vous obtenez cette erreur, parce que vous ne sélectionnez aucune ligne, afin d'éviter ce problème, vous devez utiliser:

if(jTable1.getSelectedRow() != -1){ 
    int row = jTable1.getSelectedRow(); 
    String id = jTable1.getValueAt(row, 0).toString(); 
    //rest of your code here 
}else{ 
    //show an error for example, no row is selected 
} 

---

Remarque

Au lieu de:

st.executeUpdate("Update Book set name ='" + txtName.getText() + "',type='" + txtType.getText() + "' where id =" + id); 

Vous devez utiliser PreparedStatement pour éviter toute erreur de syntaxe ou SQL Injection

Par exemple:

String query = "Update Book set name = ?, type=? where id =?"; 

try (PreparedStatement update = connection.prepareStatement(query)) { 

    update.setString(1, txtName.getText()); 
    update.setString(2, txtType.getText()); 
    update.setInt(3, id); 

    update.executeUpdate(); 
} 

Une autre chose, votre id est un int de sorte que vous ne pouvez pas définir une chaîne à votre requête comme vous avez-vous de définir un int:

String id = jTable1.getValueAt(row, 0).toString(); 

au lieu de cela, vous devez utiliser:

int id = Integer.parseInt(jTable1.getValueAt(row, 0).toString());