Таблица - удаление строки

@ut7du · Регистрация: 24.06.2016

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

Привет всем !
есть таблица отображение данных идет в DataGridView

Я удаляю так

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
private void удалитьТекущуюЗаписьToolStripMenuItem_Click(object sender, EventArgs e)
        {
            id1 = dataGridView1.CurrentRow.Index.ToString();
 
            if (MessageBox.Show("Удалить текущую запись ?") == DialogResult.OK )
            {
                using (var con = new SqlConnection(ToDB))
                {
                    con.Open();
                    var cmd = new SqlCommand(DeleteDataString + +Convert.ToInt32(id1) + ");");
                    cmd.Connection = con;
                    cmd.ExecuteNonQuery();
                    con.Close();
                    ShowTable();
                }
            }
        }

не удаляет. Подскажите где моя ошибка .
Спасибо. С уважением....

@Tessio · 01.06.2018, 13:42

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

Подскажите где моя ошибка .

Здесь

C#
1
var cmd = new SqlCommand(DeleteDataString + +Convert.ToInt32(id1) + ");");

@ut7du · 01.06.2018, 17:59 **[ТС]**

Где именно ? Покажите правильный вариант please...

@Tessio · 01.06.2018, 18:25

ut7du, Правильный вариант:

C#
1
2
3
4
var id = 1;
var cmd = new SqlCommand("DELETE FROM [TableName] WHERE [TableName].[Id] = @id");
cmd.Parameters.AddWithValue("@id", id);
cmd.ExecuteNonQuery();

@ut7du · 01.06.2018, 20:13 **[ТС]**

сделал вот так :

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public string DeleteDataString = "DELETE FROM [Ham] WHERE ([id]=";
 
private void удалитьТекущуюЗаписьToolStripMenuItem_Click(object sender, EventArgs e)
        {
            var id1 = dataGridView1.CurrentRow.Index.ToString();
            var myid = Convert.ToInt32(id1);
 
            if (MessageBox.Show("Удалить текущую запись ?") == DialogResult.OK )
            {
                using (var con = new SqlConnection(ToDB))
                {
                    con.Open();
                    var cmd = new SqlCommand(DeleteDataString + @myid);
                    cmd.Connection = con;
                    cmd.Parameters.AddWithValue("@id", id1);
                    cmd.ExecuteNonQuery();
                    con.Close();
                    ShowTable();
                }
            }
        }

Выдало :

@Tessio · 01.06.2018, 20:22

ut7du, ну так покажите что у вас в запросе DeleteDataString

Добавлено через 40 секунд
я же показал как сделать достаточно наглядно

@ut7du · 01.06.2018, 20:23 **[ТС]**

Так в 1 -вой же строчке я написал - пост #5

@Tessio · 01.06.2018, 20:25

C#
1
2
var cmd = new SqlCommand("DELETE FROM [Ham] WHERE [id] = @myid");
cmd.Parameters.AddWithValue("@myid", id1);

@ut7du · 01.06.2018, 20:26 **[ТС]**

сделал вот так :

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 private void удалитьТекущуюЗаписьToolStripMenuItem_Click(object sender, EventArgs e)
        {
            
            //var myid = 
 
            if (MessageBox.Show("Удалить текущую запись ?") == DialogResult.OK )
            {
                using (var con = new SqlConnection(ToDB))
                {
                    var id1 = dataGridView1.CurrentRow.Index.ToString();
                    int id2 = Convert.ToInt32(id1);
                    con.Open();
                    var cmd = new SqlCommand("DELETE FROM [Ham] WHERE (id = @id2");
                    cmd.Connection = con;
                    cmd.Parameters.AddWithValue("@id", id1);
                    cmd.ExecuteNonQuery();
                    con.Close();
                    ShowTable();
                }
            }
        }

тоже ошибка

@Tessio · 01.06.2018, 20:28

Что за скобки в запросе? Внимательно посмотрите на этот код

C#
1
2
var cmd = new SqlCommand("DELETE FROM [Ham] WHERE [id] = @id2");
cmd.Parameters.AddWithValue("@id2", id2);

@id2 - параметр в запросе
cmd.Parameters.AddWithValue("@id2", id2); - присваиваем этому параметру значение.

@ut7du · 01.06.2018, 20:29 **[ТС]**

исправил код:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
if (MessageBox.Show("Удалить текущую запись ?") == DialogResult.OK )
            {
                using (var con = new SqlConnection(ToDB))
                {
                    var id1 = dataGridView1.CurrentRow.Index.ToString();
                    int id2 = Convert.ToInt32(id1);
                    con.Open();
                    var cmd = new SqlCommand("DELETE FROM [Ham] WHERE [Ham].[id] = @id1");
                    cmd.Connection = con;
                    cmd.Parameters.AddWithValue("@id", id1);
                    cmd.ExecuteNonQuery();
                    con.Close();
                    ShowTable();
                }
            }

