Почему при нажатии кнопки удаления выходит ошибка типа: «ошибка синтаксиса в инструкции from»?

Question

[Nikita_Kotov228]

Привет! Я ввожу код товара в текстбокс, после чего нажимаю кнопку удалить, у меня открывается код с ошибкой. Помогите пожалуйста, первый раз такое делаю. Выделяет строчку "com.ExecuteNonQuery();". Код:

private void buttonDelete_Click(object sender, EventArgs e)
        {
            int code1 = Convert.ToInt32(textBox1.Text); //создаём переменную int и берём её значение из текстабока1 (код_товора)
            //переменная, которая хранит SQL запрос
            string query = "DELETE FROM DataBase WHERE Код = " + code1; //подставляется значение переменной код
            OleDbCommand com = new OleDbCommand(query, Connection); //создаём объект класса, подставляем сам запрос и соединение с базой (2 аргумента)
            com.ExecuteNonQuery(); //исполнение запроса
            MessageBox.Show("Данные о товаре удалены!");
            this.dataBaseTableAdapter.Fill(this.sweet_Tooth2DataSet.DataBase);//обновление таблицы
        }

Answer 1

[Sergey В.]

Скорее всего Database - зарезервированное слово. Назовите таблицу по другому

Answer 2

[cicatrix]

Это не код, это огромная дыра в безопасности!
Вот, почитайте на досуге.
Database - 'это название таблицы такое? И в ней есть поле с кириллическим названием Код?
А в текстовое поле число вводится? А если текст ввести, что будет?
А если я туда введу 1=1; DROP TABLE Database; что будет?

Comments

[DevMan]

кириллическими именами на винде никого не удивить.
а sql-i - вообще отдельная тема.

[Nikita_Kotov228]

Ребят, я этим ни разу не занимался. Мне это надо для лабы. Написать стандартные функции с подключением файла аксесс. Я всё делал по видео, но стало всё плохо...

[cicatrix]

Nikita_Kotov228, Как таблица в базе называется? И какие в ней колонки

Answer 3

[Виталий Качан]

В таблице точно колонка с именем 'код' по-русски, или может все таки 'Code'?
Ну и стоит делать хотя бы так:

using (OleDbCommand cmd = Connection.CreateCommand())
{
     cmd.CommandText = 
              "DELETE FROM DataBase WHERE Code = @code";

           // add named parameters
           cmd.Parameters.AddRange(new OleDbParameter[]
           {
               new OleDbParameter("@code", code1)
           });

           // execute
           cmd.ExecuteNonQuery();
}