2013-05-02 78 views
4

在我开始之前,我知道你不能从UDF调用存储过程,并且我知道有很多“原因”尽管对我来说很有意义,但这听起来像微软的懒惰一样)。如何避免从SQL Server中的UDF调用存储过程

我对如何设计系统来解决SQL Server中的这个缺陷更感兴趣。

这是系统的快速概览我目前有:

  • 我有一个动态报表生成器,用户指定的数据项,运算符(=,<,=,等等!)和过滤器值。这些用于通过一个或多个过滤器来构建“规则”,例如,我可能有一个规则,有两个过滤器“类别< 12”和“位置!='约克'”;

  • 有成千上万的这些“规则”,其中一些有许多过滤器;

  • 这些规则中的每个规则的输出都是一个始终具有完全相同的“形状”(即相同的列/数据类型)的法定报表。基本上这些报告产生吨位和材料清单;

  • 我有一个标量值函数为指定规则生成动态SQL,并将其作为VARCHAR(MAX)返回;

  • 我有一个被调用来运行特定规则的存储过程,它调用UDF来生成动态SQL,运行它并返回结果(这用于返回结果,但现在我将输出存储在进程键控表,使数据更容易共享,所以我返回一个处理这个数据,而不是);

  • 我有一个存储过程被调用来运行特定公司的所有规则,因此它使得要运行的规则列表顺序运行,然后将结果合并为输出。

所以这一切都完美的作品。

现在我想要一个最后的东西,一份运行公司摘要的报告,然后将成本应用于吨位/物料以产生成本报告。这似乎是这样一个简单的要求,当我最后一周开始时:'(

我的报告必须是一个表值函数,它可以与我已经编写的报告代理系统一起工作,如果我将它写为存储过程,那么它将不会通过我的报告代理运行,这意味着它不会被控制,也就是说,我不知道是谁在运行报告和什么时候运行。表值函数和两种明显的处理方法如下:

  1. 获取SQL以创建输出,运行并吸取结果。

    --Method #1 
    WHILE @RuleIndex <= @MaxRuleIndex 
    BEGIN 
    DECLARE @DSFId UNIQUEIDENTIFIER; 
    SELECT @DSFId = [GUID] FROM NewGUID; --this has to be deterministic, it isn't but the compiler thinks it is and that's good enough :D 
    DECLARE @RuleId UNIQUEIDENTIFIER; 
    SELECT @RuleId = DSFRuleId FROM @DSFRules WHERE DSFRuleIndex = @RuleIndex; 
    DECLARE @SQL VARCHAR(MAX); 
    
    --Get the SQL 
    SELECT @SQL = DSF.DSFEngine(@ServiceId, @MemberId, @LocationId, @DSFYear, NULL, NULL, NULL, NULL, @DSFId, @RuleId); 
    
    --Run it 
    EXECUTE(@SQL); 
    
    --Copy the data out of the results table into our local copy 
    INSERT INTO 
        @DSFResults 
    SELECT 
        TableId, TableCode, TableName, RowId, RowName, LocationCode, LocationName, ProductCode, ProductName, PackagingGroupCode, PackagingGroupName, LevelName, WeightSource, Quantity, Paper, Glass, Aluminium, Steel, Plastic, Wood, Other, 0 AS General 
    FROM 
        DSF.DSFPackagingResults 
    WHERE 
        DSFId = @DSFId 
        AND RuleId = @RuleId; 
    SELECT @RuleIndex = @RuleIndex + 1; 
    END; 
    
  2. 呼叫报告直接

    --Method #2 
    WHILE @RuleIndex <= @MaxRuleIndex 
    BEGIN 
    DECLARE @DSFId UNIQUEIDENTIFIER; 
    SELECT @DSFId = [GUID] FROM NewGUID; --this has to be deterministic, it isn't but the compiler thinks it is :D 
    DECLARE @RuleId UNIQUEIDENTIFIER; 
    SELECT @RuleId = DSFRuleId FROM @DSFRules WHERE DSFRuleIndex = @RuleIndex; 
    DECLARE @SQL VARCHAR(MAX); 
    
    --Run the report 
    EXECUTE ExecuteDSFRule @ServiceId, @MemberId, @LocationId, @DSFYear, NULL, NULL, NULL, @RuleId, @DSFId, 2; 
    
    --Copy the data out of the results table into our local copy 
    INSERT INTO 
        @DSFResults 
    SELECT 
        TableId, TableCode, TableName, RowId, RowName, LocationCode, LocationName, ProductCode, ProductName, PackagingGroupCode, PackagingGroupName, LevelName, WeightSource, Quantity, Paper, Glass, Aluminium, Steel, Plastic, Wood, Other, 0 AS General 
    FROM 
        DSF.DSFPackagingResults 
    WHERE 
        DSFId = @DSFId 
        AND RuleId = @RuleId; 
    SELECT @RuleIndex = @RuleIndex + 1; 
    END; 
    

