Сложный динамический SQL-запрос

Question

Мне нужна помощь в разрешении сложного SQL-запроса. Я пытаюсь создать запрос один vlock за раз. Одна проблема:: Если параметр для @PubNum равен NULL, запрос показывает «..... где PubNum =», что является проблемой. Что мне нужно, если параметр NULL, тогда PubNum не должен быть . в пункте где

Второй вопрос:

• Если @StartDate НЕ NULL и @EndDate IS NOT NULL THEN RecAddDate МЕЖДУ @StartDate и @EndDate
• Если @StartDate НЕ NULL и @EndDate IS NULL THEN RecAddDate BETWEEN @StartDate И сегодня
• Если @StartDate IS NULL и @EndDate IS NOT NULL THEN RecAddDate МЕЖДУ '01/01/2000' И @EndDate
• Если @StartDate IS NULL и @EndDate IS NULL THEN RecAddDate МЕЖДУ '01/01/2000' AND Сегодня

Любые идеи?

Полный запрос:

set ANSI_NULLS ON 
set QUOTED_IDENTIFIER ON 
go 
ALTER PROCEDURE [dbo].[usp_BookItemSearch] 
    @BookSKU varchar(30) = NULL 
    ,@SearchType int = NULL 
    ,@PubNum varchar(10) = NULL 
    ,@UserID int = NULL 
    ,@StartDate smalldatetime = NULL 
    ,@EndDate smalldatetime = NULL 
AS 
DECLARE @SQL as varchar(4000) 

SET @SQL = 'SELECT RecID, PubNum, VendorName, InvoiceNum, BookSKU, RecAddDate FROM tb_BookInventoryLog]' 

IF @BookSKU IS NOT NULL 
    BEGIN 
     IF @SearchType = 2 
      BEGIN 
       SET @SQLClause = ' WHERE BookSKU LIKE ''%' + @BookSKU + '''' --Ends with 
      END 
     IF @SearchType = 1 
      BEGIN 
       SET @SQLClause = ' WHERE BookSKU LIKE ''%' + @BookSKU + '%''' --Contains 
      END 
     IF @SearchType = 0 
      BEGIN 
       SET @SQLClause = ' WHERE BookSKU LIKE ''' + @BookSKU + '%''' --Starts with 
      END 
    END 

IF @PubNum IS NOT NULL 
    BEGIN 
     IF @SQLClause IS NOT NULL 
      BEGIN 
       SET @SQLClause = @SQLClause + ' AND PubNum = ''' + @PubNum + '''' 
      END 
     ELSE 
      BEGIN 
       SET @SQLClause = @SQLClause + ' WHERE PubNum = ''' + @PubNum + '''' 
      END 
    END 

IF @UserID IS NOT NULL 
    BEGIN 
     IF @SQLClause IS NOT NULL 
      BEGIN 
       SET @SQLClause = @SQLClause + ' AND (UserID = ' + CAST(@UserID AS VarChar) + ')' 
      END 
     ELSE 
      BEGIN 
       SET @SQLClause = @SQLClause + ' WHERE (UserID = ' + CAST(@UserID AS VarChar) + ')' 
      END 
    END 

If (@StartDate Is Not Null) AND (@EndDate Is Not Null) 
    BEGIN 
      Set @SQLClause = @SQLClause + ' And (JoiningDate BETWEEN @StartDate AND @EndDate)' 
    END 

IF (@EndDate IS NOT NULL) 
    BEGIN 
     IF (@StartDate IS NOT NULL) 
      BEGIN 
       SET @SQL = @SQL + ' WHERE RecAddDate between' + CAST(@StartDate As smalldatetime) + ' AND ' + CAST(@EndDate as smalldatetime) + '' 
      END 
     ELSE 
      BEGIN 
       SET @SQL = @SQL + ' RecAddDate BETWEEN 01/01/2000 AND @EndDate + ' 
      END 
    END 

SET @SQL = @SQL + @SQLClause + ' ORDER BY BookSKU, PubNum' 
PRINT @SQL 
--EXECUTE (@SQL) 

Answer 1

Вместо того чтобы делать динамический SQL (который вводит целый ряд проблем, и часто это на самом деле не нужно), вы могли бы просто использовать параметры и некоторые NULL проверки, как часть вашего WHERE с использованием нескольких различных методов, в зависимости от того, что вы пытаетесь сделать. Я тестировал это с помощью SQL 2000/2005, и он работает правильно (я бы тоже предположил, что это нормально в 2008/R2).

ALTER PROCEDURE [dbo].[usp_BookItemSearch] 
    @BookSKU varchar(30) = NULL 
    ,@SearchType int = NULL 
    ,@PubNum varchar(10) = NULL 
    ,@UserID int = NULL 
    ,@StartDate smalldatetime = NULL 
    ,@EndDate smalldatetime = NULL 
AS 


    SELECT RecID, PubNum, VendorName, InvoiceNum, BookSKU, RecAddDate 
    FROM [tb_BookInventoryLog] 
    WHERE (
      (@BookSKU IS NULL) OR 
      (BookSKU LIKE CASE @SearchType 
          WHEN 0 THEN  @BookSKU + '%' 
          WHEN 1 THEN '%' + @BookSKU + '%' 
          WHEN 2 THEN '%' + @BookSKU 
         END 
     ) 
     ) 
    AND ISNULL(@PubNum, PubNum) = PubNum 
    AND ISNULL(CAST(@UserID AS VARCHAR), UserID) = UserID 
    AND (
      (@StartDate IS NULL OR @EndDate IS NULL) OR 
      (JoiningDate BETWEEN @StartDate AND @EndDate) 
     ) 
    AND RecAddDate BETWEEN CASE 
           WHEN @EndDate IS NULL THEN RecAddDate 
           ELSE ISNULL(@StartDate, '01/01/2000') 
          END 
         AND ISNULL(@EndDate, GETDATE()) 
ORDER BY BookSKU, PubNum 

Answer 2

Я бы условно включать фильтры, если не было нулевым. Что-то вроде этого: -

Set @WhereClause = 'Where 1=1' 
If @PubNum is not null 
    Set @WhereClause = @WhereClause + ' AND PubNum = ''' + @PubNum + '''' 

Answer 3

Для вашего второго вопроса:

(@StartDate не отображает значение параметра, а не отображает "..... StartDate = @StartDate .....")

Если вы хотите продолжать использовать динамический SQL, рассмотрите возможность замены EXECUTE на sp_executesql. Эта хранимая процедура будет принимать параметры, которые затем можно использовать внутри динамического SQL (замена параметров). Например:

Set @SQLClause = @SQLClause + ' And (JoiningDate BETWEEN @StartDate_PARAM AND @EndDate_PARAM)'

Для выше, добавьте @EndDate_PARAM в качестве параметра, как так:

exec sp_executesql @SQL, N'@StartDate_PARAM DateTime, @EndDate_PARAM DateTime', @StartDate_PARAM = @StartDate, @EndDate_PARAM = @EndDate

Это выглядит немного странно на первый, но это хорошая вещь, чтобы сделать, если вы идете по динамическому пути SQL.

Вы найдете более подробную информацию о том, что здесь: http://msdn.microsoft.com/en-us/library/ms175170.aspx