比使用IF更適合這種情況嗎？

Question

我正在寫一個網站上使用的查詢。當需要查詢時，它直接被解析到服務器，結果全部由查詢控制。該網站只是返回表格。我在網站上做的是客戶端可以選擇的複選框，並將這些複選框解析爲查詢@var = 1或@var = 0。正因爲如此，現在我有這個代碼來檢查和添加或不取決於它是否被檢查。我的問題是，有沒有更好的辦法去這比使用IF聲明是這樣，因爲我有代碼的幾個部分，其中包括這樣的：

SET @sql = 'select distinct ' 
If @mgchk = 1 
    SET @sql = @sql + 'p.MainGroup' 
If @sgchk = 1 and @sql = 'select distinct ' 
    SET @sql = @sql + 'p.SubGroup' 
If @sgchk = 1 and @sql not like '%SubGroup' 
    SET @sql = @sql + ',p.SubGroup' 
If @ssgchk = 1 and @sql = 'select distinct ' 
    SET @sql = @sql + 'p.SubSubGroup' 
If @ssgchk = 1 and @sql not like '%SubSubGroup' 
    SET @sql = @sql + ',p.SubSubGroup' 
If @Seasonchk = 1 and @sql = 'select distinct ' 
    SET @sql = @sql + 'p.Season' 
If @Seasonchk = 1 and @sql not like '%Season' 
    SET @sql = @sql + ',p.Season' 
If @vendorchk = 1 and @sql = 'select distinct ' 
    SET @sql = @sql + 'p.VendorID' 
If @vendorchk = 1 and @sql not like '%VendorID' 
    SET @sql = @sql + ',p.VendorID' 

SET @sql = 
    @sql + 
    ' into 
     ##aa 
    from 
     RPProducts p, 
     RPIv i, 
     RPTrsd d, 
     RPTrs s 
    WHERE 
    s.StoreID = d.StoreID and 
     s.ReceiptNO = d.ReceiptNO and 
     i.UPC = d.UPC and 
     i.StoreID = d.StoreID and 
     i.IVProduct = p.Productid and 
     s.TRSdate >= '''+ convert(varchar(10), @trsfrom, 101) +''' and 
     s.TRSdate <= '''+ convert(varchar(10), @trsto, 101) +'''' 
execute sp_executesql @sql 

@mgchk/@sgchk/@ssgchk/@seasonchk/@vendorchk是複選框變量

要回答@Aaron，

全局臨時因爲動態查詢。整個查詢得到處理並在數據被拖動時立即丟棄。沒有衝突會發生在那裏。

我的日期變量是datetime，它在動態SQL中給我一個錯誤。

是的，回想一下同樣的事情來檢查，這是整個問題的原因，如果有更好的使用比IF檢查。

，我發現使用別名連接更容易...

Answer 1

-- rather than convert to a dangerously formatted string, 
-- here is a much better way to strip time from a datetime 
-- (if you need to!) 

SET @trsfrom = DATEADD(DAY, DATEDIFF(DAY, 0, @trsfrom), 0); 
SET @trsto = DATEADD(DAY, DATEDIFF(DAY, 0, @trsto), 0); 

DECLARE @sql NVARCHAR(MAX) = N'SELECT DISTINCT '; 

-- here's an easier way to strip the first comma: 

SET @sql += SUBSTRING(
    CASE WHEN @mgchk  = 1 THEN ',p.MainGroup' ELSE '' END 
    + CASE WHEN @sgchk  = 1 THEN ',p.SubGroup' ELSE '' END 
    + CASE WHEN @ssgchk = 1 THEN ',p.SubSubGroup' ELSE '' END 
    + CASE WHEN @Seasonchk = 1 THEN ',p.Season'  ELSE '' END 
    + CASE WHEN @vendorchk = 1 THEN ',p.VendorID' ELSE '' END, 2, 2000); 

SET @sql += ' INTO ##aa 
    FROM 
     dbo.RPProducts AS p -- use schema prefix! 
    INNER JOIN dbo.RPIv AS i -- use PROPER JOINS! 
     ON i.IVProduct = p.Productid 
    INNER JOIN dbo.RPTrsd AS d 
     ON i.UPC = d.UPC 
     AND i.StoreID = d.StoreID 
    INNER JOIN dbo.RPTrs AS s 
     ON s.StoreID = d.StoreID 
     AND s.ReceiptNO = d.ReceiptNO 
    WHERE s.TRSdate >= @trsfrom -- use strongly typed parameters! 
    AND s.TRSdate <= @trsto;'; 

EXEC sp_executesql @sql, 
    N'@trsfrom DATETIME, @trsto DATETIME', 
    @trsfrom, @trsto; 
----^^^^^^^^^^^^^^^^ here is how the query gets the @trsfrom & @trsto values 

我仍然認爲你一個##全局臨時表的使用是相當危險的。如果兩個人同時運行這些代碼，他們將會遇到嚴重的問題。