Question

我正在处理一些练习题，我遇到了这个问题，我在尝试执行说明

的过程时遇到错误

第15行，第1行，第1行，第1行关键字“过程”附近的语法不正确。“

有人可以帮忙吗？

编写一个接受所有UpdateTitle表列的过程Title，并使用这些值更新标题。提出以下错误消息：ISBN不存在类别和/或发布者代码无效。

Create PROCEDURE UpdateTitle (@ISBN char(10), @SuggestedPrice smallmoney,@NumberInStock smallint,@PublisherCode int,@CategoryCode int)
AS
BEGIN
IF @ISBN is null or @CategoryCode is null or @PublisherCode is null
BEGIN
 RAISERROR ('ISBN,CategoryCode, or PublisherCode is not valid please enter valid data',16,1)
END
ELSE
BEGIN    
 IF (SELECT COUNT(*) FROM Title WHERE ISBN = @ISBN) = 0
 BEGIN
    RAISERROR ('ISBN does not exist.',16,1)
 END
 ELSE
 BEGIN
    SELECT 'Table Sucessfully Updated.';
    UPDATE Title 
    SET SuggestedPrice = @SuggestedPrice
    WHERE ISBN = @ISBN;

    BEGIN
     IF (SELECT COUNT(*) FROM Title WHERE ISBN = @ISBN) = 0
     BEGIN
    RAISERROR ('ISBN does not exist.',16,1)
 END
 ELSE
 BEGIN
    SELECT 'Table Sucessfully Updated.';
    UPDATE Title 
    SET NumberInStock = @NumberInStock
    WHERE ISBN = @ISBN;
  END

          BEGIN
     IF (SELECT COUNT(*) FROM Title WHERE ISBN = @ISBN) = 0
     BEGIN
    RAISERROR ('ISBN does not exist.',16,1)
 END
 ELSE
 BEGIN
    SELECT 'Table Sucessfully Updated.';
    UPDATE Title 
    SET PublisherCode = @PublisherCode
    WHERE ISBN = @ISBN;
  END
          BEGIN
     IF (SELECT COUNT(*) FROM Title WHERE ISBN = @ISBN) = 0
     BEGIN
    RAISERROR ('ISBN does not exist.',16,1)
 END
 ELSE
 BEGIN
    SELECT 'Table Sucessfully Updated.';
    UPDATE Title 
    SET CategoryCode = @CategoryCode
    WHERE ISBN = @ISBN;
  END

 END
END
END
END
END
END
GO

然后

 Execute Procedure UpdateTitle @ISBN ='1021031040', @suggestedproce ='40' , @NumberInStock ='10', @PublisherCode = '200', @CategoryCode = '1'

Answer 1

Execute Procedure UpdateTitle ...

应该是：

EXEC dbo.UpdateTitle ...

其他一些评论：

ISBN不再限于10个字符（this change happened in 2007, if you believe WikiPedia）。
Always use the schema prefix when creating or referencing objects。
您只需要检查ISBN是否有效一次。并且你不应该使用计数恕我直言，特别是因为 - 大概 - 这是关键，它无论如何也只能返回0或1。
您不应选择“更新成功”，然后执行更新。在告诉用户成功之前，您应该确保更新成功。
没有理由将其分成多个更新。
请自由回车，缩进和空白。可读性的价值是值得输入的额外成本（因为你只输入一次，但你会多次读它）。
使用RETURN;作为退出机制，这样您就不必多次嵌套IF和ELSE。
始终在程序开始时使用SET NOCOUNT ON;。
您可能希望自定义消息以告知用户哪些参数无效。
哦是的，please future-proof your code by using semi-colons to terminate statements。

这是一个更简洁的版本，可以满足您的所有要求：

CREATE PROCEDURE dbo.UpdateTitle
  @ISBN           CHAR(10), 
  @SuggestedPrice SMALLMONEY,
  @NumberInStock  SMALLINT,
  @PublisherCode  INT,
  @CategoryCode   INT
AS
BEGIN
  SET NOCOUNT ON;

  DECLARE @msg VARCHAR(255);

  IF @ISBN IS NULL OR @CategoryCode IS NULL OR @PublisherCode IS NULL
  BEGIN
    SELECT @msg = 'The following parameter(s) were invalid:'
      + CASE WHEN @ISBN IS NULL THEN ' @ISBN' ELSE '' END 
      + CASE WHEN @CategoryCode IS NULL THEN ' @CategoryCode' ELSE '' END 
      + CASE WHEN @PublisherCode IS NULL THEN ' @PublisherCode' ELSE '' END;

    RAISERROR (@msg, 11, 1);
    RETURN;
  END

  IF NOT EXISTS (SELECT 1 FROM dbo.Title WHERE ISBN = @ISBN)
  BEGIN
    SET @msg = 'ISBN %s does not exist.';
    RAISERROR(@msg, 11, 1, @ISBN);
    RETURN;
  END

  BEGIN TRY
    UPDATE dbo.Title
      SET SuggestedPrice = @SuggestedPrice,
          NumberInStock  = @NumberInStock,
          PublisherCode  = @PublisherCode,
          CategoryCode   = @CategoryCode
      WHERE ISBN = @ISBN;

    SELECT 'Update successful.';
  END TRY
  BEGIN CATCH
    SET @msg = ERROR_MESSAGE();
    RAISERROR(@msg, 11, 1);
  END CATCH
END
GO

存储过程接受所有表字段并更新这些值

1 个答案: