2010-06-15 4 views

Antwort

8

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.

0

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.

3

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.

+0

Es ist eine gespeicherte Prozedur von jemandem geschrieben. – SoftwareGeek

+0

@BhejaFry - Aber ist es T-SQL, PL-SQL, MySQL usw.? –

+0

es ist T-SQL, wird nicht pl-sql Unterstützung goto? – SoftwareGeek

5

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.

+0

Sie machen einen großen Punkt über die Verwendung kleiner Funktionen mit GOTO. :-) Wir brauchen Lambda-Ausdrücke in T-SQL! ;-) –

0

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