Изменение данных в БД. Запрос UPDATE

@rustorkan · Регистрация: 07.06.2017

Студворк — интернет-сервис помощи студентам

Здравствуйте!
Неправильный синтаксис около ключевого слова "AS"

C#
1
SqlCommand com = new SqlCommand("UPDATE АБОНЕНТЫ SET [№] AS 'ПриложениеАбонента' = '" + textBox1.Text + "', [Фамилия] = '" + textBox_familia.Text + "', [Имя] = '" + textBox_imja.Text + "', [Отчество] = '" + textBox_otchestvo.Text + "', [Адрес] = '" + textBox_adres.Text + "'", con);

@Usaga · 13.07.2018, 05:25

rustorkan, всё верно, синтаксис не верный. Корректнее так:

SQL
1
UPDATE ИмяТаблицы SET Поле1 = значение, Поле2 = значение

Параметры в запрос нужно передавать иным образом:

C#
1
2
var com = new SqlCommand("UPDATE АБОНЕНТЫ SET [ПриложениеАбонента] = @application", con);
com.Parameters.AddWithValue("application", textBox1.Text);

Так вы обезопаситесь от SQL Injection и ускорите процесс разбора запроса, ибо у СУБД появится возможность его кешировать и переиспользовать просто заменив параметры.

Кстати, русский язык (или любой другой, кроме инглиша) использовать в именах таблиц и полей - моветон.

@rustorkan · 13.07.2018, 06:53 **[ТС]**

Usaga, тогда меняется весь столбец в dataGridview. А как изменить выделенную строку?

@Usaga · 13.07.2018, 07:01

rustorkan, нужно в запросе уточнить, какую именно нужно запись обновить. На данный момент вы обновляете вообще все записи в таблице абонентов.

У каждой записи есть свой идентификатор, его нужно указать в запросе.

@DenKG · 13.07.2018, 20:20

rustorkan, используй оператор WHERE для уточнения.

@rustorkan · 14.07.2018, 23:02 **[ТС]**

Неправильный синтаксис около конструкции ",".

C#
1
SqlCommand com = new SqlCommand("UPDATE [АБОНЕНТЫ] SET [№] = @ПриложениеАбонента, [Фамилия] = @Фамилия, [Имя] = @Имя, [Отчество] = @Отчество, [Адрес] = @Адрес WHERE [№] = 'dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value', [Фамилия] = 'dataGridabon[1, dataGridabon.CurrentCell.RowIndex].Value', [Имя] = 'dataGridabon[2, dataGridabon.CurrentCell.RowIndex].Value', [Отчество] = 'dataGridabon[3, dataGridabon.CurrentCell.RowIndex].Value', [Адрес] = 'dataGridabon[4, dataGridabon.CurrentCell.RowIndex].Value'", con);

@DenKG · 14.07.2018, 23:05

rustorkan, после WHERE для перечисления нужно использовать AND вместо запятой

@rustorkan · 14.07.2018, 23:12 **[ТС]**

DenKG, ошибка преобразования значения varchar в тип данных int.

Добавлено через 36 секунд

C#
1
2
3
4
5
6
7
8
9
10
11
SqlCommand com = new SqlCommand("UPDATE [АБОНЕНТЫ] SET [ПриложениеАбонента] = @ПриложениеАбонента, [Фамилия] = @Фамилия, [Имя] = @Имя, [Отчество] = @Отчество, [Адрес] = @Адрес WHERE [ПриложениеАбонента] = 'dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value' AND [Фамилия] = 'dataGridabon[1, dataGridabon.CurrentCell.RowIndex].Value' AND [Имя] = 'dataGridabon[2, dataGridabon.CurrentCell.RowIndex].Value' AND [Отчество] = 'dataGridabon[3, dataGridabon.CurrentCell.RowIndex].Value' AND [Адрес] = 'dataGridabon[4, dataGridabon.CurrentCell.RowIndex].Value'", con);
 
com.Parameters.AddWithValue("ПриложениеАбонента", Convert.ToInt32(textBox1.Text));
 
com.Parameters.AddWithValue("Фамилия", textBox_familia.Text);
 
com.Parameters.AddWithValue("Имя", textBox_imja.Text);
 
com.Parameters.AddWithValue("Отчество", textBox_otchestvo.Text);
 
com.Parameters.AddWithValue("Адрес", textBox_adres.Text);

@DenKG · 14.07.2018, 23:24

rustorkan, нужно параметры указывать через @. Например, "@ПриложениеАбонента"

Добавлено через 4 минуты
rustorkan, а вообще такие запросы - это немного наркомания. Лучше используй для этого СУБД и хранимые процедуры

@rustorkan · 14.07.2018, 23:24 **[ТС]**

DenKG, ошибка преобразования значения varchar "dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value" в тип данных int.

