instruction de mise à jour en Java

J'ai le code suivant pour mettre à jour un enregistrement. Le code se compile cependant, il ignore tout de l'instruction try et affiche le message d'erreur dans l'instruction catch. Je ne suis pas sûr de ce que je suis absent, il n'affiche pas toute sorte d'erreur de syntaxe.

try {           
    PreparedStatement st = db.con.prepareStatement("UPDATE item SET Name = ?, Size = ?, Price = ?, WHERE ItemCode = ?");
    st.setString(1, textArea_Code.getText());
    st.setString(2, textArea_name.getText());
    st.setString(3, textArea_size.getText());
    st.setString(4, textArea_price.getText());
    st.executeUpdate();

    JOptionPane.showMessageDialog(frame, "Updated");    

} catch (SQLException e ) {
    JOptionPane.showMessageDialog(frame, "update not successful");      
}

OriginalL'auteur FatmaTurk | 2012-03-09

10

Vous êtes à avaler l'exception, qui est généralement une mauvaise idée. À tout le moins appel e.printStackTrace() de sorte que vous avez une sortie de l'exception.

Comme il arrive, vous avez une erreur de syntaxe dans votre instruction SQL: UPDATE item SET Name = ?, Size = ?, Price = ?, WHERE ItemCode = ? - supprimer la virgule après Price = ?.

À l'adresse de la confusion sur les raisons de l'instruction de mise à JOUR ne fonctionne toujours pas, en dépit de la fixation de l'erreur de syntaxe, permettez-moi de vous expliquer plus en détail (beaucoup plus facile pour ce faire, dans la réponse, plutôt que dans les commentaires).

La ? de caractère dans votre Chaîne SQL est un espace réservé pour une valeur que vous définissez à l'un des différents set_() méthodes (dans votre cas, ne jamais setString(). Chaque espace réservé est un numéro d'index à partir de 1 - la première ? qui apparaît dans votre chaîne de caractères représente l'indice 1, le deuxième représente l'indice 2, etc.

Votre chaîne SQL ressemble à ceci:
```
UPDATE item SET Name = ?, //1
Size = ?, //2
Price = ? //3
WHERE ItemCode = ? //4
```
Vous êtes à la définition des valeurs de vos espaces comme ceci:
```
st.setString(1, textArea_Code.getText()); //ItemCode is fourth in the SQL, should be 4
st.setString(2, textArea_name.getText()); //Name is first in the SQL, should be 1
st.setString(3, textArea_size.getText()); //Size is second in the SQL, should be 2
st.setString(4, textArea_price.getText()); //Price is third in the SQL, should be 3
```
c'est une bête erreur que j'ai faite merci de me faire réaliser 🙂 mais maintenant il ne veut afficher la boîte de message "mise à jour", mais les changements ne prennent effet dans la base de données... savez-vous pourquoi cela pourrait être?
Vous disposez de quatre espaces réservés dans votre chaîne SQL, numérotés de 1 à 4. 1 est la valeur de Name, 2 est la valeur de Size, 3 est la valeur de Price et 4 est la valeur de ItemCode. Maintenant, regardez les valeurs que vous êtes de passage dans votre st.setString() appels.
Vous pouvez obtenir le nombre réel de lignes mises à jour comme ceci int updated=st.executeUpdate(); Vérifier que le comte et voir si quelque chose est réellement mis à jour. Si elle renvoie >0, puis vérifier la base de données que vous utilisez, si la validation automatique est activée ou pas.
executeUpdate retourne un int. Vous devez vérifier que la valeur de déterminer si quelque chose a été mis à jour. Si vous voyez un zéro, je voudrais vérifier la base de données.
si le problème était avec la base de données puis je suppose qu'il ne serait pas me permettre de supprimer un enregistrement de la base de données ainsi droit?

OriginalL'auteur Anthony Grist
2

Avez-vous essayé de changer que l'instruction de mise à JOUR à quelque chose comme ceci:
```
"UPDATE item SET Name = ?, Size = ?, Price = ? WHERE ItemCode = ?"
```
J'ai pris la dernière , (virgule) après la Price = ? car il peut entrer en conflit avec le reste du SQL.

Edit:

Aussi, vous pouvez réorganiser les textArea_Code.getText() ligne à la dernière valeur de l'instruction préparée. Sinon, le ItemCode valeur ne peut être mise en correspondance avec l'ordre de la ? des espaces réservés (comme le moment, ça ressemble à textArea_price.getText() est de s'habituer, comme la valeur de ItemCode... qui pourrait être dangereux pour ce qui est mis à jour!)

Au lieu de cela, que penser de quelque chose comme ceci:
```
PreparedStatement st = db.con.prepareStatement("UPDATE item SET Name = ?, Size = ?, Price = ? WHERE ItemCode = ?");
st.setString(1, textArea_name.getText());
st.setString(2, textArea_size.getText());
st.setString(3, textArea_price.getText());
st.setString(4, textArea_Code.getText());
st.executeUpdate();
```
De cette façon, tous les ? espaces réservés correspondre correctement avec les valeurs définies:

Nom: st.setString(1, textArea_nom.getText());

Taille: st.setString(2, textArea_taille.getText());

Prix: st.setString(3, textArea_prix.getText());

ItemCode: st.setString(4, textArea_Code.getText());

C'est parce que les paramètres doivent être dans le même ordre que les espaces réservés (voir cette documentation pour plus d'informations.)

J'ai fait le changement pour mon instruction SQL, mais il n'est toujours pas de mise à jour de la base de données..
J'ai mis à jour cette réponse, vous pouvez essayer de réarrangement de l'ordre de votre espace réservé cordes. Voir la réponse pour plus de détails.
il travaille maintenant 🙂 maintenant, je peux voir comment l'ordre peut également faire une différence..merci beaucoup..
Pas de problème; et... bienvenue à Débordement de Pile! Comme vous posez des questions, n'oubliez pas de laisser savoir à la communauté si une réponse vous aide par upvoting d'une réponse, ou accepter la réponse! Merci~

OriginalL'auteur summea
0

Qu'est-ce que la sortie de executeUpdate? Est-il retourner n'importe quel nombre de lignes mises à jour? Si non, alors il est évident que votre clause de mise à jour doit être, bien, mis à jour!

Si le executeUpdate retourne un nombre positif, indiquant que la mise à jour nombre de lignes, peut-être votre commit politique n'est pas définie à validation automatique. Dans ce cas, vous pouvez s'engager après executeUpdate.
```
con.commit();
```
ses triés maintenant la question était à l'ordre du setString appels

OriginalL'auteur JUG
0
1. Pourrait vous essayez d'ajouter un e.printStackTrace() dans votre bloc catch ?
2. Dans votre requête suppression de la , avant de OÙ
  Mise à JOUR de l'élément Nom du JEU de = ?, Size = ?, Prix = ? OÙ ItemCode = ?
3. Peut-être il ya un problème avec votre base de données ... est votre élément de la table dans le schéma par défaut?... le printStackTrace() vous aideront à trouver la réponse
OriginalL'auteur Hugues
0

Après l'obtention de DB connexion automatique s'engager à true pour que DB sera mis à jour sans aucune erreur.
```
connection = DBConnection.getInstance().getConnection();
connection.setAutoCommit(true);
```
OriginalL'auteur user1907542

Vous devez vous connecter pour publier un commentaire.