Почему после выполнения INSERT отсутствуют данные в таблице?

Question

[Игорь]

Всем привет.
Есть форма, в ней пара полей для ввода и кнопка, по нажатию на которую нужно сохранить данные из полей для ввода. Сохранение реализуется методом addNewWord класса представленного ниже.
1) При нажатии на кнопку, код отрабатывает без ошибок, но итоге в таблице данные не появляются. Почему? При этом, если нажать на кнопку больше 1 раза, появляется ошибка: "...Cannot insert duplicate key in object ...". Т.е. получается, что после первого нажатия кнопки вставка была и в таблицу данные были добавлены. Но когда в Visual Studio открываю таблицу, там пусто.
2) Почему пояляется ошибка "Cannot insert duplicate key in object ", если поле ID (PRIMARY KEY) в таблице объвлено как IDENTITY ?

class DbWriter
    {
        private SqlConnection connection;
        private SqlCommand query;

        public DbWriter()
        {
            string sqlCon = @"Data Source=(LocalDB)\v11.0;AttachDbFilename=|DataDirectory|\database.mdf;Integrated Security=True;Connect Timeout=30";
            this.connection = new SqlConnection(sqlCon);
            this.connection.Open();

            this.query = new SqlCommand();
            query.Connection = this.connection;
            query.CommandType = System.Data.CommandType.Text;
        }
        
        public void addNewWord(string deutch_wort, string rissich_wort)
        {
            SqlTransaction safetransaction = connection.BeginTransaction();
            this.query.CommandText = "insert into wort (deutch, russich) values ('" + deutch_wort + "', '" + rissich_wort + "') SELECT SCOPE_IDENTITY()";
            this.query.Transaction = safetransaction;
            this.query.ExecuteNonQuery();
            safetransaction.Commit();
        }
    }

Comments

[Алексей Немиро]

Возможно файл базы перезаписывается после каждой перекомпиляции программы. Можно отключить в свойствах файла (выбрать файл в VS и на панели свойств указать, что с ним делать при построении).

В самой Visual Studio, вроде как должен быть быть не тот файл базы, который использовала программа (в папке bin).

SCOPE_IDENTITY в запросе не нужен, т.к. ExecuteNonQuery.

Данные в запрос лучше передавать параметрами:

this.query.CommandText = "insert into wort (deutch, russich) values (@deutch, @russich)";
this.query.Parameters.Add("@deutch", SqlDbType.NVarChar).Value = deutch_wort
this.query.Parameters.Add("@russich", SqlDbType.NVarChar).Value = rissich_wort

Про дубликаты, без схемы таблицы сложно сказать.

[Игорь]

Алексей Немиро:
Спасибо вам за ответ. Кое что для себя прояснил, но решить мою проблему это не помогло.
Да, вы правы, файл БД перезаписывался при компиляции. В свойствах файла mdf, как вы советовали, я указал чтобы он обновлялся только если новый:

Но это не помогло. Тогда я в студии в ServerExplorer добавил новое подключение к файлу mdf, но уже который создается при дебаге (путь у него: <путь к проекту>/bin/debug/). Но в этой базе таблица тоже пустая. Думал что в этом файле БД увижу то, что добавлял из программы во время ее работы.
На скриншоте вкладка Server Explorer (database.mdf - файл изначально добавленный к проекту, database.mdf1 - подключение которое я добавил отдельно, это файл из <путь к проекту>/bin/debug/):

на скриншоте я также постарался показать то, как я смотрю сожержимое таблицы - правый клик на имени таблицы, в контекстном меню пункт Show Table Data.

метод для записи в таблицы я изменил как вы советовали:
public void addNewWord(string deutch_wort, string rissich_wort)
{

SqlTransaction safetransaction = connection.BeginTransaction();
this.query.CommandText = "insert into dbo.wort (deutch, russich) values (@deutch, @russich)";
this.query.Parameters.Add("@deutch", System.Data.SqlDbType.NVarChar).Value = deutch_wort;
this.query.Parameters.Add("@russich", System.Data.SqlDbType.NVarChar).Value = rissich_wort;
safetransaction.Commit();
}

Остальное осталось без изменений.

[Игорь]

Алексей Немиро:

Разобрался. Алексей, спасибо! Ваши советы помогли, направили в нужную сторону. Но вы ответили мне в комментарии к моему посту, поэтому я не могу ответить ваш ответ как Решение.

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

public void addNewWord(string deutch_wort, string rissich_wort)
{

query.Transaction = connection.BeginTransaction();
this.query.CommandText = "insert into dbo.wort (deutch, russich) values (@deutch, @russich)";
this.query.Parameters.Add("@deutch", System.Data.SqlDbType.NVarChar).Value = deutch_wort;
this.query.Parameters.Add("@russich", System.Data.SqlDbType.NVarChar).Value = rissich_wort;
query.ExecuteNonQuery();
query.Transaction.Commit();
}

Answer 1

[Oxoron]

Видимо, у вас поля deutch, russich тоже должны быть уникальными. Плюс, что значит, "открываю таблицу через MS". Там по-разному можно извратиться, в т.ч. увидеть лишь структуру.

Вообще, если БД небольшая и несложная, рекомендую поставить EF или NHibernate. Простейшая установка и изучение займет пару-тройку часов, потом жить станет сильно проще.

Comments

[Станислав Макаров]

Соглашусь, что с ORM-кой проще, но если это в учебных целях, то пусть лучше сначала сам напишет пару INSERT-ов. Мапперы добавляют уровень абстракции, и если чтото идет не так, нужно понимать что происходит под капотом...