DRY CLR 表值函数

Question

我正在使用 CLR 表值函数< /a> 到 SELECT 并返回使用许多变量的复杂数据库搜索的结果。

文档显示您以类似于以下的方式构建这样的函数：

public partial class UserDefinedFunctions
{
    private class ResultRow
    // This class holds a row which we want to return.
    {
        public SqlInt32 CustId;
        public SqlString Name;

        public ResultRow(SqlInt32 custId_, SqlString name_)
        {
            CustId = custId_;
            Name = name_;
        }
    }

    [SqlFunction(
        DataAccess = DataAccessKind.Read,
        FillRowMethodName = "Test_FillRow",
        TableDefinition = "CustId int" +
                          "Name nvarchar(50)")]
    public static IEnumerable Test()
    // This function contains the actual logic.
    {
        ArrayList results = new ArrayList();

        using (SqlConnection connection = new SqlConnection("context connection=true"))
        {
            connection.Open();

            using (SqlCommand select = new SqlCommand(
                "SELECT TOP 100 custid, name FROM Customers",
                connection))
            {
                using (SqlDataReader reader = select.ExecuteReader())
                {
                    while (reader.Read())
                    {
                        results.Add(new ResultRow(
                            reader.GetSqlInt32(0),  // CustId
                            reader.GetSqlString(1)  // Name
                        ));
                    }
                }
            }
        }
        return results;
    }

    public static void Test_FillRow(
        object resultsObj,
        out SqlInt32 custid,
        out SqlString name)
    // This function takes a row and tells SQL Server what variables we want to 
    // return from it and what types it contains.
    {
        ResultRow selectResults = (ResultRow)resultsObj;

        custid = selectResults.CustId;
        name = selectResults.Name;
    }
}

问题是，它相当重复。首先，在 SqlFunction 块中定义表。然后，当您在返回的结果中添加或删除列时，您必须确保更新它以及

• ResultRow 中的定义 ResultRow
• 中构造函数的参数
• ResultRow 中的赋值
• 从 Test() 中的读取器获取的类型
• Test_FillRow() 中的输出参数、
• Test_FillRow() 中的赋值
• 以及 SQL 查询本身，这是您真正要考虑的唯一部分。

我正在开发这样一个函数，它接受二十多个参数，返回更多行，并包含这八个可能出错的地方。 八。所有的错误都很微不足道，很容易修复，但很容易犯这些错误，因为代码中有很多地方我必须手动保持同步。

这是违反DRY的，但我不知道如何消除重复。有没有更简洁的方法来编写 CLR 表值函数？

Answer 1

如果将 ResultRow 替换为 object[]，则可以使用 reader.GetValues(object[]) 并消除在 FillRow() 之前必须知道行中的内容，然后 FillRow 负责了解字段在原始查询。

这确实是一种权衡，你可以放弃一直使用强类型，以换取不必一直使用强类型，但很难两者兼顾:-)

Answer 2

如果将参数集放入类中，则可以使用受其基类约束的通用方法来实现数据访问。我发现通常有一个类用于在组中传递参数，通常在其他地方以及重构时会发现有用。

将二十个参数传递给函数通常不是一个好主意。

这是我对函数参数普遍同意的一个很好的总结：
http://benbiddington.wordpress.com/2009/ 06/22/book-review-of-clean-code/

具体来说：

如果函数需要两个或三个以上的参数，则很可能
至少其中一些应该包含在自己的类中。

在这种情况下，这样做的好处可能是提取一些通用方法，帮助您更严格地遵守 DRY。