我能想到以下解决方法(其中没有一个是特别满意的)的:

  • 改写一些这在CLR(但这只是打破规则的一大麻烦);

  • 使用存储过程来生成我的报告(但这意味着我失去了对执行的控制权,除非我为这个SINGLE报告开发一个新系统,与几十个现有报告完全不同)。

  • 从报告中分离执行,所以我有一个过程来执行报告,另一个过程只是提取输出(但没有办法告诉报告何时完成而没有更多的工作);

  • 等到Microsoft看到感觉并允许从UDF执行存储过程。

还有其他想法吗?


编辑3月 - 2013年,这里是如何挂在一起的非常简单的例子:

--Data to be reported 
CREATE TABLE DataTable (
    MemberId INT, 
    ProductId INT, 
    ProductSize VARCHAR(50), 
    Imported INT, 
    [Weight] NUMERIC(19,2)); 
INSERT INTO DataTable VALUES (1, 1, 'Large', 0, 5.4); 
INSERT INTO DataTable VALUES (1, 2, 'Large', 1, 6.2); 
INSERT INTO DataTable VALUES (1, 3, 'Medium', 0, 2.3); 
INSERT INTO DataTable VALUES (1, 4, 'Small', 1, 1.9); 
INSERT INTO DataTable VALUES (1, 5, 'Small', 0, 0.7); 
INSERT INTO DataTable VALUES (1, 6, 'Small', 1, 1.2); 

--Report Headers 
CREATE TABLE ReportsTable (
    ReportHandle INT, 
    ReportName VARCHAR(50)); 
INSERT INTO ReportsTable VALUES (1, 'Large Products'); 
INSERT INTO ReportsTable VALUES (2, 'Imported Small Products'); 

--Report Detail 
CREATE TABLE ReportsDetail (
    ReportHandle INT, 
    ReportDetailHandle INT, 
    DatabaseColumn VARCHAR(50), 
    DataType VARCHAR(50), 
    Operator VARCHAR(3), 
    FilterValue VARCHAR(50)); 
INSERT INTO ReportsDetail VALUES (1, 1, 'ProductSize', 'VARCHAR', '=', 'Large'); 
INSERT INTO ReportsDetail VALUES (2, 1, 'Imported', 'INT', '=', '1'); 
INSERT INTO ReportsDetail VALUES (2, 1, 'ProductSize', 'VARCHAR', '=', 'Small'); 
GO 
CREATE FUNCTION GenerateReportSQL (
    @ReportHandle INT) 
RETURNS VARCHAR(MAX) 
AS 
BEGIN 
    DECLARE @SQL VARCHAR(MAX); 
    SELECT @SQL = 'SELECT SUM([Weight]) FROM DataTable WHERE 1=1 '; 
    DECLARE @Filters TABLE (
     FilterIndex INT, 
     DatabaseColumn VARCHAR(50), 
     DataType VARCHAR(50), 
     Operator VARCHAR(3), 
     FilterValue VARCHAR(50)); 
    INSERT INTO @Filters SELECT ROW_NUMBER() OVER (ORDER BY DatabaseColumn), DatabaseColumn, DataType, Operator, FilterValue FROM ReportsDetail WHERE ReportHandle = @ReportHandle; 
    DECLARE @FilterIndex INT = NULL; 
    SELECT TOP 1 @FilterIndex = FilterIndex FROM @Filters; 
    WHILE @FilterIndex IS NOT NULL 
    BEGIN 
     SELECT TOP 1 @SQL = @SQL + ' AND ' + DatabaseColumn + ' ' + Operator + ' ' + CASE WHEN DataType = 'VARCHAR' THEN '''' ELSE '' END + FilterValue + CASE WHEN DataType = 'VARCHAR' THEN '''' ELSE '' END FROM @Filters WHERE FilterIndex = @FilterIndex; 
     DELETE FROM @Filters WHERE FilterIndex = @FilterIndex; 
     SELECT @FilterIndex = NULL; 
     SELECT TOP 1 @FilterIndex = FilterIndex FROM @Filters; 
    END; 
    RETURN @SQL; 
