Удаление строки из базы данных MS SQL Server 2012

@lukan2 · Регистрация: 28.01.2010

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

Доброго времени суток!
Только начинаю разбираться с тонкостями взаимодействия .NET и баз данных SQL (использую SQL server 2012). Так же использую Entity Framework. Возникла проблема с удалением выбранной строки из базы данных. Пишу приложение c использованием WinForm. Когда добавляю строку в базу данных проблем не возникает:

C#

int Id = 200;
            using (var context = new Database103SQLEntities())
            {
                foreach (var cw in context.Coworkers1)
                {
                    if (cw.Coworker_Id == '\0')
                        break;
                    else if (cw.Coworker_Id == Id)
                        Id++;
                }
 
                Coworkers cowork = new Coworkers//создаём строковое представление
                {
                    Coworker_Id = Id,
                    CoworkerSurname = textBox1.Text,
                    CoworkerName = textBox2.Text,
                    CoworkerFathername = textBox3.Text,
                    CoworkerPosition = textBox4.Text,
                    CoworkerStatus = textBox5.Text,
                    CoworkerLogin = textBox6.Text,
                    CoworkerPassword = textBox7.Text
                    
                };
                
                context.Coworkers1.Add(cowork);//добавляем строку в базу данных
                try
                {
                    context.SaveChanges();//сохраняем изменения
                    this.Close();
                }
                catch (Exception )
                {
                    MessageBox.Show("Ошибка добавления сотрудника", "Информация", MessageBoxButtons.OK);
                }
 
            }

А вот с удалением проблемы. Удаляю выбранную через листвью строку. Делаю аналогичным образом используя только метод .Remove:

C#

 int value=0;
            string str;
            ListView.SelectedIndexCollection collection = listView1.SelectedIndices;
            if (collection.Count != 0)
            {
                using (var context = new Database103SQLEntities())
                {
                    foreach (var DelItem in context.Coworkers1)
                    {
                        str = listView1.SelectedItems[0].SubItems[6].Text;
                        value = Convert.ToInt32(str);
                        if (DelItem.Coworker_Id == value)
                        {
                            Coworkers cowork1 = new Coworkers() //создаём строковое представление
                            {
                                Coworker_Id = DelItem.Coworker_Id,
                                CoworkerSurname = DelItem.CoworkerSurname,
                                CoworkerName = DelItem.CoworkerName,
                                CoworkerFathername = DelItem.CoworkerFathername,
                                CoworkerPosition = DelItem.CoworkerPosition,
                                CoworkerStatus = DelItem.CoworkerStatus,
                                CoworkerLogin = DelItem.CoworkerLogin,
                                CoworkerPassword = DelItem.CoworkerPassword
 
                            };
                            context.Coworkers1.Remove(cowork1);  //удаляем строку - компилятор ругается
                            //не удалось удалить объект, поскольку он не найден в ObjectStateManager
                            try
                            {
                                context.SaveChanges(); //сохраняем изменения
                                this.Close();
                            }
                            catch (Exception)
                            {
                                MessageBox.Show("Ошибка удаления сотрудника", "Информация", MessageBoxButtons.OK);
                            }
                            break;
                        }
                    }
                    
                    
                }
                listView1.Items.RemoveAt(collection[0]);

Вероятно я чего-то не понимаю... Перечитал несколько тем на msdn, но там не совсем то, что в моём случае. Заранее благодарен за помощь!

@Монфрид · 13.04.2014, 13:34

у вас велосипед на велосипеде в коде.
Начнём с того, что вы зачем то задаёте вручную Id. Этого не надо делать.
А удаление у вас вообще шедеврально.

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

C#

foreach (var DelItem in context.Coworkers1)
{
str = listView1.SelectedItems[0].SubItems[6].Text;
value = Convert.ToInt32(str);
if (DelItem.Coworker_Id == value)
{

не нужно никакие циклы использовать чтобы найти элемент по его Id.
У context'a куча методов для поиска объектов как по Id, так и по любому свойству другому (Where, Find, FirstOrDefault..).

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

C#

Coworkers cowork1 = new Coworkers() //создаём строковое представление//WTF??!!
 {
Coworker_Id = DelItem.Coworker_Id,
 CoworkerSurname = DelItem.CoworkerSurname,
CoworkerName = DelItem.CoworkerName,
 CoworkerFathername = DelItem.CoworkerFathername,
CoworkerPosition = DelItem.CoworkerPosition,
CoworkerStatus = DelItem.CoworkerStatus,
 CoworkerLogin = DelItem.CoworkerLogin,
 CoworkerPassword = DelItem.CoworkerPassword
};

Ну хорошо, нашли вы в своём цикле нужный объект в базе, а дальше зачем вы создаёте новый объект с такими же свойствами?
Потратьте пол часа на чтение какого нибудь туториала по EntityFramework, дабы не делать вот таких вот велосипедов.

В вашем случае, чтобы удалить объект, можно сделать так:

C#

            ListView.SelectedIndexCollection collection = listView1.SelectedIndices;
            if (collection.Count != 0)
            {
            int idValue=Convert.ToInt32(listView1.SelectedItems[0].SubItems[6].Text);
             using (var context = new Database103SQLEntities())
                {
                        Coworkers coworker=context.Coworkers1.Find(idValue);
                        if(coworker!=null)
                        {
                                 context.Coworkers1.Remove(coworker);
                                 context.SaveChanges();
                                 listView1.Items.RemoveAt(collection[0]);
                        }   
                 }
             }

@lukan2 · 13.04.2014, 13:48 **[ТС]**

Монфрид, спасибо! Всё очень чётко и рационально!
Выложил просто один из первых вариантов своего кода, т.к. не знал в чём именно причина ошибки. Был бы благодарен Вам, если бы ещё посоветовали хороший туториал по EF, т.к. в msdn отрывками читать очень неудобно.

@Монфрид · 13.04.2014, 14:05

lukan2, можно это почитать: http://professorweb.ru/my/LINQ... 4/14_1.php
вообще, всё в entityFramework вертится вокруг linq, так что не зная его, будет тяжело что-то делать.
Там вот в статье как раз делаются простые запросы к бд на linq (вставка, выборка, удаление)

@lukan2 0 / 0 / 0 Регистрация: 28.01.2010 Сообщений: 16
	13.04.2014, 13:48 [ТС]	3
	Монфрид, спасибо! Всё очень чётко и рационально! Выложил просто один из первых вариантов своего кода, т.к. не знал в чём именно причина ошибки. Был бы благодарен Вам, если бы ещё посоветовали хороший туториал по EF, т.к. в msdn отрывками читать очень неудобно. 0

@Монфрид 1245 / 1055 / 293 Регистрация: 07.03.2012 Сообщений: 3,245
	13.04.2014, 14:05	4
	lukan2, можно это почитать: http://professorweb.ru/my/LINQ... 4/14_1.php вообще, всё в entityFramework вертится вокруг linq, так что не зная его, будет тяжело что-то делать. Там вот в статье как раз делаются простые запросы к бд на linq (вставка, выборка, удаление) 1

Удаление строки из базы данных MS SQL Server 2012

Решение