все равно ошибка

@ut7du · 01.06.2018, 20:31 **[ТС]**

вот скрин с ошибкой

@Tessio · 01.06.2018, 20:50

ut7du, вы не торопитесь, внимательно посмотрите на свой код, а теперь на мой

C#
1
2
3
int id2 = Convert.ToInt32(id1);
var cmd = new SqlCommand("DELETE FROM [Ham] WHERE [Ham].[id] = @id");
cmd.Parameters.AddWithValue("@id", id2);

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

C#
1
2
var id1 = dataGridView1.CurrentRow.Index.ToString();
int id2 = Convert.ToInt32(id1);

CurrentRow.Index - возвращает int, вы его переводите в строку и конвертируете обратно в целочисленное.
Остается вопрос, при чем тут индекс текущей строки datagridview, ведь id в бд и индекс строки может не совпадать.

@ut7du · 01.06.2018, 21:10 **[ТС]**

Вот исправил код:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 private void удалитьТекущуюЗаписьToolStripMenuItem_Click(object sender, EventArgs e)
        {
            if (MessageBox.Show("Удалить текущую запись ?") == DialogResult.OK )
            {
                using (var con = new SqlConnection(ToDB))
                {
                    int id1 = dataGridView1.CurrentRow.Index;    
                    con.Open();
                    var cmd = new SqlCommand("DELETE FROM [Ham] WHERE [Ham].[id] = @id1");
                    cmd.Connection = con;
                    cmd.Parameters.AddWithValue("@id", id1);
                    cmd.ExecuteNonQuery();
                    con.Close();
                    ShowTable();
                }
            }
        }

Все равно выдает ошибку

Добавлено через 2 минуты
Необходимо определить скалярную переменную @id - вылет с такой ошибкой

@Tessio · 01.06.2018, 21:22

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

Необходимо определить скалярную переменную @id - вылет с такой ошибкой

Ну так в запроса @id1, а в параметре @id

Добавлено через 3 минуты
Должно же быть одинаково и там и там.
Вот здесь можно почитать про параметры SqlCommand.Parameters

@ut7du · 01.06.2018, 21:29 **[ТС]**

вот с таким кодом не удаляет

C#
1
2
3
4
5
6
7
8
9
10
11
using (var con = new SqlConnection(ToDB))
                {
                    int id1 = dataGridView1.CurrentRow.Index;    
                    con.Open();
                    var cmd = new SqlCommand("DELETE FROM [Ham] WHERE [Ham].[id] = @id1");
                    cmd.Connection = con;
                    cmd.Parameters.AddWithValue("@id1", id1);
                    cmd.ExecuteNonQuery();
                    con.Close();
                    ShowTable();
                }

@Tessio · 01.06.2018, 21:33

ut7du,

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

Остается вопрос, при чем тут индекс текущей строки datagridview, ведь id в бд и индекс строки может не совпадать.

@ut7du · 02.06.2018, 12:13 **[ТС]**

так как вытащить текущий id ?

Добавлено через 14 часов 31 минуту
Решил ! Все работает !

Новые блоги и статьи Все статьи Все блоги /
Вывод данных через динамический список в справочнике Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .	Функция заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .
Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем	10 пpимет, которые всегда сбываются Maks 31.03.2026 1. Чтобы, наконец, пришла маршрутка, надо закурить. Если сигарета последняя, маршрутка придет еще до второй затяжки даже вопреки расписанию. 2. Нaдоели зима и снег? Не надо переезжать. Достаточно. . .	Перемещение выделенных строк ТЧ из одного документа в другой Maks 31.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .

@ut7du 0 / 0 / 0 Регистрация: 24.06.2016 Сообщений: 161 Записей в блоге: 1
	01.06.2018, 17:59 [ТС]
	Где именно ? Покажите правильный вариант please... 0

@Tessio 79 / 102 / 44 Регистрация: 12.05.2015 Сообщений: 476
	01.06.2018, 20:22
	ut7du, ну так покажите что у вас в запросе `DeleteDataString` Добавлено через 40 секунд я же показал как сделать достаточно наглядно 0

@ut7du 0 / 0 / 0 Регистрация: 24.06.2016 Сообщений: 161 Записей в блоге: 1
	01.06.2018, 20:23 [ТС]
	Так в 1 -вой же строчке я написал - пост #5 0

@ut7du 0 / 0 / 0 Регистрация: 24.06.2016 Сообщений: 161 Записей в блоге: 1
	01.06.2018, 20:31 [ТС]
	вот скрин с ошибкой Миниатюры 0

@ut7du 0 / 0 / 0 Регистрация: 24.06.2016 Сообщений: 161 Записей в блоге: 1
	02.06.2018, 12:13 [ТС]
	так как вытащить текущий id ? Добавлено через 14 часов 31 минуту Решил ! Все работает ! 0