重用 SqlDataRecord 是否安全？

Question

在实现表值参数时，生成IEnumerable<SqlDataRecord> 以供参数使用的最常见方法之一是这样的代码（例如，https://stackoverflow.com/a/10779567/18192）：

public static IEnumerable<SqlDataRecord> Rows(List<int> simpletable)
{
    var smd = new []{ new SqlMetaData("id", SqlDbType.Int)};
    var sqlRow = new SqlDataRecord(smd);
    foreach (int i in simpletable)
    {
        sqlRow.SetInt32(0, i);
        yield return sqlRow;
    }
}
//...
var param = sqlCmd.Parameters.AddWithValue("@retailerIDs", Rows(mydata)); 
param.SqlDbType = SqlDbType.Structured;
param.TypeName = "myTypeName";

此代码似乎确实有效。虽然重用SqlMetaData 不会引起太多警钟，但在foreach 循环之外声明SqlDataRecord 对我来说非常可疑：

一个可变对象被修改然后重复生成。

作为一个令人担忧的例子，在 LinqPad 中调用 var x = Rows(new[] { 100, 200}.ToList()).ToList().Dump() 会输出 200,200。这种方法似乎依赖于实现细节（行被单独处理），但我没有看到任何承诺这一点的文档。

是否有一些缓解因素使这种方法安全？

Answer 1

这种方法似乎依赖于实现细节（行是 单独处理），但我没有看到任何文件 承诺这一点。

是否有一些缓解因素可以使这种方法安全？

正如 user1249190 指出的那样，https://docs.microsoft.com/en-us/dotnet/api/microsoft.sqlserver.server.sqldatarecord#remarks 的备注部分明确建议重用 SQLDataRecord：

该类与 SqlPipe 一起使用，将结果集发送到 来自托管代码存储过程的客户端。写常见的时候 语言运行时 (CLR) 应用程序，您应该重用现有的 SqlDataRecord 对象，而不是每次都创建新对象。 创建许多新的 SqlDataRecord 对象可能会严重耗尽内存 并对性能产生不利影响。

显然，此建议不适用于跨线程使用：文档还明确警告“任何实例成员都不能保证是线程安全的。”

Answer 2

如果您在 foreach 循环之外根本不需要它，我不明白您为什么要重复使用它。

我发现这个问题Is there a reason for C#'s reuse of the variable in a foreach? 链接到另一个问题Is it better coding practice to define variables outside a foreach even though more verbose? 中的这个答案，Jon Skeet 回答说：

在循环之外声明变量没有任何好处，除非你想在迭代之间保持它们的值。

（请注意，这通常不会产生行为差异，但如果变量是由 lambda 表达式或匿名方法捕获的，则情况并非如此。）

Answer 3

不，重复使用变量是不安全的。代码一遍又一遍地修改同一个对象。这是错误的代码，应该返回一个新对象。这是一个快速的 linqpad 示例，显示了上述代码的问题：

void Main()
{
    //This code proves that the object is being modified.
   Thing prevRow = null;
    foreach (var curRow in Rows(new List<int>() { 1, 2, 3 }))
    {
        Console.WriteLine(curRow);
        Console.WriteLine(prevRow);
        prevRow = curRow;
    }

    //Because the object is modified instead of a new object being returned,
    // this code does something unexpected; it returns the same object 3
    // times! Instead of three unique objects representing the values 1, 2, 3.
    var rowsAsList = Rows(new List<int>() { 1, 2, 3 }).ToList();
    foreach (var curRow in rowsAsList)
    {
        Console.WriteLine(curRow);
    }
}

public class Thing
{
    public int i;
}

IEnumerable<Thing> Rows(List<int> simpletable)
{
    var sqlRow = new Thing() {i=-1};
    foreach (int i in simpletable)
    {
        sqlRow.i = i;
        yield return sqlRow;
    }
}