将List <>传递给SQL存储过程

Question

我经常不得不将多个项目加载到数据库中的特定记录.例如:网页显示要包含在单个报表中的项目,所有这些都是数据库中的记录(报表是报表中的记录,项目是项目表中的记录).用户通过Web应用程序选择要包含在单个报告中的项目,并假设他们选择3个项目并提交.通过将记录添加到名为ReportItems(ReportId,ItemId)的表中,该过程将这3个项添加到此报告中.

目前,我会在代码中执行以下操作:

public void AddItemsToReport(string connStr, int Id, List<int> itemList)
{
    Database db = DatabaseFactory.CreateDatabase(connStr);

    string sqlCommand = "AddItemsToReport"
    DbCommand dbCommand = db.GetStoredProcCommand(sqlCommand);

    string items = "";
    foreach (int i in itemList)
        items += string.Format("{0}~", i);

    if (items.Length > 0)
        items = items.Substring(0, items.Length - 1);

    // Add parameters
    db.AddInParameter(dbCommand, "ReportId", DbType.Int32, Id);
    db.AddInParameter(dbCommand, "Items", DbType.String, perms);
    db.ExecuteNonQuery(dbCommand);
}

这在存储过程中:

INSERT INTO ReportItem (ReportId,ItemId)
SELECT  @ReportId,
          Id
FROM     fn_GetIntTableFromList(@Items,'~')

函数返回一列整数表.

我的问题是:有没有更好的方法来处理这样的事情？注意,我不是在询问数据库规范化或类似的事情,我的问题与代码有关.

Answer 1

如果选择SQL Server 2008是一个选项,那么有一个名为"表值参数"的新功能可以解决这个问题.

在这里和这里查看有关TVP的更多详细信息,或者只是向Google询问"SQL Server 2008表值参数" - 您将找到大量信息和示例.

强烈推荐 - 如果你可以转移到SQL Server 2008 ...

Answer 2

您的字符串连接逻辑可能会简化:

string items = 
    string.Join("~", itemList.Select(item=>item.ToString()).ToArray());

这将节省一些字符串连接,这在.Net中很昂贵.

我认为保存物品的方式没有任何问题.你正在限制前往数据库,这是一件好事.如果您的数据结构比整数列表更复杂,我建议使用XML.

注意:我在评论中被问到这是否会为我们节省任何字符串连接(它确实存在缺陷).我认为这是一个很好的问题,并希望对此采取后续行动.

如果你剥开open string.Join with Reflector你会看到微软正在使用一些不安全的(在.Net意义上的)技术,包括使用char指针和一个名为UnSafeCharBuffer的结构.他们正在做什么,当你真正把它煮沸时,是使用指针走过一个空字符串并建立连接.请记住,.Net中字符串连接如此昂贵的主要原因是每个连接都会在堆上放置一个新的字符串对象,因为字符串是不可变的.那些内存操作很昂贵.String.Join(..)本质上是分配内存一次,然后用指针操作它.非常快.

Answer 3

您的技术的一个潜在问题是它不处理非常大的列表 - 您可能超过数据库的最大字符串长度.我使用一个帮助器方法,将整数值连接成一个字符串的枚举,每个字符串都小于指定的最大值(以下实现也可选择检查并删除重复的ID):

public static IEnumerable<string> ConcatenateValues(IEnumerable<int> values, string separator, int maxLength, bool skipDuplicates)
{
    IDictionary<int, string> valueDictionary = null;
    StringBuilder sb = new StringBuilder();
    if (skipDuplicates)
    {
        valueDictionary = new Dictionary<int, string>();
    }
    foreach (int value in values)
    {
        if (skipDuplicates)
        {
            if (valueDictionary.ContainsKey(value)) continue;
            valueDictionary.Add(value, "");
        }
        string s = value.ToString(CultureInfo.InvariantCulture);
        if ((sb.Length + separator.Length + s.Length) > maxLength)
        {
            // Max length reached, yield the result and start again
            if (sb.Length > 0) yield return sb.ToString();
            sb.Length = 0;
        }
        if (sb.Length > 0) sb.Append(separator);
        sb.Append(s);
    }
    // Yield whatever's left over
    if (sb.Length > 0) yield return sb.ToString();
}

然后你使用它像:

using(SqlCommand command = ...)
{
    command.Connection = ...;
    command.Transaction = ...; // if in a transaction
    SqlParameter parameter = command.Parameters.Add("@Items", ...);
    foreach(string itemList in ConcatenateValues(values, "~", 8000, false))
    {
        parameter.Value = itemList;
        command.ExecuteNonQuery();
    }
}

Answer 4

你要么做你已经得到的东西,传入一个分隔的字符串,然后解析出一个表值,或者另一个选择是传递一个XML并且有点相同:

http://weblogs.asp.net/jgalloway/archive/2007/02/16/passing-lists-to-sql-server-2005-with-xml-parameters.aspx

我还没有机会查看SQL 2008,看看他们是否添加了任何新功能来处理这类事情.

Answer 5

为什么不使用表值参数？ http://msdn.microsoft.com/en-us/library/bb675163.aspx