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.

[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.

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

Pourquoi ?

[Bluedeep]

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.

[Pilru]

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

Couteuse ?

[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+

[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

[Pilru]

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

[Bluedeep]

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.

[hwoarang]

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

Testé avec SQL Server :

Code :

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

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

[Pilru]

Testé avec SQL Server :

Code :

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

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.

[Nathanael Marchand]

MERGE peut être pas mal aussi!