Discussion :

[Moutombi]

Salut à tous!
J'ai le code suivant. Lorsque j'insère une données dont la clé primaire n'existes pas dans la table, l'insertion se déroule bien avec même un message de confirmation. Mais lorsque je rentre une ligne dont la clé primaire exite déjà voici le message suivant:

Violation de la contrainte PRIMARY KEY 'PK_Reglmtcli'. Impossible d'insérer une clé en double dans l'objet 'dbo.Reglmtcli'.
L'instruction a été arrêtée.

Au fait, je voudrai qu'au moment de l'insertion il y ai d'abord une sorte de verification de l'existence de la clé puis ,s'il existe ,un message de confirmation qu'une ligne ayant une telle clé exite déjà.
Sinon, le code insere les donnees.
Voici le code en question:

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
50
51
52
 public string RempApos(string str)
        {
            return str.Replace("'", "''");
        }
 
 
 
        void BT_Send_Click(object sender, EventArgs e)
        {
 
            // Déclaration de l'objet de connexion
            System.Data.SqlClient.SqlConnection oConnection;
 
            // Création de l'objet de connexion
            oConnection = new System.Data.SqlClient.SqlConnection();
 
            // Paramétrage de la chaîne de connexion
            string sConnectionString;
            sConnectionString = "Data Source=.\\SQLEXPRESS;Initial Catalog=Database1.mdf;Integrated Security=True;User Instance=True;Context Connection=False;";
            sConnectionString = sConnectionString + " Trusted_Connection=True;";
            oConnection.ConnectionString = sConnectionString;
 
 
            // Connexion à la base de données
            oConnection.Open();
 
            // Déclaration de l'objet SqlCommand et de la requête SQL
            System.Data.SqlClient.SqlCommand oCommand;
            string sSQL;
 
            //Ecriture de la requête
 
            sSQL = "INSERT INTO [Reglmtcli](Numreg,Codecli,Nomcli,Objet,Datereg,Modepay,Montant,RestImpu,Encourcli,Codebanq,Codegui,Numcmpte,CleRib,Domicili,Titulaire,Numcheq,Datecheq,Observation)";
            sSQL = sSQL + " VALUES('" + ZeNumreg.Text + "','" + ZeCodecli.Text + "','" + RempApos(ZeNomcli.Text) + "','" + RempApos(ZeObjet.Text) + "','" + ZeDatereg.Text + "','" + ZeModepay.Text + "','" + ZeMontant.Text + "','" + ZeRestImpu.Text + "','" + ZeEncourcli.Text + "','" + ZeCodebanq.Text + "','" + ZeCodegui.Text + "','" + ZeNumcmpte.Text + "','" + ZeCleRib.Text + "','" + ZeDomicili.Text + "','" + RempApos(ZeTitulaire.Text) + "','" + ZeNumcheq.Text + "','" + ZeDatecheq.Text + "','" + RempApos(ZeObservation.Text) + "')";
 
 
            // Création de l'objet SqlCommand
            oCommand = new System.Data.SqlClient.SqlCommand(sSQL, oConnection);
 
            // On appelle la méthode ExecuteNonQuery pour éxécuter notre commande
            oCommand.ExecuteNonQuery();
 
            // On écrit notre message de confirmation
            LB_Message.Text = "Le reglement n° " + ZeNumreg.Text + " a été enregistré !";
 
            // On rend le message visible
            LB_Message.Visible = true;
 
            // Fermeture de la connexion
            oConnection.Close();
 
        }

Merci d'avance !

[PatteDePoule]

Tu peux simplement mettre un Try Catch, et dépendant de l'erreur retournée tu affiches ton message.

Sinon tu fais un bête SELECT pour voir si ta clé existe. Mais ça implique de faire 2 requêtes à chaque fois que tu veux insérer.

[Moutombi]

Au fait, j'ai essayé ce que tu dis là. Mais le pb c'est que ça ne marche pas! Au fait, je voudrai avoir un petit exemple pour avoir une idée.

