2010-06-15 17 views

Répondre

8

Dépend du SQL - certains des dialectes ne fournissent pas de mécanisme utile pour le contrôle de flux autre que GOTO.

GOTO est généralement de mauvaise forme.

0

Ma conjecture serait non. Ma règle générale avec les instructions goto dans n'importe quelle langue moderne est, si vous les utilisez, quelque chose ne va pas avec votre conception.

3

n °

Comme avec d'autres langues, il y a presque toujours une meilleure option que d'utiliser un Goto.

Si vous nous indiquez le package SQL que vous utilisez et ce que vous essayez d'accomplir, nous pourrions vous donner une idée de ce qui pourrait convenir.

+0

C'est une procédure stockée écrite par quelqu'un. – SoftwareGeek

+0

@BhejaFry - Mais est-ce T-SQL, PL-SQL, MySQL, etc.? –

+0

c'est T-SQL, le support de pl-sql ne va-t-il pas? – SoftwareGeek

5

Pas dans le code de production, mais pour les tests pourrait être ok. Par exemple, vouloir fournir un test de régression pour une procédure stockée où le "bit commun" est l'appel à la procédure en cours de test et les instructions de débogage.

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! 

Être t-sql Noob J'espérais procédure ou une fonction de déclarer dans la portée de faire le « peu commun » mais c'était le meilleur que je pouvais venir avec beaucoup après googler. Pourquoi auriez-vous besoin de configurer une procédure stockée pour chaque bit de code que vous souhaitez réutiliser. Surtout pour le travail de non-production.

+0

Vous faites un excellent point sur l'utilisation de petites fonctionnalités en utilisant GOTO. :-) Nous avons besoin d'expressions Lambda dans T-SQL! ;-) –

0

goto est un mot-clé avec sa propre fonctionnalité. Nous pouvons utiliser goto chaque fois qu'il est nécessaire de passer directement à un certain niveau.

Prenons l'exemple ... Dans ma procédure stockée, je dois travailler avec la prise de données dans 4 tables temporaires. à tous les niveaux après avoir inséré des enregistrements dans la table temporaire J'ai besoin de vérifier si les enregistrements sont présents dans cette temp temp, si aucun enregistrement n'a été inséré, donc je peux directement sauter en utilisant goto. étiquette est le point où nous devrions sauter:

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