C# - Submitting Windows form to SQLite database

I can use the following code to submit to a database on button click:

private void btn_add_Click(object sender, EventArgs e)
        {

            try
            {
                myConnection.Open();
                SQLiteCommand myCommand = new SQLiteCommand(myConnection);
                myCommand.CommandText = "Insert Into Person (FirstName,LastName,Email,Address,HomePhone,WorkPhone,MobilePhone) " +
                                        "Values('" + txt_firstName.Text + "','" + txt_lastName.Text + "','" + txt_email.Text + "','" +
                                         txt_address.Text + "','" + txt_homePhone.Text + "','" +
                                         txt_workPhone.Text + "','" + txt_mobilePhone.Text + "')";
                myCommand.ExecuteNonQuery();
                myConnection.Close();
                readData();
                //MessageBox.Show("Your new data saved.");
                txt_address.Text = "";
                txt_email.Text = "";
                txt_firstName.Text = "";
                txt_homePhone.Text = "";
                txt_lastName.Text = "";
                txt_mobilePhone.Text = "";
                txt_workPhone.Text = "";
            }
            catch (Exception)
            {

                throw;
            }
        }

This works fine as long as it is just text. I want a variation on this using text, radio buttons, and a combobox. All values should be saved to the database on button click. I'm using the code:

myConnection.Open();
SQLiteCommand myCommand = new SQLiteCommand(myConnection);
myCommand.CommandText = "Insert Into Albums (Artist,AlbumName,Year,CD,Cass,Vinyl,R2R,8Track,Genre) " +
                        "Values('" + txt_artistName.Text + "','" + txt_albumTitle.Text + "','" + txt_year.Text + "','" +
                                     radbtnCD.Select + "','" + radbtnCass.Select + "','" + radbtnVinyl.Select + "','" +
                                     radbtnRtoR.Select + "','" + radbtn8Trk.Select + "','" + combx_genre.Select + "')";

This gives the following error:

Operator '+' cannot be applied to operands of type 'string' and 'method group'. The error goes away if I remove all the components but the text. Can someone point me in the correct direction on how to mix the controls and submit them to the same database record? Thanks in advance.

3 Replies

Know the answer? Post it — somebody with the same question will find it here.

Wim Sturkenboom11y ago
Accepted answer
radbtnVinyl.Select() is a method to select a radio button, not a property that indicates if the radio button is selected or not. To find if a radiobutton is selected or not, you need to use the 'Checked' property; you can convert to string :

radioButton1.Checked.ToString();

For the combobox, you can use the 'SelectedItem' property and convert to string in a similar way as above.
Stan Sypek11y ago
Thanks. So simple, I can't believe I couldn't find this on Google. I figured the ToString, but I was using Select instead of Checked so of course it didn't work. Thanks a lot.

http://i.imgur.com/HpaSuzo.jpg
Manish Kumar Choudhary11y ago
Hi Stan Sypek,

Use ..
adioButtonName>.Checked.ToString();