Est-ce une bonne pratique d'utiliser les instructions 'goto' dans les requêtes SQL?Sql - instruction goto
Répondre
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.
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.
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.
C'est une procédure stockée écrite par quelqu'un. – SoftwareGeek
@BhejaFry - Mais est-ce T-SQL, PL-SQL, MySQL, etc.? –
c'est T-SQL, le support de pl-sql ne va-t-il pas? – SoftwareGeek
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.
Vous faites un excellent point sur l'utilisation de petites fonctionnalités en utilisant GOTO. :-) Nous avons besoin d'expressions Lambda dans T-SQL! ;-) –
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
http://xkcd.com/292/ – ircmaxell
@ircmaxell mes sentiments exactement. – Meiscooldude
Il n'y a pas de 'GOTO' dans SQL - seulement T-SQL –