MS AdventureWorks dbo.ufnGetProductListPrice

Question

我正在研究 Ms SQL AdventureWorks 2014 到 model 一個我們公司的內部數據庫

我通常在 Postgres 上工作，我試圖“理解”SQL 女士存儲過程...... :-)) 但是......

商店過程 [dbo.ufnGetProductListPrice ] 對我來說似乎很奇怪。 the SQL code is found here: ( https://dataedo.com/samples/html/AdventureWorks/doc/AdventureWorks_2/functions/dbo_ufnGetProductListPrice_116.html )

 CREATE FUNCTION [dbo].[ufnGetProductListPrice](@ProductID [int], @OrderDate [datetime])
RETURNS [money] 
AS 
BEGIN
    DECLARE @ListPrice money;

    SELECT @ListPrice = plph.[ListPrice] 
    FROM [Production].[Product] p 
        INNER JOIN [Production].[ProductListPriceHistory] plph 
        ON p.[ProductID] = plph.[ProductID] 
            AND p.[ProductID] = @ProductID 
            AND @OrderDate BETWEEN plph.[StartDate] AND COALESCE(plph.[EndDate], CONVERT(datetime, '99991231', 112)); -- Make sure we get all the prices!

    RETURN @ListPrice;
END;

function 正在使用以下兩個表：

• [Production.Product]（ https://dataedo.com/samples/html/AdventureWorks/doc/AdventureWorks_2/tables/Production_Product_153.html ）

• [Production.ProductListPriceHistory] - ( https://dataedo.com/samples/html/AdventureWorks/doc/AdventureWorks_2/tables/Production_ProductListPriceHistory_159.html )

特別是我的疑問是：

1. function 正在獲取參數@ProductID。 相同的 @ProductID 用作 Production.Product 的主鍵和表 Production.ProductListPriceHistory 的主鍵的一部分，因此在 Product.[ProductID] = ProductListPriceHistory.[ProductID] 上進行連接似乎沒有幫助我們可以直接在 ProductListPriceHistory.[ProductID] 上測試 ProductID
   為什么要創建這樣的連接？ 好像沒救了...

2. 作為第二個參數接收的給定 @Orderdate 日期時間在 JOIN 中根據以下條件進行檢查

AND @OrderDate BETWEEN plph.[StartDate] AND COALESCE(plph.[EndDate], CONVERT(datetime, '99991231', 112)); -- 確保我們得到所有的價格！

但是，如果我們為 @Orderdate = 1/1/2022 調用存儲過程，

• 考慮到 [EndDate] 可能是 NULL，
• 我們可以在 ProductListPriceHistory.StartDate 中有兩條帶有我們@ProductID 的記錄，第一條帶有 StartDate=1/1/2020，第二條 StartDate=1/1/2021，

這樣的“BETWEEN”條件應該匹配他們兩個，而顯然我們會期待最后一個......這是一個錯誤嗎？

Answer 1

你是對的，這段代碼有很多嚴重的缺陷，我建議你找更好的教程。

我已經注意到對每個缺陷的評論

CREATE FUNCTION [dbo].[ufnGetProductListPrice](@ProductID [int], @OrderDate [datetime])
RETURNS [money]   -- money is a bad data type due to rounding problems, decimal should be used
                  -- scalar UDFs are very slow, this should be a inline Table Valued Function
AS 
BEGIN
    DECLARE @ListPrice money;

    SELECT @ListPrice = plph.[ListPrice]    -- as you point out, there should be some kind of aggregation
    FROM [Production].[Product] p           -- as you point out: join is unnecessary
        INNER JOIN [Production].[ProductListPriceHistory] plph 
        ON p.[ProductID] = plph.[ProductID] 
            AND p.[ProductID] = @ProductID 
                           -- BETWEEN should never be used on dates, use ">= AND <"
            AND @OrderDate BETWEEN plph.[StartDate] AND COALESCE(plph.[EndDate], CONVERT(datetime, '99991231', 112));   -- ISNULL is better for performance than COALESCE

    RETURN @ListPrice;
END;

更好的 function 將是這個

CREATE FUNCTION dbo.ufnGetProductListPrice (
  @ProductID int,
  @OrderDate datetime
)
RETURNS TABLE
AS RETURN

SELECT plph.ListPrice   -- you can return multiple rows
-- alternatively you can use aggregation
-- SELECT ListPrice = AVG(plph.ListPrice)

FROM Production.ProductListPriceHistory plph 
WHERE plph.ProductID = @ProductID 
  AND @OrderDate >= plph.StartDate
  AND @OrderDate < ISNULL(plph.EndDate, CONVERT(datetime, '99991231', 112));

而不是使用標量 UDF

SELECT p.*, dbo.ufnGetProductListPrice(p.ProductId, GETDATE())
FROM Production.Product p;

您對 TVF 使用APPLY

SELECT p.*, plp.*
FROM Production.Product p
CROSS APPLY dbo.ufnGetProductListPrice(p.ProductId, GETDATE()) plp;

Answer 2

@Charlieface 感謝您的回復。 關於 function ufnGetProductStandardCost 的另一個說明

AdventureWorks 存儲歷史零售價格和成本價格，讓您在特定時間找到特定產品的價格（列表、成本）。 這就是我對數據庫的這一部分感興趣的原因

It even offers a "dual" function for Cost prices ufnGetProductStandardCost https://dataedo.com/samples/html/AdventureWorks/doc/AdventureWorks_2/functions/dbo_ufnGetProductStandardCost_117.html
有同樣的錯誤：-(

我認為 ufnGetProductListPrice(ProductID, OrderDate) 的想法是報告單一價格或 Null 如果當時價格不存在。
不是表格或列表行。

讓我們回想一下我之前假設的情況

• 搜索訂單日期 2022 年 1 月 1 日
• StartDate=1/1/2020, EndDate=null 的一條記錄
• StartDate=1/1/2021, EndDate=null 的一條記錄

這種 function 的一個想法是：

• 使用子查詢查找給定 @ProductId 的 MAX() StardDate<@OrderDate
• 使用此類 StartDate 在外部查詢中獲取對應的 ListPrice 或返回 NULL 如果我們在 @OrderDate 之前沒有 StartDate ListPrice（例如 @OrderDate=1/1/2019）

你覺得這個查詢怎么樣？ 有什么改善嗎？