END; 
GO 
CREATE PROCEDURE ExecuteReport (
    @ReportHandle INT) 
AS 
BEGIN 
    --Get the SQL 
    DECLARE @SQL VARCHAR(MAX); 
    SELECT @SQL = dbo.GenerateReportSQL(@ReportHandle); 
    EXECUTE (@SQL); 
END; 
GO 
--Test 
EXECUTE ExecuteReport 1; 
EXECUTE ExecuteReport 2; 
SELECT dbo.GenerateReportSQL(1); 
SELECT dbo.GenerateReportSQL(2); 
GO 
--What I really want 
CREATE FUNCTION RunReport (
    @ReportHandle INT) 
RETURNS @Results TABLE ([Weight] NUMERIC(19,2)) 
AS 
BEGIN 
    INSERT INTO @Results EXECUTE ExecuteReport @ReportHandle; 
    RETURN; 
END; 
--Invalid use of a side-effecting operator 'INSERT EXEC' within a function 
+1

你能否将此问题简化为[SSCCE](http://sscce.org/)? – 2013-05-02 20:48:31

+0

我已经尽了我最大的努力 – 2013-05-03 11:31:16

+1

“我不知道谁在报道什么时候报道。”如果这就是你想达到的目标,那么我肯定有很多解决方案可以捕获这些信息。 – 2013-05-03 12:39:48

回答

2

如果我是你的情况,我不会试图破解什么。我想设置这样的对象:

CREATE TABLE [dbo].[ReportCollection] (
    [ReportCollectionID] int, 
    [ReportID] int 
) 

CREATE TABLE [dbo].[ReportResult] (
    [ReportID] int, 
    [LocationCode] int, 
    [LocationName] nvarchar(max) 
) 

CREATE PROCEDURE [dbo].[usp_ExecuteReport] (
    @ReportID int 
) 
AS 
    INSERT [dbo].[ReportResult] 
    SELECT @ReportID, 1, N'StackOverflow' 
END 

CREATE FUNCTION [dbo].[udf_RetrieveReportCollectionResults] (
    @ReportCollectionID int 
) 
RETURNS @Results TABLE ([ReportID], [LocationCode], [LocationName]) 
AS 
BEGIN 
    SELECT * 
    FROM [dbo].[ReportResult] rr 
    JOIN [dbo].[ReportCollection] rc 
     ON rr.ReportID = rc.ReportID 
    WHERE rc.ReportCollectionID = @ReportCollectionID 
END 

,并利用它们是这样的:

INSERT [dbo].[ReportCollection] VALUES (1, 1) 
INSERT [dbo].[ReportCollection] VALUES (1, 2) 

EXEC [dbo].[usp_ExecuteReport] @ReportID = 1 
EXEC [dbo].[usp_ExecuteReport] @ReportID = 2 

SELECT * FROM [dbo].[udf_RetrieveReportCollectionResults](1) 

每次运行的报告时,开始一个新的集合。您的应用程序应该启动所有报告并在之后整理结果。

-

如果你真的想叫从UDF存储过程(请不要),做xp_cmdshell的搜索。

0

如果你真的想把这个功能作为一个功能,那么最简单的方法就是CLR集成。

您不必重做一切 - 只写调用存储过程返回&存储的特效结果集作为它的own.1

这样,所有当前的SQL开发是一个不变CLR包装功能。

相关问题