Holen Sie die ID der eingefügten Zeile mit C#

Question

Ich habe eine Abfrage zum Einfügen einer Zeile in eine Tabelle, die ein Feld namens ID hat, die mit einem AUTO_INCREMENT für die Spalte aufgefüllt wird. Ich brauche diesen Wert für die nächste Bit-Funktionalität zu erhalten, aber wenn ich folgendes ausführen, es gibt immer 0, obwohl der tatsächliche Wert nicht 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()); 

Nach meinem Verständnis sollte dies die ID zurückkehren Spalte, aber es gibt nur 0 jedes Mal zurück. Irgendwelche Ideen?

EDIT:

Wenn ich laufen:

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

ich:

{"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"} 

Answer 1

[Edit: added "wählen Sie" vor Referenzen() LAST_INSERT_ID]

Was ist mit dem Laufen "select last_insert_id();" nach Ihrer Eingabe t?

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

---

Edit: Wie duffymo erwähnt, würden Sie wirklich gut like this mit parametrisierte Abfragen bedient werden.

---

Edit: Bis Sie eine parametrisierte Version wechseln, Sie Frieden mit string.Format finden könnten:

comm.CommandText = string.Format("{0} '{1}', {2}, {3}, {4}, {5}); select last_insert_id();", 
    insertInvoice, invoiceDate.ToString(...), bookFee, adminFee, totalFee, customerID); 

Answer 2

Es stört mich jemand zu sehen ein Date zu nehmen und es in einem Speicher Datenbank als String. Warum sollte der Säulentyp nicht die Realität widerspiegeln?

Ich bin auch überrascht zu sehen, dass eine SQL-Abfrage mit String-Verkettung aufgebaut wird. Ich bin ein Java-Entwickler und ich kenne C# überhaupt nicht, aber ich würde mich fragen, ob es irgendwo in der Bibliothek einen Bindemechanismus in der Art von java.sql.PreparedStatement gäbe. Es wird empfohlen, SQL-Injection-Angriffen vorzubeugen. Ein weiterer Vorteil sind mögliche Leistungsvorteile, da das SQL-Protokoll analysiert, verifiziert, einmal zwischengespeichert und wiederverwendet werden kann.

Answer 3

Tatsächlich gibt die ExecuteScalar-Methode die erste Spalte der ersten Zeile des zurückgegebenen DataSets zurück. In Ihrem Fall erstellen Sie nur einen Insert, Sie fragen tatsächlich keine Daten ab. Sie müssen die scope_identity() nach dem Einfügen abfragen (das ist die Syntax für SQL Server) und dann haben Sie Ihre Antwort. Siehe hier:

Linkage

EDIT: Als Michael Haren wies darauf hin, die Sie erwähnt in Ihrem Tag Sie verwenden MySQL, verwenden LAST_INSERT_ID(); anstelle von scope_identity();

Answer 4

Verwenden Sie LastInsertedId.

Meine Anregung mit Beispiel hier: http://livshitz.wordpress.com/2011/10/28/returning-last-inserted-id-in-c-using-mysql-db-provider/

Answer 5

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