Ist es gute Praxis, 'Goto' Anweisungen in SQL-Abfragen zu verwenden?Sql - goto Anweisung
Antwort
Hängt vom SQL ab - einige der Dialekte bieten keinen nützlichen Mechanismus zur Flusskontrolle außer GOTO.
GOTO ist in der Regel schlechte Form.
Meine Vermutung wäre nein. Meine allgemeine Regel mit goto Anweisungen in jeder modernen Sprache ist, wenn Sie sie verwenden, stimmt etwas mit Ihrem Design nicht.
Nr
Wie bei anderen Sprachen, gibt es fast immer eine bessere Option als ein Springen zu verwenden.
Wenn Sie uns mitteilen, welches SQL-Paket Sie verwenden und was Sie erreichen möchten, können wir Ihnen vielleicht eine Idee geben, was genau passt.
Es ist eine gespeicherte Prozedur von jemandem geschrieben. – SoftwareGeek
@BhejaFry - Aber ist es T-SQL, PL-SQL, MySQL usw.? –
es ist T-SQL, wird nicht pl-sql Unterstützung goto? – SoftwareGeek
Nicht im Produktionscode, aber zum Testen könnte in Ordnung sein.
Zum Beispiel Regressionstests für eine gespeicherte Prozedur bereitstellen, wobei das "Common Bit" der Aufruf des zu testenden Verfahrens und Debug-Anweisungen ist.
declare @test int;
set @test = 1;
goto tests
common:
print "common bit"
tests:
if @test = 1 print "1";
if @test = 2 print "2";
if @test = 3 print "3";
set @test = @test + 1;
if @test <= 3 goto common
print "finished " + cast(@test as varchar(5))
go -- goto can not be used past go!
Als T-SQL-Noob Ich war für Prozedur oder Funktion der Hoffnung, im Rahmen zu erklären, die „gemeinsame Bit“ zu tun, aber das war die beste, das ich mit nach vielen googeln kommen konnte. Warum sollten Sie für jedes Codebeispiel, das Sie erneut verwenden möchten, eine gespeicherte Prozedur einrichten? Speziell für Nicht-Produktionsarbeiten.
Sie machen einen großen Punkt über die Verwendung kleiner Funktionen mit GOTO. :-) Wir brauchen Lambda-Ausdrücke in T-SQL! ;-) –
goto ist ein Keyword mit einer eigenen Funktion. Wir können Goto verwenden, wann immer es erforderlich ist, direkt zu einem bestimmten Level zu springen.
Nehmen wir das Beispiel ... In meiner gespeicherten Prozedur muss ich mit Daten arbeiten, die 4 temporäre Tabellen aufnehmen. auf jeder Ebene nach dem Einfügen von Datensätzen in der temporären Tabelle Ich muss überprüfen, ob Datensätze in diesem Temp-Temp vorhanden sind oder nicht, wenn keine Datensätze eingefügt wurden, so kann ich anstatt weiter zu verarbeiten direkt mit goto springen. Etikett ist der Punkt, wo wir springen sollte:
CREATE TABLE #tmpMsNos (custPo CHAR(24))
CREATE TABLE #tmpValidBilltos (billto CHAR(12))
CREATE TABLE #tmpOrders (
fh_pkey INT
,fh_id CHAR(8)
,custPo CHAR(24)
,lastchOfCustInsert DATETIME
)
CREATE TABLE #tmpOrdersFiltered (
fh_pkey INT
,fh_id CHAR(8)
,custPo CHAR(24)
,lastchOfCustInsert DATETIME
,onbDate DATETIME
,rapDate DATETIME
)
CREATE TABLE #tmpLoad (
custPo CHAR(24)
,ld_pkey INT
,ld_wkpmpn CHAR(25)
,lda_barcode VARCHAR(30)
,ld_createdOn DATETIME
,ReceivedDate DATETIME
,DispatchedDate DATETIME
)
INSERT INTO #tmpMsNos
SELECT cast(itemValue AS CHAR(24))
FROM dbo.fn_array_to_table(@pMsNos, ',')
IF (
NOT EXISTS (
SELECT 1
FROM #tmpMsNos
)
)
BEGIN
GOTO label
END
INSERT INTO #tmpValidBilltos
SELECT CONVERT(CHAR(12), xm_doref)
FROM xmlref x
WHERE xm_element = 'THD-BoxtruckRequest'
AND xm_attribute = 'THD-BoxtruckBillto'
IF (
NOT EXISTS (
SELECT 1
FROM #tmpValidBilltos
)
)
BEGIN
GOTO label
END
INSERT INTO #tmpOrders
SELECT fh.fh_pkey
,fh.fh_id
,fh.fh_custPo
,max(coc.ch_dt)
FROM #tmpMsNos msNos
INNER JOIN fcfgthd fh ON msNos.custPo = fh.fh_custPo
INNER JOIN #tmpValidBilltos bt ON bt.billto = fh.fh_bt_id
LEFT JOIN chofcust coc ON coc.ch_fhpkey = fh.fh_pkey
WHERE fh.fh_statcode NOT IN (
98 --CAN
,99 --DEL
)
AND fh.fh_ship_dt > @startDate
GROUP BY fh.fh_pkey
,fh.fh_id
,fh.fh_custPo
IF (
NOT EXISTS (
SELECT 1
FROM #tmpOrders
)
)
BEGIN
GOTO label
END
INSERT INTO #tmpOrdersFiltered
SELECT t.fh_pkey
,t.fh_id
,t.custPo
,t.lastchOfCustInsert
,MAX(cocONB.ch_dt)
,MAX(cocRAP.ch_dt)
FROM (
SELECT tmpO.fh_pkey
,tmpo.fh_id
,tmpO.custPo
,tmpO.lastchOfCustInsert
FROM #tmpOrders tmpO
INNER JOIN (
SELECT custpo
,max(lastchOfCustInsert) AS MaxInserteddate
FROM #tmpOrders
GROUP BY custpo
) tmpOgrouped ON tmpO.custpo = tmpOgrouped.custpo
AND tmpO.lastchOfCustInsert = tmpOgrouped.MaxInserteddate
) AS t
LEFT JOIN chofcust cocRAP ON cocRAP.ch_fhpkey = t.fh_pkey
AND cocRAP.ch_stat = 2 -- RAP --TODO: Add comment with status code like 98, 99 -- CAN, DEL for readability - Paresh
LEFT JOIN chofcust cocONB ON cocONB.ch_fhpkey = t.fh_pkey
AND cocONB.ch_stat = 5 -- ONB --TODO: Add comment with status code like 98, 99 -- CAN, DEL for readability - Paresh
GROUP BY t.fh_pkey
,t.fh_id
,t.custPo
,t.lastchOfCustInsert
--TODO: Take an exit if no order found into #tmpOrdersFiltered table, while taking a early exit make sure it doesn't break the calling code (C#) - Paresh
IF (
NOT EXISTS (
SELECT 1
FROM #tmpOrdersFiltered
)
)
BEGIN
GOTO label
END
INSERT INTO #tmpLoad
SELECT o.custPo
,l.ld_pkey
,l.ld_wkpmpn
,la.lda_barcode
,max(coc.ch_dt)
,CASE ISNULL(w.xl_date, '')
WHEN ''
THEN o.rapDate
ELSE w.xl_date
END AS ReceivedDate
,CASE ISNULL(mm.me_ecpkey, '')
WHEN ''
THEN o.ONBDate
ELSE NULL
END AS DispatchedDate
FROM #tmpOrdersFiltered o
INNER JOIN fcload l ON l.ld_fhpkey = o.fh_pkey
LEFT JOIN loadanc la ON la.lda_ldpkey = l.ld_pkey
LEFT JOIN wkxaclog w ON w.xl_ldpkey = l.ld_pkey
LEFT JOIN multiexceps mm ON mm.me_ldpkey = l.ld_pkey
AND mm.me_ecpkey = @missingitemexcep
LEFT JOIN chofcust COC ON coc.ch_ldpkey = l.ld_pkey
AND coc.ch_stat = 64 -- 64= ILH
GROUP BY o.custPo
,l.ld_pkey
,l.ld_wkpmpn
,la.lda_barcode
,w.xl_date
,o.rapDate
,mm.me_ecpkey
,o.ONBDate
http://xkcd.com/292/ – ircmaxell
@ircmaxell meine Gefühle genau. – Meiscooldude
Es gibt kein 'GOTO' in SQL - nur T-SQL –