soucis requete de suppression winforms

**doumbix** · 13/11/2012, 16h53

Bonjour , depuis hier je debug avec les points d’arrêt de VS mais je vois pas le soucis avec la suppression mais il me dit "suppression effectuée" comme ce que j'ai écris dans mon Messagebox mais il n'y a pas de suppression ni dans la listbox ni dans ma base . voici le code

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
private void suppression_Click(object sender, EventArgs e)
        {
            cx.Open();
            cmd.CommandText= "delete from contact where nom='"+nom1.Text+"'and prenom='"+prenom1.Text+"'and adresse='"+adresse1.Text+"'and Telephone='"+telephone1.Text+"'and ville='"+ville1.Text+"'and sexe='"+sexe1.Text+"'";
            cmd.ExecuteNonQuery();
            cx.Close();
            MessageBox.Show("suppression effectuée ...");
 
            listContact();
            nom1.Text = "";
            prenom1.Text = "";
            adresse1.Text = "";
            telephone1.Text = "";
            ville1.Text = "";
            sexe1.Text = "";
 
        }

**sisqo60** · 13/11/2012, 20h36

Bonjour,

Je vais éviter de remettre en cause le code bien que je sois persuadé au vu de ce que je vois qu'il doit y avoir pas mal d'autres problèmes, mais avant tout, ExecuteNonQuery doit renvoyer un entier qui représente le nombre de lignes affectées, récupères cette valeur et on pourra continuer à investiguer et éventuellement te poster un code plus "joli"

**doumbix** · 13/11/2012, 22h32

Effectivement le resultat est 0 , je l'ai récupéré dans un label comme ceci

Code :

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

1
2
3
4
5
 
cx.Open();
  cmd.CommandText= "delete from contact where nom='"+nom1.Text+"'and prenom='"+prenom1.Text+"'and adresse='"+adresse1.Text+"'and Telephone='"+telephone1.Text+"'and ville='"+ville1.Text+"'and sexe='"+sexe1.Text+"'";
            int recup =  cmd.ExecuteNonQuery();
            result.Text =recup.ToString();

