1

ストアド プロシージャを使用して SQL Server データベースに挿入するレコードのリストがあります。現在この方法でやっていますが、何か良い方法はありますか？

アプリのピーク時に毎秒 100 ～ 200 行を挿入しています。ストアド プロシージャは値のみを取得し、新しい行を挿入します。

    public void InsertRecords(List<stRecord> records)
    {
        foreach (var item in records)
        {
            if (CheckforDuplicateRecord(item) == false)
            {
                using (con = new SqlConnection(connectionString))
                {
                    con.Open();

                    SqlCommand cmd = new    SqlCommand(StoredProcedures.Service_Insert_record.ToString(), con);
                    cmd.CommandType = CommandType.StoredProcedure;

                    cmd.Parameters.Add("@item1", SqlDbType.NChar);
                    cmd.Parameters.Add("@item2", SqlDbType.NChar);

                    cmd.Parameters[0].Value = item.localUsername;
                    cmd.Parameters[1].Value = item.BetfairUsername;


                    try
                    {
                        cmd.ExecuteNonQuery();
                    }
                    catch (Exception exp)
                    {
                        throw exp;
                    }
                }
            }
        }
    }
4

2 に答える 2

9

これはまさにテーブル値パラメーターの目的です。リストを一度に渡すことができます。

SQL Server の場合:

CREATE TYPE dbo.Usernames AS TABLE
(
  localUsername   NVARCHAR(32),
  BetfairUsername NVARCHAR(32)
);
GO

CREATE PROCEDURE dbo.Service_Insert_MultipleRows
  @u AS dbo.Usernames READONLY
AS
BEGIN
  SET NOCOUNT ON;

  INSERT INTO dbo.DestinationTable(localUsername, BetfairUsername)
    SELECT localUsername, BetfairUsername FROM @u;
END
GO

今C＃で：

DataTable tvp = new DataTable();
tvp.Columns.Add(new DataColumn("localUsername"));
tvp.Columns.Add(new DataColumn("BetfairUsername"));

foreach(var item in records)
{ 
    tvp.Rows.Add(item.localUsername, item.BetfairUsername); 
}

using (con)
{
    SqlCommand cmd = new SqlCommand("Service_Insert_MultipleRows", con);
    cmd.CommandType = CommandType.StoredProcedure;
    SqlParameter tvparam = cmd.Parameters.AddWithValue("@u", tvp);
    tvparam.SqlDbType = SqlDbType.Structured;
    con.Open();
    cmd.ExecuteNonQuery();
}
于 2012-02-24T22:09:28.840 に答える
0

あなたができる明らかなことの1つは、メソッドの最初に1回SqlCommandだけ作成することです。これは、何度も何度も作成しても意味がありません。

public void InsertRecords(List<stRecord> records)
{
   using (con = new SqlConnection(connectionString))
   using (SqlCommand cmd = new SqlCommand(StoredProcedures.Service_Insert_record.ToString(), con))
   {
       cmd.CommandType = CommandType.StoredProcedure;

       // are those paramters *REALLY* just 1 character long?? 
       // that's what you have now, with the way you define it!
       cmd.Parameters.Add("@item1", SqlDbType.NChar);
       cmd.Parameters.Add("@item2", SqlDbType.NChar);

       // otherwise, you need to define the LENGTH of the NCHAR parameter!
       // cmd.Parameters.Add("@item2", SqlDbType.NChar, 15);

       foreach (var item in records)
       {
           if (CheckforDuplicateRecord(item) == false)
           {
              cmd.Parameters["@item1"].Value = item.localUsername;
              cmd.Parameters["@item2"].Value = item.BetfairUsername;

              try
              {
                 con.Open();
                 cmd.ExecuteNonQuery();
                 con.Close();
              }
              catch (Exception exp)
              {
                 throw;
              }
            }
       }
    }
于 2012-02-24T21:49:19.540 に答える