Добавление записи в базу данных mysql

20.05.2013, 20:52. **Просмотров** 9927. **Ответов** 1

Всем привет. Мне нужно объединить 3 таблицы, я их объединила. Вот моя проблема у меня не получается добавление данных, где две таблицы объедены все хорошо, а когда больше двух не получается
вот код добавления:

C#

            string z1 = textBox1.Text;
            string z2 = dateTimePicker1.Text;
            string z3 = dateTimePicker2.Text;
            string z4 = textBox4.Text;
            string z5 = textBox5.Text;
            const string Connect = "Database=kurs;Data Source=localhost;User Id=root;Password=1";
            MySqlConnection con = new MySqlConnection(Connect);
            con.Open(); //Устанавливаем соединение с базой данных.
            MySqlCommand cmd = new MySqlCommand();
 
            cmd.CommandText = (string.Format("INSERT  INTO srok_vidachi(ID,data_vidachi,data_vozvrata,nomer_student,nomer_kniga) VALUES('{0}','" + dateTimePicker1.Value.ToString("yyyy-MM-dd") + "','" + dateTimePicker2.Value.ToString("yyyy-MM-dd") + "',(select student.Nomer from student where FIO='{3}'),(select kniga.Nomer from kniga where kniga.nazvanie='{3}'))", z1, z2, z3, z4, z5));
            cmd.Connection = con;
            cmd.ExecuteNonQuery();
            MessageBox.Show("Добавление срока выдачи прошло успешно", "Добавление срока выдачи прошло успешно", MessageBoxButtons.OK, MessageBoxIcon.Asterisk);
 
            con.Close(); //Обязательно закрываем соединение!

Помогите пожалуйста кто знает, заранее спасибо

@ImNIK · 21.05.2013, 14:16

1. Поставь distinct во вложенных запросах (select DISTINCT student.Nomer from student where FIO='{3}'). Ведь могут совпасть ФИО студентов и название книг - это уже должно помочь, но.
2. Код из-за инъекций в SQL почти не читаем.... передавай все значения через параметры!
3. Даже инъекция сделана через одно место, вначале string.format... затем строки плюсуются...

Добавлено через 1 час 13 минут
Как-то так в общем:

C#

 
            string z1 = textBox1.Text;
            string z2 = dateTimePicker1.Text;
            string z3 = dateTimePicker2.Text;
            string z4 = textBox4.Text;
            string z5 = textBox5.Text;
            const string Connect = "Database=kurs;Data Source=localhost;User Id=root;Password=1";
           using (SqlConnection conn = new SqlConnection(Connect))
            {
                conn.Open(); //Устанавливаем соединение с базой данных.
                MySqlCommand cmd = new MySqlCommand();
                cmd.Connection = conn;
                cmd.CommandText = @"INSERT INTO srok_vidachi(ID,data_vidachi,data_vozvrata,nomer_student,nomer_kniga)
                                    VALUES(
                                           @ID, 
                                           @data_vidachi, 
                                           @data_vozvrata, 
                                           (select distinct student.Nomer from student where student.FIO=@FIO),
                                           (select distinct kniga.Nomer from kniga where kniga.nazvanie=@nazvanie)
                                           )";
                cmd.Parameters.Add("@ID", MySqlDbType.Int);
                cmd.Parameters["@ID"].Value = z1;
                cmd.Parameters.Add("@data_vidachi", MySqlDbType.DateTime);
                cmd.Parameters["@data_vidachi"].Value = z2;
                cmd.Parameters.Add("@data_vozvrata", MySqlDbType.DateTime);
                cmd.Parameters["@data_vozvrata"].Value = z3;
                cmd.Parameters.Add("@FIO", MySqlDbType.VarChar);
                cmd.Parameters["@FIO"].Value = z4;
                cmd.Parameters.Add("@nazvanie", MySqlDbType.VarChar);
                cmd.Parameters["@nazvanie"].Value = z5;
                cmd.ExecuteNonQuery();
                MessageBox.Show("Добавление срока выдачи прошло успешно", "Добавление срока выдачи прошло успешно", MessageBoxButtons.OK, MessageBoxIcon.Asterisk);
                //  conn.Close(); при использовании using (SqlConnection conn... не обязательно  //Обязательно закрываем соединение!  
            }

Опции темы