(je suis débutant , si vous pouvez me donner les liens utiles aussi , j'en serais content)

**sisqo60** · 13/11/2012, 22h53

Débutant ne te dispense pas de chercher, il existe une fonction rechercher dans le forum et aussi sur Google. Bref, une recherche de requêtes sql paramétrées + c# →solution...

Ps : copie la requête dans ton analyseur de requête lié à ton sgbd et compare avec tes données

**lead8209** · 14/11/2012, 11h29

Au delà des requêtes paramétrées qui sont cela dit très utiles; ne peux tu pas accéder à ton contact par un id plutôt que de passer tous les champs et de prier pour que le serveur SQL reconnaisse un contact avec toutes ces infos

?
Ça serait plus pertinent...

Moi je prends les paris sur un champ qui comporte une simple cote -> ' et qui met le bazar.

**doumbix** · 14/11/2012, 12h56

Bonjour , Je ne fais que chercher Pourtant ... Pas de soucis , je vais continuer a chercher , Merci pour ses éclaircissements .

**doumbix** · 14/11/2012, 13h41

Mes efforts sont enfin récompensés !!! Merci , je commence a découvrir .
ca marche avec une bonne recherche

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
private void suppression_Click(object sender, EventArgs e)
        {
 
 
            string maRequete = "delete from contact where nom=@nom";
            cmd = new SqlCommand(maRequete, cx);
            cmd.Parameters.Add(new SqlParameter("@nom", SqlDbType.VarChar, 25));
            cmd.Parameters["@nom"].Value = nom1.Text;
            try
            {
                cmd.Connection.Open();
               int recup=cmd.ExecuteNonQuery();
                cmd.Connection.Close();
                result.Text = recup.ToString();
                MessageBox.Show("suppression effectuée ...");
                listContact();
                nom1.Text = "";
                prenom1.Text = "";
                adresse1.Text = "";
                telephone1.Text = "";
                ville1.Text = "";
                sexe1.Text = "";
 
            }
            catch (SqlException ex)
            {
                System.Console.Write(ex.Message);
            }

**lead8209** · 14/11/2012, 15h01

Attention à ne pas supprimer plusieurs contacts qui portent le même nom avec cette requête.
Je ne crois pas me tromper en disant qu'il est conseillé d'utiliser les id dans ton cas.
Qqe chose du genre "delete from contact where id = @id" où @id = 23 ?
La tu es sûr qu'on parle bien de CE contact et non pas de Mr toto michel ou de Mr titi michel qui portent le même nom.

**doumbix** · 14/11/2012, 15h22

Je sais , c’était juste pour tester avec le nom

. Après je vais juste le remplacer par id

**doumbix** · 15/11/2012, 12h48

re Bonjour , un autre petit soucis . il me dit que la connexion n'est pas fermée or je pense l'avoir bien fermée . Merci pour tout éclairage .

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
try
            {
                cmd.Connection.Open();
                 cmd.ExecuteNonQuery();
                //result.Text = recup.ToString();
                cmd.Connection.Close();                
                MessageBox.Show("modification effectuée ...");
                cmd.Connection.Close();                
                listContact();
                nom1.Text = "";
                prenom1.Text = "";
                adresse1.Text = "";
                telephone1.Text = "";
                ville1.Text = "";
                sexe1.Text = "";
            }
            catch (SqlException ex)
            {
                System.Console.Write(ex.Message);
            }

**sisqo60** · 15/11/2012, 15h07

Sauf si tu as eu une exception, le close ne se fait pas...

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
string maRequete = "delete from contact where nom=@nom";
            try
            {
                cmd = new SqlCommand(maRequete, cx);
                cmd.Parameters.Add(new SqlParameter("@nom", SqlDbType.VarChar, 25));
                cmd.Parameters["@nom"].Value = nom1.Text;
                cmd.Connection.Open();
                int recup=cmd.ExecuteNonQuery();
                result.Text = recup.ToString();
                MessageBox.Show("suppression effectuée ...");
                listContact();
                nom1.Text = "";
                prenom1.Text = "";
                adresse1.Text = "";
                telephone1.Text = "";
                ville1.Text = "";
                sexe1.Text = "";
 
            }
            catch (SqlException ex)
            {
                System.Console.Write(ex.Message);
            }
            finally
            {
                cmd.Dispose();
                cx.Close();
            }

**doumbix** · 15/11/2012, 16h32

cela ne se ferme toujours pas et malgré que je me suis inspiré de votre code puis remis mon ancien ... il me dit toujours que ce n'est pas fermée

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
private void modification_Click(object sender, EventArgs e)
        {
            string maRequete = "update from contact set nom=@nom,prenom=@prenom,adresse=@adresse,Telephone=@telephone,ville=@ville,sexe=@sexe ";
            cmd = new SqlCommand(maRequete, cx);
            cmd.Parameters.Add(new SqlParameter("@nom", SqlDbType.VarChar, 25));
            cmd.Parameters.Add(new SqlParameter("@prenom", SqlDbType.VarChar, 25));
            cmd.Parameters.Add(new SqlParameter("@adresse", SqlDbType.VarChar, 25));
            cmd.Parameters.Add(new SqlParameter("@telephone", SqlDbType.VarChar, 25));
            cmd.Parameters.Add(new SqlParameter("@ville", SqlDbType.VarChar, 25));
            cmd.Parameters.Add(new SqlParameter("@sexe", SqlDbType.VarChar, 25));
 
            cmd.Parameters["@nom"].Value = nom1.Text;
            cmd.Parameters["@prenom"].Value = prenom1.Text;
            cmd.Parameters["@adresse"].Value = adresse1.Text;
            cmd.Parameters["@telephone"].Value = telephone1.Text;
            cmd.Parameters["@ville"].Value = ville1.Text;
            cmd.Parameters["@sexe"].Value = sexe1.Text;
 
            try
            {
                cmd.Connection.Open();
                 cmd.ExecuteNonQuery();
                //result.Text = recup.ToString();
                cmd.Connection.Close();
                cmd.Dispose();
                cx.Close();
 
                MessageBox.Show("modification effectuée ...");
                cmd.Connection.Close();                
                listContact();
                nom1.Text = "";
                prenom1.Text = "";
                adresse1.Text = "";
                telephone1.Text = "";
                ville1.Text = "";
                sexe1.Text = "";
            }
            catch (SqlException ex)
            {
                System.Console.Write(ex.Message);
            }
        }
    }
}

**Bluedeep** · 15/11/2012, 17h14

La connexion est fermée au moins deux fois.

Une fois par l'appel à la méthode Close de l'object Connection sous-jacent à la commande :
cmd.Connection.Close();et une autre fois par appel à la méthode Close sur la variable contenant la connexion elle même :
cx.Close();(car cx, je suppose qu'il s'agit de la connexion .... ).

**doumbix** · 15/11/2012, 18h33

Pourtant c'est ce que j'ai fait je l'ai au moins fermé 2 fois , ce qui m’étonne , l'insertion , la suppression marchent nickel sauf la mise a jour

**doumbix** · 17/11/2012, 00h39

Merci de toute façon pour tout , J'ai résolu 90% des soucis .
A bientôt

soucis requete de suppression winforms [Débutant]

Windows Forms

Discussions similaires

Partager

Partager