Discussion :

[Oussama_Gabes]

Salut,

Je voulais faire la suppression d'enregistrement de la base de donnée depuis une interface java ... l'ajout s'effectue normalement mais la suppression ne marche pas...

Voilà le code de la requête:

Code :

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

String requete="DELETE FROM Utilisateur where (Code='"+cntCode+"',Nom='"+cntNom+"',Prenom='"+cntPrenom+"');

Voilà le code de l'ajout qui marche:

Code :

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

String requete="insert into Utilisateur values('"+cntCode+"','"+cntNom+"','"+cntPrenom+"','"+cntLogin+"','"+cntPassword+"','"+cntDroit+"')";

Alors comment je fais la suppression? ou est l'erreur???

(La base est en Access)

Merci pour votre passage

[CyberChouan]

Tu devrais aller lire la de la rubrique SQL.

Ta syntaxe pour le DELETE est mauvaise. Les différents éléments de la clause "WHERE" sont séparées par des opérateurs logiques (AND, OR, etc.) et non par des virgules.

Code :

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

String requete="DELETE FROM Utilisateur where Code='"+cntCode+"' AND Nom='"+cntNom+"' AND Prenom='"+cntPrenom+"'";

En plus, en Java on utilise plus volontier la classe PreparedStatement pour laisser au driver du SGBD le soin de gérer ses particularités (caractères d'échappement, etc.) : ça permet par exemple d'avoir des chaînes de caractère contenant ' sans problème.

C'est expliqué dans la Java.

[Oussama_Gabes]

Merciiiiiiiii

La syntaxe est maintenant correcte... le programme compile et 0 erreurs.

Mais la suppression ne se fait pas je ne comprend pas pourquoi et il n'affiche pas le message d'erreur que j'ai spécifier.

Voilà le code si vous pouvez encore m'aidez...

Merci encore

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
if(B==VSupprim)
		{
 
			if((SCode.getText().equals(""))||(SNom.getText().equals(""))||(SPrenom.getText().equals("")))
				{//Message d'erreur.
					JOptionPane.showMessageDialog(null, "Tous les champs doivent etre remplis ","Erreur",JOptionPane.ERROR_MESSAGE);
				}
 
			else
			{
 
			//Extraction des données des champs
			String cCode=SCode.getText();
			String cNom=SNom.getText();
			String cPrenom=SPrenom.getText();
 
 
			String requete="DELETE FROM Utilisateur where Code='"+cCode+"' AND Nom='"+cNom+"' AND Prenom='"+cPrenom+"'";
 
			try
			{//conection à la base et éxecution de la requete
				Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
 
  				Connection con = DriverManager.getConnection("jdbc:odbc:Base");
 				Statement stmt = con.createStatement();
 
				  int nb = stmt.executeUpdate(requete);
				  if (nb==0)
  					{//Message d'erreur.
  						JOptionPane.showMessageDialog(null, "Erreur lors de la suppression ","Message",JOptionPane.WARNING_MESSAGE);
 
 
  					}
 
  				else
 
				  	{
  						JOptionPane.showMessageDialog(null, "Utilisateur supprimé avec succès ","Message",JOptionPane.INFORMATION_MESSAGE);
  					}					
					 stmt.close();
					 con.close();
  			}
 
  			catch(Exception eb){
				}
 
			}
 
		}

Code :

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

VSupprim.addActionListener(this);

[tchize_]

tu ne fais pas de commit sur ta transaction.

[Oussama_Gabes]

Desolé, je ne comprends pas... j'arrive pas à trouver l'erreur... peut etre dans la syntaxe???

quand je remplace la ligne de la suppression par une requete d'ajout elle tourne normallement mais pas la suppression

[tchize_]

je viens de relire ton code. Supprime cette horreur!! et affche l'erreur en question!!:

Code :

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

1
2
  			catch(Exception eb){
				}

[OButterlin]

Tu devrais également t'assurer que la connexion sera libérée dans tous les cas (erreur par exemple).
Il serait préférable de coder dans ce genre

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
 
Connection connection = null;
try
{
   connection = DriverManager.getConnection(...);
   ...
}
catch (Exception e)
{
   ...
}
finally
{
   if (connection != null) connection.close();
}

[cashmoney]

Bonjour,

tchize_ et OButterlin ont raison, tu mets un "try "et un catch, mais tu "try" quoi au juste??

Si tu ne sais ce que tu "try", tu ne vas jamais ciblé ton erreur, donc tu seras toujours en interrogation??