Le même enregistrement est inséré plusieurs fois

Question

Chaque fois que j'insère un enregistrement dans ma base de données, l'enregistrement est ajouté trois fois.

try 
{ 
    con.Open(); 

    object addedkey = cmd.ExecuteScalar(); 

    if ((addedkey != null && (addedkey != DBNull.Value))) 
    { 
     addedkey = Convert.ToInt32(cmd.ExecuteScalar()); 
    } 

    added = cmd.ExecuteNonQuery(); 

    lblResult.Text = added.ToString() + " record opgeslagen."; 
    lblResult.Text = addedkey.ToString(); 
} 

Mais quand je commente sur ExecuteScalar et ExecuteNonQuery tout fonctionne bien. Donc, si je commente l'un des deux, l'enregistrement sera inséré deux fois.

Cela a commencé à se produire après avoir ajouté un bouton de suppression, ce qui n'a rien à voir avec la commande d'insertion.

Est-ce que quelqu'un sait ce qui cause cela?

Merci d'avance. :)

Answer 1

Vous exécutez votre insertion chaque fois que vous exécutez une méthode 'Execute ...()' sur la commande. Vous l'exécutez trois fois dans votre code, vous insérez donc 3x enregistrements.

object addedkey = cmd.ExecuteScalar(); //you run the insert command here 
    if ((addedkey != null && (addedkey != DBNull.Value))) 
    { 
     addedkey = Convert.ToInt32(cmd.ExecuteScalar());//you run the insert command here 
    } 
    added = cmd.ExecuteNonQuery();//you run the insert command here 
    lblResult.Text = added.ToString() + " record opgeslagen."; 
    lblResult.Text = addedkey.ToString(); 

Answer 2

Vous exécutez 3 fois. Donc vraiment, vous devez refactoriser le code après la première exécution.

Vous pouvez modifier

addedkey = Convert.ToInt32(cmd.ExecuteScalar()); 

à cette question - cela empêchera la 2ème exécution:

addedkey = Convert.ToInt32(addedKey); 

et changer ceci:

added = cmd.ExecuteNonQuery(); 

à cette question - cela empêchera la 3ème exécution (ceci suppose que added est un int, et d'être le retour de votre exécution.)

added = addedKey; 

Si added est en fait un boolean puis mettez-le dans votre if car elle aura été validée par votre déclaration if

IE

{ 
    addedkey = Convert.ToInt32(addedKey);//you run the insert command here 
    added = true; 
}