C#
1
2
3
4
5
6
7
8
9
10
11
SqlCommand com = new SqlCommand("UPDATE [АБОНЕНТЫ] SET [ПриложениеАбонента] = @ПриложениеАбонента, [Фамилия] = @Фамилия, [Имя] = @Имя, [Отчество] = @Отчество, [Адрес] = @Адрес WHERE [ПриложениеАбонента] = 'dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value' AND [Фамилия] = 'dataGridabon[1, dataGridabon.CurrentCell.RowIndex].Value' AND [Имя] = 'dataGridabon[2, dataGridabon.CurrentCell.RowIndex].Value' AND [Отчество] = 'dataGridabon[3, dataGridabon.CurrentCell.RowIndex].Value' AND [Адрес] = 'dataGridabon[4, dataGridabon.CurrentCell.RowIndex].Value'", con);
 
com.Parameters.AddWithValue("@ПриложениеАбонента", Convert.ToInt32(textBox1.Text));
 
com.Parameters.AddWithValue("@Фамилия", textBox_familia.Text);
 
com.Parameters.AddWithValue("@Имя", textBox_imja.Text);
 
com.Parameters.AddWithValue("@Отчество", textBox_otchestvo.Text);
 
com.Parameters.AddWithValue("@Адрес", textBox_adres.Text);

@DenKG · 14.07.2018, 23:27

rustorkan, ну здесь тебе нужно привести значение из таблицы в int

C#
1
Convert.ToInt32(dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value)

@rustorkan · 14.07.2018, 23:35 **[ТС]**

DenKG, не помогает.
Ошибка преобразования значения varchar "Convert.ToInt32(dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value)" в тип данных int.

C#
1
2
3
4
5
6
7
8
9
10
11
SqlCommand com = new SqlCommand("UPDATE [АБОНЕНТЫ] SET [ПриложениеАбонента] = @ПриложениеАбонента, [Фамилия] = @Фамилия, [Имя] = @Имя, [Отчество] = @Отчество, [Адрес] = @Адрес WHERE [ПриложениеАбонента] = 'Convert.ToInt32(dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value)' AND [Фамилия] = 'dataGridabon[1, dataGridabon.CurrentCell.RowIndex].Value' AND [Имя] = 'dataGridabon[2, dataGridabon.CurrentCell.RowIndex].Value' AND [Отчество] = 'dataGridabon[3, dataGridabon.CurrentCell.RowIndex].Value' AND [Адрес] = 'dataGridabon[4, dataGridabon.CurrentCell.RowIndex].Value'", con);
 
com.Parameters.AddWithValue("@ПриложениеАбонента", Convert.ToInt32(textBox1.Text));
 
com.Parameters.AddWithValue("@Фамилия", textBox_familia.Text);
 
com.Parameters.AddWithValue("@Имя", textBox_imja.Text);
 
com.Parameters.AddWithValue("@Отчество", textBox_otchestvo.Text);
 
com.Parameters.AddWithValue("@Адрес", textBox_adres.Text);

@DenKG · 14.07.2018, 23:38

rustorkan, и еще в WHERE, раз уж ты решил со всем этим заморочиться, правильнее писать так:

C#
1
2
("...WHERE [ПриложениеАбонента] = '" + Convert.ToInt32(dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value)
 + "' AND...", con);

Добавлено через 2 минуты
rustorkan, в твоем случае из-за обычных одинарных апострофов запрос пытается принять строку Convert.ToInt32(dataGridabon[0, dataGridabon.CurrentCell.RowIndex].Value) как обычный символ. Для того, чтобы этого не было, нужно использовать ' '' + ... + " '

@rustorkan · 14.07.2018, 23:46 **[ТС]**

DenKG, ссылка на объект не указывает на экземпляр объекта.

Добавлено через 42 секунды

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
Abon_Form F = new Abon_Form();
 
SqlCommand com = new SqlCommand("UPDATE [АБОНЕНТЫ] SET [ПриложениеАбонента] = @ПриложениеАбонента, [Фамилия] = @Фамилия, [Имя] = @Имя, [Отчество] = @Отчество, [Адрес] = @Адрес WHERE [ПриложениеАбонента] = '" + Convert.ToInt32(F.dataGridabon[0, F.dataGridabon.CurrentCell.RowIndex].Value) + "' AND [Фамилия] = '" + F.dataGridabon[1, F.dataGridabon.CurrentCell.RowIndex].Value + "' AND [Имя] = '" + F.dataGridabon[2, F.dataGridabon.CurrentCell.RowIndex].Value + "' AND [Отчество] = '" + F.dataGridabon[3, F.dataGridabon.CurrentCell.RowIndex].Value + "' AND [Адрес] = '" + F.dataGridabon[4, F.dataGridabon.CurrentCell.RowIndex].Value + "'", con);
 
