Получите идентификатор вставленной строки, используя С#

StackOverflow https://stackoverflow.com/questions/405910

  •  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();

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top