Почему не работает UPDATE?

Question

[baroman4ik]

"Ошибка синтаксиса в инструкции UPDATE.", хотя вроде всё написано правильно.

private void button1_Click(object sender, EventArgs e)
        {
            string connectString = "Provider=Microsoft.Jet.OLEDB.4.0;Data Source=BD1.mdb";
            con = new OleDbConnection(connectString);
            con.Open();
            string query = "";
            if (frm.radVal == 0)
            {
                query = string.Format("UPDATE Developers SET Developer_name = '" + textBox1.Text +"', Developer_specialty = '"+ textBox3.Text +"', team_id = "+ textBox2.Text +", Password = '"+ textBox4.Text +"' WHERE Developer_id = "+ id +";");
            }
            else if (frm.radVal == 1)
            {
                query = "UPDATE Managers SET Manager_name = '" + textBox1.Text + "', Admin_id = '" + textBox3.Text + "', Password = '" + textBox2.Text + "' WHERE Manager_id = '" + id + "'";
            }
            else if (frm.radVal == 2)
            {
                query = "UPDATE Admins SET Admin_name = '" + textBox1.Text + "', Password = '" + textBox2.Text + "' WHERE Admin_id = '" + id + "'";
            }

            cmd = new OleDbCommand(query, con);
            MessageBox.Show(query);
            int number = cmd.ExecuteNonQuery();
            Console.WriteLine("Обновлено объектов: {0}", number);
            //MessageBox.Show(cmd.ExecuteScalar().ToString());
            if (button1.Focused)
            {
                MessageBox.Show("Данные успешно обновлены");
            }

        }

Comments

[Foggy Finder]

чему в итоге равно `query`? Но сам запрос лучше так не строить, есть параметры

[Михаил]

sql injection? что это?

Answer 1

[Дмитрий Беляев]

https://docs.microsoft.com/ru-ru/dotnet/api/system...

Comments

[baroman4ik]

Доку эту видел, но так как я эту строку составляю в string добавить параметр так не выйдет, а как переделать чтобы весь остальной код отправки работал не знаю как

Answer 2

[Семён Аржанов]

Посмотри в строчке = cmd.ExecuteNonQuery() там ошибка!

Comments

[baroman4ik]

имеете ввиду что должен быть прямой вызов а не присваивание?

Answer 3

[Константин Цветков]

team_id = "+ textBox2.Text +", Если textBox2.Text пустой, будет ошибка.

строка формируется нормально, но всё равно ошибка

Проверьте ещё раз (и не путайте MS SQL и Access SQL — это разные вещи, хотя где-то совместимые): реководство.

Comments

[baroman4ik]

перед отправкой запроса делал вывод и строка формируется нормально, но всё равно ошибка