com.Parameters.AddWithValue("@ПриложениеАбонента", Convert.ToInt32(textBox1.Text));
 
com.Parameters.AddWithValue("@Фамилия", textBox_familia.Text);
 
com.Parameters.AddWithValue("@Имя", textBox_imja.Text);
 
com.Parameters.AddWithValue("@Отчество", textBox_otchestvo.Text);
 
com.Parameters.AddWithValue("@Адрес", textBox_adres.Text);

@DenKG · 14.07.2018, 23:46

rustorkan, в какой строке? Поставь точку остановы и посмотри, после какой строки вылетает Exception

@rustorkan · 14.07.2018, 23:52 **[ТС]**

DenKG,

C#
1
SqlCommand com = new SqlCommand("UPDATE [АБОНЕНТЫ] SET [ПриложениеАбонента] = @ПриложениеАбонента, [Фамилия] = @Фамилия, [Имя] = @Имя, [Отчество] = @Отчество, [Адрес] = @Адрес WHERE [ПриложениеАбонента] = '" + Convert.ToInt32(F.dataGridabon[0, F.dataGridabon.CurrentCell.RowIndex].Value) + "' AND [Фамилия] = '" + F.dataGridabon[1, F.dataGridabon.CurrentCell.RowIndex].Value + "' AND [Имя] = '" + F.dataGridabon[2, F.dataGridabon.CurrentCell.RowIndex].Value + "' AND [Отчество] = '" + F.dataGridabon[3, F.dataGridabon.CurrentCell.RowIndex].Value + "' AND [Адрес] = '" + F.dataGridabon[4, F.dataGridabon.CurrentCell.RowIndex].Value + "'", con);

@DenKG · 15.07.2018, 00:07

Сообщение от rustorkan

Abon_Form F = new Abon_Form();

Зачем это?

@rustorkan · 15.07.2018, 01:08 **[ТС]**

DenKG, DataGridView находится на одной форме. При нажатии на кнопку Изменить открывается вторая форма с заполненными TextBox из выделенной строки в DataGridView. Мне нужно эти изменения сохранить.

@DenKG · 15.07.2018, 01:16

rustorkan, то есть DataGrid Находится на одной форме, ТекстБоксы на другой и значения из этих текстБоксов ты хочешь передать и сохранить их в БД?

@rustorkan · 15.07.2018, 01:23 **[ТС]**

DenKG, верно.

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@rustorkan 0 / 0 / 0 Регистрация: 07.06.2017 Сообщений: 70
	13.07.2018, 06:53 [ТС]
	Usaga, тогда меняется весь столбец в dataGridview. А как изменить выделенную строку? 0

@Usaga 14087 / 9305 / 1348 Регистрация: 21.01.2016 Сообщений: 34,952
	13.07.2018, 07:01
	rustorkan, нужно в запросе уточнить, какую именно нужно запись обновить. На данный момент вы обновляете вообще все записи в таблице абонентов. У каждой записи есть свой идентификатор, его нужно указать в запросе. 0

@DenKG 11 / 11 / 7 Регистрация: 23.12.2015 Сообщений: 950
	13.07.2018, 20:20
	rustorkan, используй оператор `WHERE` для уточнения. 0

@DenKG 11 / 11 / 7 Регистрация: 23.12.2015 Сообщений: 950
	14.07.2018, 23:05
	rustorkan, после `WHERE` для перечисления нужно использовать `AND` вместо запятой 0

@DenKG 11 / 11 / 7 Регистрация: 23.12.2015 Сообщений: 950
	14.07.2018, 23:24
	rustorkan, нужно параметры указывать через @. Например, "@ПриложениеАбонента" Добавлено через 4 минуты rustorkan, а вообще такие запросы - это немного наркомания. Лучше используй для этого СУБД и хранимые процедуры 0

@DenKG 11 / 11 / 7 Регистрация: 23.12.2015 Сообщений: 950
	14.07.2018, 23:46
	rustorkan, в какой строке? Поставь точку остановы и посмотри, после какой строки вылетает Exception 0

@rustorkan 0 / 0 / 0 Регистрация: 07.06.2017 Сообщений: 70
	15.07.2018, 01:08 [ТС]
	DenKG, DataGridView находится на одной форме. При нажатии на кнопку Изменить открывается вторая форма с заполненными TextBox из выделенной строки в DataGridView. Мне нужно эти изменения сохранить. 0

@DenKG 11 / 11 / 7 Регистрация: 23.12.2015 Сообщений: 950
	15.07.2018, 01:16
	rustorkan, то есть DataGrid Находится на одной форме, ТекстБоксы на другой и значения из этих текстБоксов ты хочешь передать и сохранить их в БД? 0

@rustorkan 0 / 0 / 0 Регистрация: 07.06.2017 Сообщений: 70
	15.07.2018, 01:23 [ТС]
	DenKG, верно. 0