Discussion :

[makin_toch]

Bonjour,
J'utilise un code c# pour importer des données du fichier texte vers ma base de donnée.

le code fonctionne mais j'ai un problème lors la duplication d'un clé primaire.
Je cherche comment ajouter un test,Si n'existe pas ajouter SInon n'ajouter pas.

voici mon code et la ligne en rouge qui génère l'erreur.

[SqlException (0x80131904): Violation of PRIMARY KEY constraint 'PK_airnode'. Cannot insert duplicate key in object 'dbo.airnode'

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
StringBuilder insertCmd = new StringBuilder();

        string line;
        string[] myData;
        string[] delimiter = { "," };

        // Read the file and display it line by line.
        System.IO.StreamReader file = new System.IO.StreamReader("c:\\air.stat");
        while ((line = file.ReadLine()) != null)
        {
            myData = line.Split(delimiter, StringSplitOptions.None);

            insertCmd.Append("INSERT INTO airnode VALUES(");
            insertCmd.Append("'" + myData[0] + "', ");
            insertCmd.Append(myData[1] + ", ");
            insertCmd.Append(myData[2] + ", ");
            insertCmd.Append(myData[3] + ", ");
            insertCmd.Append(myData[4] + ", ");
            insertCmd.Append(myData[5] + ", ");
            insertCmd.Append(myData[6] + ", ");
            insertCmd.Append(myData[7] + ", ");
            insertCmd.Append(myData[8] + ") ");
        }

        file.Close();

        SqlConnection con = new SqlConnection(@"Data Source=.\SQLEXPRESS;AttachDbFilename=|DataDirectory|\SecurityTutorials.mdf;Integrated Security=True;User Instance=True");
        SqlCommand cmd;
        con.Open();
        cmd = new SqlCommand(insertCmd.ToString(), con);
        cmd.CommandType = CommandType.Text;
        int affectedRows = cmd.ExecuteNonQuery();
        con.Close();

Merci de m'aider

[jbrasselet]

Tu peux faire d'abord un select count(*) pour savoir si ta donnée existe. Suivant le résultat tu fais un update ou un insert.

NB : utilises plutôt une requête paramétrée afin de sécuriser ton accès à la base de données

[Bluedeep]

Tu peux faire d'abord un select count(*) pour savoir si ta donnée existe. Suivant le résultat tu fais un update ou un insert.

Et encapsuler le tout dans une PS car il serait un peu bizarre de faire un A/R Client-Serveur pour cette opération.

NB : utilises plutôt une requête paramétrée afin de sécuriser ton accès à la base de données

En plus, oui; il faudrait mettre cette phase en automatique

[jbrasselet]

Et encapsuler le tout dans une PS car il serait un peu bizarre de faire un A/R Client-Serveur pour cette opération.

Oui en effet, ce serait encore l'étape suivante

[Pilru]

Inclure ton code dans un Try...Catch évitera au framework .NET de lancer automatiquement l'exception.

[Bluedeep]

Inclure ton code dans un Try...Catch évitera au framework .NET de lancer automatiquement l'exception.

Intercepter une exception sur clef dupliquée ne saurait constituer une méthode recommandable de design d'application.

Pour être plus clair, c'est complétement pourrave de procéder ainsi.

Si au taf je devais trouver ça dans un dev lors d'une revue de code, l'auteur se ferait sérieusement souffler dans les bronches.