ExecuteNonQuery() & Violation of PRIMARY

Version imprimable

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.

Citation:

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

Code:

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 :ccool:

04/05/2011, 10h31
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
04/05/2011, 10h43
Bluedeep

Citation:

Envoyé par 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.

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

Citation:

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 :D
04/05/2011, 10h44
jbrasselet

Citation:

Envoyé par Bluedeep

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 :ccool:
04/05/2011, 12h24
Pilru

Inclure ton code dans un Try...Catch évitera au framework .NET de lancer automatiquement l'exception.
04/05/2011, 13h06
Bluedeep

Citation:

Envoyé par Pilru

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.
04/05/2011, 13h34
Pilru

Si L'objectif est d'éviter un plantage de l'appli sur un dup key, le try catch est une solution valable (pas la meilleure). Et permet le cas échéant de remonter un message "civilisé" à l'utilisateur.

Citation:

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

Pourquoi ?
04/05/2011, 13h53
Bluedeep

Citation:

Envoyé par Pilru

Si L'objectif est d'éviter un plantage de l'appli sur un dup key, le try catch est une solution valable (pas la meilleure). Et permet le cas échéant de remonter un message "civilisé" à l'utilisateur.

Pourquoi ?

L'exception n'est pas une situation "normale", sa gestion est couteuse.
Ici de plus on fait remonter au programme client une situation qui doit être filtrée au niveau de l'action demandée au SGBD.
04/05/2011, 14h05
Pilru

Citation:

Envoyé par Bluedeep

L'exception n'est pas une situation "normale", sa gestion est couteuse.

Couteuse ?
04/05/2011, 14h21
hwoarang

Si l'objectif est de ne pas ajouter l'enregistrement lorsque la clé existe, tu peux ajouter un "WHERE NOT EXISTS (SELECT * FROM MATABLE WHERE ID = ID_QUE_J_INSERE)"

a+
04/05/2011, 14h35
jbrasselet

Je pense que l'utilisation du try/catch dépend vraiment de ce que l'on veut faire exactement.
Si on veut un message d'erreur pour repérer les lignes en erreur ça peut être très pratique. Si on s'en fout, autant faire une proc stock
04/05/2011, 15h02
Pilru

Citation:

Envoyé par hwoarang

Si l'objectif est de ne pas ajouter l'enregistrement lorsque la clé existe, tu peux ajouter un "WHERE NOT EXISTS (SELECT * FROM MATABLE WHERE ID = ID_QUE_J_INSERE)"

a+

Sauf erreur de ma part, un ordre INSERT ne prend pas de clause WHERE
04/05/2011, 15h16
Bluedeep

Citation:

Envoyé par Pilru

Sauf erreur de ma part, un ordre INSERT ne prend pas de clause WHERE

Non, mais on peut très bien scripter :

IF NOT EXISTS(....)
INSERT .....

Ce qui est nettement plus logique que de retourner une erreur vers le client.

Citation:

Envoyé par Pilru

Sauf erreur de ma part, un ordre INSERT ne prend pas de clause WHERE

Testé avec SQL Server :
Code:

1 2 3 4 DECLARE @t TABLE(id int, lab varchar(50)); INSERT INTO @t (id, lab) (SELECT 1, 'test1' WHERE NOT EXISTS (SELECT * FROM @t WHERE id = 1)); INSERT INTO @t (id, lab) (SELECT 1, 'test2' WHERE NOT EXISTS (SELECT * FROM @t WHERE id = 1)); SELECT * FROM @t

Citation:

Envoyé par hwoarang

Testé avec SQL Server :

Code:

1
2
3
4
DECLARE @t TABLE(id int, lab varchar(50));
INSERT INTO @t (id, lab) (SELECT 1, 'test1' WHERE NOT EXISTS (SELECT * FROM @t WHERE id = 1));
INSERT INTO @t (id, lab) (SELECT 1, 'test2' WHERE NOT EXISTS (SELECT * FROM @t WHERE id = 1));
SELECT * FROM @t

OK, ce n'est donc pas un ordre insert avec une clause where.

04/05/2011, 17h49
Nathanael Marchand

MERGE peut être pas mal aussi!