多批处理事务 sql 脚本中的错误

Question

  BEGIN TRAN
  SET XACT_ABORT ON

  GO

  BEGIN TRY
   IF OBJECT_ID('dbo.Offer_GetByStudyId', 'p') IS NULL
  EXEC ('CREATE PROCEDURE Offer_GetByStudyId as select 1')
  END TRY

 BEGIN CATCH
 THROW;
  END CATCH

 GO
 IF @@error <> 0 and @@trancount > 0 ROLLBACK
 IF @@trancount = 0 BEGIN SET NOCOUNT ON; SET NOEXEC ON; END
 GO

  BEGIN TRY

  ALTER PROCEDURE dbo.Offer_GetByStudyId
  @StudyId NVARCHAR(MAX) = NULL
  AS
 BEGIN
  DECLARE @Conditions NVARCHAR(MAX) = '';

IF @StudyId IS NOT NULL
BEGIN
    SET @Conditions = @Conditions + ' AND o.StudyId = ' + cast(@StudyId as varchar(10))
END

DECLARE @sql NVARCHAR(MAX) = 'SELECT
    o.StudyId as StudyId,
    o.SampleId as SampleId,
    o.Status as Status,
    o.Title as Title,
    o.Topic as Topic,
    o.Description as Description,
    o.TestOffer as TestOffer,
    T.CPI as CPI
    FROM Offers o
    LEFT JOIN [dbo].[Terms] T ON (o.[Id] = T.[OfferId]) AND T.Active = 1
    WHERE 1 = 1' + @Conditions

    EXEC(@sql)
END
END TRY
BEGIN CATCH
    THROW;
END CATCH

这是我的 SQL 脚本，我试图让一个多批处理脚本作为单个事务运行，所以如果一个语句失败，所有语句都将回滚。但是我在这里不断收到此错误：

BEGIN 附近的语法不正确。期待外部

他们所说的开始是后面的：

@StudyId NVARCHAR(MAX) = NULL 

Answer 1

您需要删除“GO”语句。这些不是 TSQL 语言的一部分，它们只是告诉 SSMS/SQLCMD 上面的批处理应该执行的语句。我不清楚跨多个 GO 语句拆分事务的行为。我将开始删除“GO”。

https://msdn.microsoft.com/en-us/library/ms188037.aspx

Answer 2

如果您从 TRY CATCH 中取出声明 ALTER PROCEDURE dbo.Offer_GetByStudyId (Line 22 - 47)，它将起作用。
即删除TRY CATCH (Line 20 and Line 48 - 51)

Answer 3

我意识到我已经把答案埋在下面了，所以我会把它放到顶部以使其更清楚：你不能包装任何流控制语句来尝试创建或更改过程

---

一个更简单的例子演示了这个问题：

create procedure ABC
as
go
select * from sys.objects
alter procedure ABC as

哪个产生了消息：

Msg 111, Level 15, State 1, Procedure ABC, Line 2
'CREATE/ALTER PROCEDURE' must be the first statement in a query batch.

这是为CREATE PROCEDURE 明确记录的，但由于某种原因不适用于ALTER PROCEDURE。

CREATE PROCEDURE 语句不能与单个批处理中的其他 Transact-SQL 语句组合。

因此结果是，您不能将任何流控制语句包装在尝试创建或更改过程的过程中。

---

原因其实很简单 - BEGIN 和 END 不是必需 围绕存储过程的主体 - 而且一个存储过程实际上可以包含多个“顶级”@ 987654330@/END对：

create procedure ABC as
begin 
    select * from sys.objects
end
begin 
    select * from sys.columns
end

很好 - 所以 SQL Server 在定义存储过程时知道其范围的唯一方法是“从CREATE/ALTERPROCEDURE 直到批处理结束。”