Muss ich nach dem Erstellen einer neuen Zeile in der Datenbank den Primärschlüssel an die Zeile oder ein Datenübertragungsobjekt zurückgeben?

Question

Ich habe ein dreistufiges System, SQL Server-Backend, handgeschriebene Datenzugriffsschicht und gespeicherte Prozeduren verwenden.

Ich habe eine Tabelle namens EventTable. Jede Zeile ist ein 'Event'. Ein Ereignis hat einen Primärschlüssel und ein Startdatum.

CREATE TABLE EventTable 
(
    ID INT IDENTITY(100,1) PRIMARY KEY, 
    StartTime DateTime NOT NULL 
) 

Es gibt eine gespeicherte Prozedur namens EventTable_Create. Übrigens, ist die create Methode in Ordnung wie geschrieben?

CREATE PROCEDURE Event_Create 
    @NewID INT OUT 
AS 
    DECLARE @START DATETIME 
    SELECT @START = getdate() 
    INSERT INTO EventTable VALUES(@START, NULL) 
    SELECT @NewID = MAX(ID) FROM EventTable 
GO 

Die Datenzugriffsebene gibt ein int an den Aufrufer zurück, sollte sie stattdessen eine Instanz eines Datenübertragungsobjekts namens Event zurückgeben? Wenn das wahr ist, sollte ich sowohl die neu erstellte ID als auch die Startzeit zurückgeben, damit die Datenzugriffsebene das Ereignisübertragungsobjekt erstellen kann?

Answer 1

Es hängt alles davon ab, was Sie in Ihrer Domain-Schicht über den Datenzugriff benötigen. Da Sie dies durch Übergeben der Informationen von der Domänenebene erstellen, nehme ich an, dass Sie die meisten Ereignisinformationen bereits in Ihrer Domänenlogik haben. In diesem Fall benötigen Sie lediglich eine ID, um sie als Teil Ihres Ereignisobjekts zu füllen, falls Sie eines haben.

Es hängt also alles davon ab, was Sie in Ihrer Domainebene benötigen und aus welchen Gründen.

Answer 2

Soweit was Sie benötigen, um zurückzukehren, würde jeder Weg wahrscheinlich funktionieren. Wie CodeToGlory sagte, kommt es darauf an. Ich würde mich darauf konzentrieren, beide Werte selbst zurückzugeben.

Ich würde ändern, wie ich die Identitätsspalte in der gespeicherten Prozedur abrufen. Es gibt ein paar Möglichkeiten, um es zu bekommen.

Statt: SELECT @NewID = MAX (ID) FROM EXPERIMENTTABLE

ich tun würde: Set @NewID = @@ Identität

oder

Set @NewID = SCOPE_IDENTITY()

Hier ist ein Auszug aus Büchern online mit einem Beispiel. Ich kann Ihnen helfen zu entscheiden, welche zu verwenden:

Zum Beispiel gibt es zwei Tabellen, T1 und T2, und ein INSERT-Trigger ist auf T1 definiert. Wenn eine Zeile in T1 eingefügt wird, wird der Trigger ausgelöst und eine Zeile in T2 eingefügt. In diesem Szenario werden zwei Bereiche veranschaulicht: der Einschub in T1 und der Einschub in T2 durch den Auslöser.

Unter der Annahme, dass sowohl T1 als auch T2 Identitätsspalten haben, geben @@ IDENTITY und SCOPE_IDENTITY unterschiedliche Werte am Ende einer INSERT-Anweisung in T1 zurück. @@ IDENTITY gibt den letzten Wert der Identitätsspalte zurück, der über einen Bereich in der aktuellen Sitzung eingefügt wurde. Dies ist der in T2 eingefügte Wert. SCOPE_IDENTITY() gibt den in T1 eingefügten IDENTITY-Wert zurück. Dies war die letzte Einfügung im selben Bereich. Die Funktion SCOPE_IDENTITY() gibt den Nullwert zurück, wenn die Funktion aufgerufen wird, bevor INSERT-Anweisungen in einer Identitätsspalte im Gültigkeitsbereich auftreten.

Answer 3

Sie sollten MAX (ID) nicht verwenden, um den Primärschlüssel des neu hinzugefügten Datensatzes abzurufen.

Verwenden Sie stattdessen Identität.

INSERT INTO tbl(...) VALUES(...); 

SELECT @@Identity as NewID; 

Answer 4

Sie sollten den einzigen Primärschlüssel zurückgeben, wenn Sie nicht die gesamte Datenstruktur benötigen. Das Zurücksetzen der gesamten Datenstruktur ohne einen bestimmten Grund ist verschwenderisch.