c# query fault with insert to database

Question

I have a query that uses update and also insert but i get a error with insert

this is my code and alwas getting a error

{"Must declare the scalar variable \"@PatientID\"."}

    private void btnOpslaan_Click(object sender, EventArgs e)
    {
        if (IsGeldigeData())
        {
            patient.PatientID = Convert.ToInt32(txtPatientID.Text);
            patient.Naam = txtNaam.Text;
            patient.Voornaam = txtVoornaam.Text;
            patient.Straat = txtStraat.Text;
            patient.Huisnummer = txtHuisnr.Text;
            if (txtPostcodeID.Text != "") patient.PostcodeID = Convert.ToInt32(txtPostcodeID.Text);
            patient.Geboortedatum = dtGeboortedatum.Value;
            if (txtHuidige_Afdeling.Text != "") patient.Huidige_Afdeling = Convert.ToInt32(txtHuidige_Afdeling.Text);
            patient.Huidige_Kamer = txtHuidige_Kamer.Text;
            if (txtHuidige_HuisartsID.Text != "") patient.Huidige_Huisarts = Convert.ToInt32(txtHuidige_HuisartsID.Text);
            if (txtHuidige_ZiekenhuisartsID.Text != "") patient.Huidige_Ziekenhuisarts = Convert.ToInt32(txtHuidige_ZiekenhuisartsID.Text);
            if (txtHuidige_Opnamenr.Text != "") patient.Huidige_Opnamenr = Convert.ToInt32(txtHuidige_Opnamenr.Text);

            string insertStatement = @"UPDATE Patient SET  Naam = @Naam, Voornaam = @Voornaam, Straat = @Straat, 
                                    Huisnummer = @Huisnummer, PostcodeID = @PostcodeID, Geboortedatum = @Geboortedatum, 
                                    Huidige_Afdeling = @Huidige_Afdeling, Huidige_Kamer = @Huidige_Kamer, Huidige_HuisartsID = 
                                    @Huidige_HuisartsID, Huidige_ZiekenhuisartsID = @Huidige_ZiekenhuisartsID,
                                    Huidige_Opnamenr = @Huidige_Opnamenr WHERE PatientID = @PatientID
                                    IF @@ROWCOUNT=0 
                                    INSERT INTO Patient (Naam, Voornaam, Straat, Huisnummer, PostcodeID, Geboortedatum, 
                                    Huidige_Afdeling, Huidige_Kamer, Huidige_HuisartsID, Huidige_ZiekenhuisartsID, Huidige_Opnamenr) 
                                    VALUES  (@Naam,@Voornaam,@Straat,@Huisnummer,@PostcodeID,@Geboortedatum,@Huidige_Afdeling,@Huidige_Kamer,
                                                @Huidige_HuisartsID,@Huidige_ZiekenhuisartsID,@Huidige_Opnamenr)";
            SqlCommand insertCommand = new SqlCommand(insertStatement, cs);
            SqlCommand updateCommand = new SqlCommand(insertStatement, cs);

Where are you setting your parameters?

Claudio Redi
– Claudio Redi

2013-06-15 22:10:35 +00:00
Commented Jun 15, 2013 at 22:10 — Claudio Redi
– Claudio Redi, Commented Jun 15, 2013 at 22:10

Steve · Accepted Answer · 2013-06-16 07:56:59Z

It is not enough to prepare the sql statement with the placeholders for the parameters value.
You need also to create the parameter collection for the command....

 insertCommand.Parameters.AddWithValue("@PatientID",  patient.PatientID);
 .... and so on for every parameter placeholders

Please rememeber that using AddWithValue is a shortcut with some serious drawbacks.
The Framework decides for you the appropriate datatype and size for the parameter looking at the value you pass. This, in some cases, could lead to very bad performances.
For example, in case of strings the Framework will pass always a NVarChar parameter with size equals to the string length. (It has to guess the varchar/nvarchar/size) So, at every call, the parameter could be of different lenghts. This will be no good for performances because the Sql Server internal optimizer could not store and reuse the query.

A bit lengthier than previous one but more precise approach is: (Assuming Naam is a NVarchar(50) database field).

 insertCommand.Parameters.Add("@Naam", SqlDbType.NVarChar, 50).Value = patient.Naam;

Please see this fine article on

How Data Access Code Affects Database Performance

Collectives™ on Stack Overflow

c# query fault with insert to database

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related