EDIT
Voici en effet le code que j'ai écrit, mais ça marche 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
29
30
31
32
33
34
 SqlCommand oCommand = new SqlCommand("SELECT * FROM  Accès  WHERE Nom ='" + ZeNomUtilisateur + "'", oConnexion);
            try
            {
                // Ouverture de la connexion et exécution de la requête
                oConnexion.Open();
 
                //Exécution de requête SQL
 
                SqlDataReader drUtilisateur = oCommand.ExecuteReader();
 
                // Parcours de la liste des utilisateurs
 
                while (drUtilisateur.Read())
                {
                    if (drUtilisateur["Nom"].ToString() == ZeNom) 
                    {
                        lb_Message.Text="Cet enregistrement existe déjà"
                    }
           else
           {
 
            // On appelle la méthode ExecuteNonQuery pour éxécuter notre commande
            oCommand.ExecuteNonQuery();
 
            // On écrit notre message de confirmation
            LB_Message.Text = "Le reglement n° " + ZeNumreg.Text + " a été enregistré !";
 
            // On rend le message visible
            LB_Message.Visible = true;
 
            // Fermeture de la connexion
            oConnection.Close();
 
        }

Merci d'avance!

[calagan99]

Moutombi, tu essayes d'insérer un enregistrement avec une requête SELECT ?

Comme nous te le disions hier, la meilleure chose à faire et de se poser une journée pour lire et comprendre des tutoriaux sur le développement en .Net.
Là, il te manque les bases et tu buteras sans cesse sur des questions simples tant que tu n'auras pas fait l'effort de te documenter un minimum.

Qui plus est, utilise des requêtes paramétrées plutôt que des requêtes avec concaténation.
Tes requêtes seront sécurisées et plus lisibles.

[Moutombi]

Au fait, j'ai utilisé try catch comme a suggéré PatteDePoule, et voici ce que ça donne. Mais apparament le code ne fais rien.
Voici le bout de 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
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
 
        public string RempApos(string str)
        {
            return str.Replace("'", "''");
        }
 
 
 
        void BT_Send_Click(object sender, EventArgs e)
        {
 
            // Déclaration de l'objet de connexion
            System.Data.SqlClient.SqlConnection oConnection;
 
            // Création de l'objet de connexion
            oConnection = new System.Data.SqlClient.SqlConnection();
 
            // Paramétrage de la chaîne de connexion
            string sConnectionString;
            sConnectionString = "Data Source=.\\SQLEXPRESS;Initial Catalog=Database1.mdf;Integrated Security=True;User Instance=True;Context Connection=False;";
            sConnectionString = sConnectionString + " Trusted_Connection=True;";
            oConnection.ConnectionString = sConnectionString;
 
 
 
 
            // Déclaration de l'objet SqlCommand et de la requête SQL
            System.Data.SqlClient.SqlCommand oCommand;
            string sSQL;
 
            //Ecriture de la requête
 
            sSQL = "INSERT INTO [Reglmtcli](Numreg,Codecli,Nomcli,Objet,Datereg,Modepay,Montant,RestImpu,Encourcli,Codebanq,Codegui,Numcmpte,CleRib,Domicili,Titulaire,Numcheq,Datecheq,Observation)";
            sSQL = sSQL + " VALUES('" + ZeNumreg.Text + "','" + ZeCodecli.Text + "','" + RempApos(ZeNomcli.Text) + "','" + RempApos(ZeObjet.Text) + "','" + ZeDatereg.Text + "','" + ZeModepay.Text + "','" + ZeMontant.Text + "','" + ZeRestImpu.Text + "','" + ZeEncourcli.Text + "','" + ZeCodebanq.Text + "','" + ZeCodegui.Text + "','" + ZeNumcmpte.Text + "','" + ZeCleRib.Text + "','" + ZeDomicili.Text + "','" + RempApos(ZeTitulaire.Text) + "','" + ZeNumcheq.Text + "','" + ZeDatecheq.Text + "','" + RempApos(ZeObservation.Text) + "')";
 
 
            // Création de l'objet SqlCommand
            oCommand = new System.Data.SqlClient.SqlCommand(sSQL, oConnection);
 
            //
 
            SqlCommand oCmd = new SqlCommand("SELECT * FROM  Reglmtcli  WHERE Nom ='" + ZeNumreg + "'", oConnection);
 
            try
            {
 
                // Connexion à la base de données
                oConnection.Open();
 
                //Exécution de requête SQL
 
                SqlDataReader drReglmtcli = oCmd.ExecuteReader();
                while (drReglmtcli.Read())
                {
                    if (drReglmtcli.HasRows)
                    {
                        Console.Write("Enregistrement existant");
                    }
                    else
                    {
                        // On appelle la méthode ExecuteNonQuery pour éxécuter notre commande
                        oCommand.ExecuteNonQuery();
 
                        // On écrit notre message de confirmation
                        LB_Message.Text = "Le reglement n° " + ZeNumreg.Text + " a été enregistré !";
 
 
                        // On rend le message visible
                        LB_Message.Visible = true;
                    }
                }
            }
            catch
            {
 
 
                // Fermeture de la connexion
                oConnection.Close();
 
            }
        }

[StringBuilder]

Bonjour,

Tu mélanges tout !

Déjà, on dit pas "au fait", mais "en fait" dans le contexte que tu utilises

Sinon, tu as deux solutions :

Solution 1 : Recommandée. Tu ne vérifies rien avant l'insertion dans la base, et tu gères l'erreur s'il y en a une. C'est ce qu'il y a de plus rapide et de plus fiable.

Solution 2 : Déconseillée. Tu recherches la présence de la ligne dans la base, et s'il n'y a rien, tu insères les données. Deux fois plus lentes (deux accès à la base) et non fiable (une autre personne peut insérer la ligne entre le moment où tu vérifies qu'elle n'existe pas, et le moment où tu l'insères).


Pour mettre en place la solution 1 :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
cmd.CommandText = "..."; // Par pitié, utilise une requête paramétrée, et pas ton capharnaüm géant !
 
try
{
   cmd.ExecuteNonQuery();
   Console.Write("Youpi tralala");
}
catch
{
   Console.Write("Boum !");
}

Et c'est tout !