关键字“And”存储过程附近的语法不正确答案

【问题标题】：Incorrect syntax near the keyword 'And' stored procedure关键字“And”存储过程附近的语法不正确
【发布时间】：2019-01-03 17:31:59
【问题描述】：

USE [productDb]
GO

SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO

ALTER PROCEDURE [dbo].[usp_ProcName]
    @Name VARCHAR(100),
    @PDate NVARCHAR(MAX) NULL,
    @City VARCHAR(100),
    @Birthday VARCHAR(100)
AS 
BEGIN
    IF (@Name = '')
       IF @Birthday = -1
       BEGIN
           SET @PDate = And ProductDate >= '08/04/1883'
       END
       ELSE 
       BEGIN
           SET @PDate = And ProductDate <= '08/04/1883'
       END
   ELSE
   BEGIN
       SELECT * FROM Product
   END
END

我收到以下错误：

消息 156，级别 15，状态 1，过程 usp_ProcName，第 18 行
关键字“And”附近的语法不正确。

消息 156，级别 15，状态 1，过程 usp_ProcName，第 22 行
关键字“And”附近的语法不正确。

【问题讨论】：

提示：@PDate 是一个字符串，所以你需要字符串操作符。
@PetroK 太奇怪了，完全无效。但是这里的代码总体来说有很大的问题。
@PDate 将包含一个字符串。不是引擎将这样处理的完整 SQL 查询的任意片段。没有办法存储 SQL 语句的片段，也没有办法将这些片段“组合”成整个语句。（除了在整个过程中使用字符串并强制一个单独的解析/编译/执行阶段，我尽量避免）。
你应该问一个关于如何实现你的最终目标的问题。你目前的方法偏离了轨道
这里唯一有意义的情况是 @PDate 是一个完整的 SQL 查询，并且您在执行它时试图在查询的末尾添加一个条件值。这不是有效的 T-SQL。相反，您可以使用SET @PDate = @PDate + ' AND ProductDate >= ''08/04/1883'''。注意：您仍然需要使用EXEC (@PDate) 单独执行查询。如果@PDate 不是 SQL 查询字符串，则可以忽略此答案。

标签： sql sql-server

【解决方案1】：

你的存储过程有两个问题：

不正确地使用SET 运算符。这是一个错误
正如 Sean Lange 所说，存在从 datetime 到 varchar(max) 的隐式转换。这种方法允许您在需要时更改格式，或进行日期数学运算。

让我们更准确地看一下第一个错误： SET 运算符用于分配数据。你在SET写了错误的语句：

set @PDate = And ProductDate >= '08/04/1883'
    End
    Else
    Begin
        set @PDate = And ProductDate <= '08/04/1883'

您应该在 SET 运算符中设置数据。例如：

Begin
    set @PDate = GETDATE()
End
Else
    Begin
        set @PDate = GETDATE()
    end

第二个问题是从datetime 到varchar(max) 的隐式转换。它可以通过使用正确的数据类型来修复。因此，这种方法允许您在需要时更改格式，或进行日期数学：

ALTER PROCEDURE [dbo].[usp_ProcName]
 @Name          VARCHAR(100) 
, @PDate         DateTime = NULL
, @City          VARCHAR(100) 
, @Birthday      VARCHAR(100) 
AS 
IF (@Name='')
If @Birthday=-1
Begin
    set @PDate = GETDATE()
End
Else
    Begin
        set @PDate = GETDATE()
    end

【讨论】：

但 @PDate 是 varchar(max) 不是日期时间数据类型。从 datetime 到 varchar(max) 的隐式转换极不可能是 OP 真正想要的。
@SeanLange 感谢您的正确评论。我已经使用您的 cmets 编辑了我的答案。谢谢。