Как исправить ошибки?

Question

[Андрей Фомин]

Здравствуйте!
Вот мои несколько ошибок:
1. Ошибка синтаксическая в sql-запросе, но что не так я в упор не вижу

sqlUpdate =
                @"UPDATE Автомобиль SET WHERE VIN_номер = @VIN AND Номер_заказа = @Заказ AND Номер_филиала = @Филиал";

2. Данный код удаляет запись из базы данных, и вроде как переменная объявлена, так в чем проблема?

string sqlDelete = "";
                int zakaz = Convert.ToInt32(Nomer_zakaza.Text);
                int filial = Convert.ToInt32(Nomer_filiala.Text);
                sqlDelete =
                @"DELETE FROM Автомобиль WHERE VIN_номер = @VIN AND Номер_заказа = @Заказ AND Номер_филиала = @Филиал";
                SqlConnection connection = new SqlConnection("Data Source = LAPTOP-KJLU9H2F\\SQLEXPRESS; Initial Catalog = 'Автосалон'; Integrated Security = True");
                connection.Open();
                SqlCommand command = connection.CreateCommand();
                command.CommandText = sqlDelete;
                command.Parameters.Add("@VIN", SqlDbType.VarChar, 20).Value = VIN.Text;
                command.Parameters.Add("@Заказ", SqlDbType.Int).Value = zakaz;
                command.Parameters.Add("@Филиал", SqlDbType.Int).Value = filial;
                SqlCommand cmd = new SqlCommand(sqlDelete, connection);
                cmd.ExecuteNonQuery();

3. Данный код добавляет запись из базы данных, и здесь тоже переменная объявлена, так в чем проблема?

string sqlInsert = "";
                int zakaz = Convert.ToInt32(Nomer_zakaza.Text);
                int filial = Convert.ToInt32(Nomer_filiala.Text);
                sqlInsert =
                @"INSERT INTO Автомобиль (VIN_номер, Номер_заказа, Номер_филиала) VALUES (@VIN,@Заказ,@Филиал)";
                SqlConnection connection = new SqlConnection("Data Source = LAPTOP-KJLU9H2F\\SQLEXPRESS; Initial Catalog = 'Автосалон'; Integrated Security = True");
                connection.Open();
                SqlCommand command = connection.CreateCommand();
                command.CommandText = sqlInsert;
                command.Parameters.Add("@VIN", SqlDbType.VarChar, 20).Value = VIN.Text;
                command.Parameters.Add("@Заказ", SqlDbType.Int).Value = zakaz;
                command.Parameters.Add("@Филиал", SqlDbType.Int).Value = filial;
                SqlCommand cmd = new SqlCommand(sqlInsert, connection);
                cmd.ExecuteNonQuery();

Comments

[FanatPHP]

SET WHERE
серьёзно, вот прямо не видишь?

[Андрей Фомин]

FanatPHP, Понял
WHERE надо убрать

[FanatPHP]

эээ, совсем убрать?
Ну, я надеюсь, ты не на боевой базе тестируешь

[Андрей Фомин]

FanatPHP, нет, сделал специально тестовую
А разве WHERE убрать - это не правильно?

[FanatPHP]

ну а сам как думаешь?
для чего нужен этот запрос? Что он делает, какую задачу решает?

[Андрей Фомин]

FanatPHP, Задача этого запроса - обновить запись в таблице

[FanatPHP]

запись? а как база данных узнает - какую именно запись надо обновить?

Answer 1

[Rsa97]

1. После SET должен идти список изменяемых полей со значениями вида SET field1 = 'value1', field2 = @value2
2, 3. Вы создаёте команду command, заполняете её параметры, затем создаёте новую команду cmd, которую и пытаетесь вызвать не заполнив её параметры.

Comments

[Андрей Фомин]

Ладно, если я правильно понял, то здесь SqlCommand cmd = new SqlCommand(sqlInsert, connection); вместо sqlInsert надо записать command, но тогда получим сообщение о том, что не удается преобразовать из "System.Data.SqlClient.SqlCommand" в "string". Как тогда быть?

[Rsa97]

Андрей Фомин, Вы неправильно поняли.

[Андрей Фомин]

Rsa97, Я ожидал такой исход.
Покажите тогда как правильно должно быть

[Rsa97]

Андрей Фомин, Правильно создавать только одну команду, которую и выполнять.

[Андрей Фомин]

Rsa97, Хорошо, ну вот команда есть, но как ее правильно запустить?
Если я правильно понял, то за исполнение отвечает этот кусок?

SqlCommand cmd = new SqlCommand(sqlInsert, connection);
                cmd.ExecuteNonQuery();

[Андрей Фомин]

Rsa97, ну или есть говорить прямо, как должно все выглядеть, что все нормально работало?

[Rsa97]

Андрей Фомин,

SqlCommand cmd = new SqlCommand(sqlInsert, connection);
cmd.ExecuteNonQuery();

заменить на
command.ExecuteNonQuery();

[Андрей Фомин]

Rsa97, Я понял, спасибо за помощь)
Еще один момент хотел уточнить
вот у меня есть прараметры:

command.Parameters.Add("@VIN", SqlDbType.VarChar, 20).Value = VIN.Text;
                command.Parameters.AddWithValue("@Страна", SqlDbType.Text).Value = Strana.Text;
                command.Parameters.AddWithValue("@Модель", SqlDbType.Text).Value = Model.Text;
                command.Parameters.AddWithValue("@База", SqlDbType.Int).Value = baza;
                command.Parameters.AddWithValue("@Кузов", SqlDbType.Text).Value = Kuzov.Text;
                command.Parameters.AddWithValue("@Двигатель", SqlDbType.Text).Value = Dvigatel.Text;
                command.Parameters.AddWithValue("@Коробка", SqlDbType.Text).Value = Transmissia.Text;
                command.Parameters.AddWithValue("@Год", SqlDbType.Int).Value = god;
                command.Parameters.AddWithValue("@Завод", SqlDbType.Int).Value = zavod;
                command.Parameters.AddWithValue("@Номер", SqlDbType.Int).Value = nomer;
                command.Parameters.Add("@ID", SqlDbType.Int).Value = ID;

И вот таблица с которой идет работа

Параметры совпадают с таблицей, но получаю вот это:

Что делать?

[Rsa97]

Андрей Фомин, Текст запроса покажите.

[Андрей Фомин]

Rsa97, Пожалуйста

sqlSelect =
                @"SELECT * FROM VIN_номер WHERE VIN_номер = @VIN AND Регион_Страна_Фирма_расшифровка = @Страна 
                AND Модель_расшифровка = @Модель AND Колесная_база_расшифровка = @База AND Тип_кузова_расшифровка = @Кузов 
                AND Двигатель_расшифровка = @Двигатель AND Тип_трансмиссии_расшифровка = @Коробка AND 
                Год_выпуска_расшифровка = @Год AND Отделение_завода_расшифровка = @Год 
                AND Порядковый_номер = @Номер AND ID = @ID";

[Rsa97]

Андрей Фомин, А таблица точно называется VIN_номер?
Ну и, за исключением Отделение_завода_расшифровка = @Год ошибок, вроде, не вижу.

[Андрей Фомин]

Rsa97, да, точно

[Rsa97]

Андрей Фомин, Тогда не знаю. Убирайте все параметры и добавляйте по одному, ищите, когда ошибка появится.

[Андрей Фомин]

Rsa97, А что с отделение завода было не так?

[Foggy Finder]

неужели не видите что вы тут два раза с параметром "год" сравниваете?

Год_выпуска_расшифровка = @Год AND Отделение_завода_расшифровка = @Год

[Андрей Фомин]

Foggy Finder, Спасибо, а что тогда подскажете в данной ситуации?

Вот код:

private void button4_Click(object sender, EventArgs e)
        {
            try
            {
                string sqlSelect = "";
                int baza = Convert.ToInt32(Baza.Text);
                int god = Convert.ToInt32(God_vipuska.Text);
                int zavod = Convert.ToInt32(Otdelenie_zavoda.Text);
                int nomer = Convert.ToInt32(Poryadkoviy_nomer.Text);
                int ID = Convert.ToInt32(id2.Text);
                sqlSelect =
                @"SELECT * FROM VIN_номер WHERE VIN_номер = @VIN AND Регион_Страна_Фирма_расшифровка = @Страна 
                AND Модель_расшифровка = @Модель AND Колесная_база_расшифровка = @База AND Тип_кузова_расшифровка = @Кузов 
                AND Двигатель_расшифровка = @Двигатель AND Тип_трансмиссии_расшифровка = @Коробка AND 
                Год_выпуска_расшифровка = @Год AND Отделение_завода_расшифровка = @Завод AND Порядковый_номер = @Номер AND ID = @ID";
                SqlConnection connection = new SqlConnection("Data Source = LAPTOP-KJLU9H2F\\SQLEXPRESS; Initial Catalog = 'Автосалон'; Integrated Security = True");
                connection.Open();
                SqlCommand command = connection.CreateCommand();
                command.CommandText = sqlSelect;
                command.Parameters.Add("@VIN", SqlDbType.VarChar, 20).Value = VIN.Text;
                command.Parameters.AddWithValue("@Страна", SqlDbType.Text).Value = Strana.Text;
                command.Parameters.AddWithValue("@Модель", SqlDbType.Text).Value = Model.Text;
                command.Parameters.AddWithValue("@База", SqlDbType.Int).Value = baza;
                command.Parameters.AddWithValue("@Кузов", SqlDbType.Text).Value = Kuzov.Text;
                command.Parameters.AddWithValue("@Двигатель", SqlDbType.Text).Value = Dvigatel.Text;
                command.Parameters.AddWithValue("@Коробка", SqlDbType.Text).Value = Transmissia.Text;
                command.Parameters.AddWithValue("@Год", SqlDbType.Int).Value = god;
                command.Parameters.AddWithValue("@Завод", SqlDbType.Int).Value = zavod;
                command.Parameters.AddWithValue("@Номер", SqlDbType.Int).Value = nomer;
                command.Parameters.Add("@ID", SqlDbType.Int).Value = ID;
                SqlDataAdapter adapter = new SqlDataAdapter();
                adapter.SelectCommand = command;
                DataTable table = new DataTable();
                adapter.Fill(table);
                dataGridView2.DataSource = table;
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }

        }

И вот таблица:

[Андрей Фомин]

Foggy Finder, С этим не нужно помогать,я уже разобрался, нужно помочь найти ошибку в синтаксисе рядом с "AND":

sqlUpdate =
                @"UPDATE VIN_номер SET VIN_номер = @VIN AND Регион_Страна_Фирма_расшифровка = @Страна 
                AND Модель_расшифровка = @Модель AND Колесная_база_расшифровка = @База AND Тип_кузова_расшифровка = @Кузов 
                AND Двигатель_расшифровка = @Двигатель AND Тип_трансмиссии_расшифровка = @Коробка AND 
                Год_выпуска_расшифровка = @Год AND Отделение_завода_расшифровка = @Завод AND Порядковый_номер = @Номер WHERE ID = @ID";

[Foggy Finder]

Андрей Фомин, вам же в ответе написали что перечислять поля в Update нужно через "," а не через AND