Получите идентификатор вставленной строки, используя С#
-
03-07-2019 - |
Вопрос
У меня есть запрос на вставку строки в таблицу, в которой есть поле с именем ID, которое заполняется с использованием AUTO_INCREMENT в столбце.Мне нужно получить это значение для следующего фрагмента функциональности, но когда я запускаю следующее, оно всегда возвращает 0, даже если фактическое значение не равно 0:
MySqlCommand comm = connect.CreateCommand();
comm.CommandText = insertInvoice;
comm.CommandText += "\'" + invoiceDate.ToString("yyyy:MM:dd hh:mm:ss") + "\', " + bookFee + ", " + adminFee + ", " + totalFee + ", " + customerID + ")";
int id = Convert.ToInt32(comm.ExecuteScalar());
Насколько я понимаю, это должно возвращать столбец ID, но каждый раз возвращает 0.Есть идеи?
РЕДАКТИРОВАТЬ:
Когда я бегу:
"INSERT INTO INVOICE (INVOICE_DATE, BOOK_FEE, ADMIN_FEE, TOTAL_FEE, CUSTOMER_ID) VALUES ('2009:01:01 10:21:12', 50, 7, 57, 2134);last_insert_id();"
Я получил:
{"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'last_insert_id()' at line 1"}
Решение
[Редактировать:добавлено «выбрать» перед ссылками на Last_insert_id()]
А как насчет бега»select last_insert_id();
"после твоей вставки?
MySqlCommand comm = connect.CreateCommand();
comm.CommandText = insertInvoice;
comm.CommandText += "\'" + invoiceDate.ToString("yyyy:MM:dd hh:mm:ss") + "\', "
+ bookFee + ", " + adminFee + ", " + totalFee + ", " + customerID + ");";
+ "select last_insert_id();"
int id = Convert.ToInt32(comm.ExecuteScalar());
Редактировать: Как упомянул Даффимо, вам действительно будет полезно использовать параметризованные запросы. так.
Редактировать: Пока вы не переключитесь на параметризованную версию, вы можете найти покой с помощью string.Format:
comm.CommandText = string.Format("{0} '{1}', {2}, {3}, {4}, {5}); select last_insert_id();",
insertInvoice, invoiceDate.ToString(...), bookFee, adminFee, totalFee, customerID);
Другие советы
MySqlCommand comm = connect.CreateCommand();
comm.CommandText = insertStatement; // Set the insert statement
comm.ExecuteNonQuery(); // Execute the command
long id = comm.LastInsertedId; // Get the ID of the inserted item
Используйте ЛастИнсертедИд.
Посмотрите мое предложение с примером здесь: http://livshitz.wordpress.com/2011/10/28/returning-last-inserted-id-in-c-using-mysql-db-provider/
Меня беспокоит то, что кто-то берет дату и сохраняет ее в базе данных как строку.Почему бы не сделать так, чтобы тип столбца отражал реальность?
Я также удивлен, увидев, что SQL-запрос строится с использованием конкатенации строк.Я разработчик Java и совсем не знаю C#, но мне интересно, нет ли где-нибудь в библиотеке механизма привязки наподобие java.sql.PreparedStatement?Рекомендуется для защиты от атак SQL-инъекций.Еще одним преимуществом является возможный выигрыш в производительности, поскольку SQL можно анализировать, проверять, кэшировать один раз и использовать повторно.
Фактически метод ExecuteScalar возвращает первый столбец первой строки возвращаемого набора данных.В вашем случае вы выполняете только вставку, а не запрашиваете какие-либо данные.Вам нужно запроситьscope_identity() после вставки (это синтаксис SQL Server), и тогда вы получите ответ.Глянь сюда:
РЕДАКТИРОВАТЬ: Как отметил Майкл Харен, вы упомянули в своем теге, что используете MySql, используйте Last_insert_id();вместоscope_identity();