程序执行大量sql语句，请大神指点该如何优化

lanxuan

服务器是阿里云8核16g，程序需要在晚上十二点进行数据计算，计算过程中，cpu使用100%，计算完成之后一个小时内，用户反映系统特别卡，请大神指点该怎么优化。
现在程序的调用方法是：

[C#] 纯文本查看 复制代码

/// <summary>
        /// 执行多条SQL语句，实现数据库事务。
        /// </summary>
        /// <param name="SQLStringList">多条SQL语句</param>		
        public static int ExecuteSqlTran(List<String> SQLStringList)
        {
            using (SqlConnection conn = new SqlConnection(connectionString))
            {
                conn.Open();
                SqlCommand cmd = new SqlCommand();
                cmd.Connection = conn;
                SqlTransaction tx = conn.BeginTransaction();
                cmd.Transaction = tx;
                try
                {
                    int count = 0;
                    for (int n = 0; n < SQLStringList.Count; n++)
                    {
                        string strsql = SQLStringList[n];
                        Log.xierizhi2(strsql);
                        if (strsql.Trim().Length > 1)
                        {
                            cmd.CommandText = strsql;
                            count += cmd.ExecuteNonQuery();
                        }
                    }
                    tx.Commit();
                    return count;
                }
                catch
                {
                    tx.Rollback();
                    return 0;
                }
            }
        }

SQLStringList里面有12万条sql语句，拼接SQLStringList花了差不多90秒，ExecuteSqlTran执行了70秒，搞不懂是这个方法不够优化，还是访问量太大造成的系统卡，请各位指点。@站长苏飞

客服~小小

执行的慢一点可以，另外使用Sql优化顾问做个监控，然后建一下统计，和索引应该会好很好

站长苏飞

循环执行太快了应该是，中间加个时间间隔应该会好很多

lanxuan

客服~小小 发表于 2016-12-27 10:11
执行的慢一点可以，另外使用Sql优化顾问做个监控，然后建一下统计，和索引应该会好很好

谢谢，执行慢一点可以 是什么意思？是说这个速度正常么？

lanxuan

站长苏飞 发表于 2016-12-27 10:16
循环执行太快了应该是，中间加个时间间隔应该会好很多

是ExecuteSqlTran 这个方法执行的太快了吗？

[C#] 纯文本查看 复制代码

public static int ExecuteSqlTran(List<String> SQLStringList)
        {
            using (SqlConnection conn = new SqlConnection(connectionString))
            {
                conn.Open();
                SqlCommand cmd = new SqlCommand();
                cmd.Connection = conn;
                SqlTransaction tx = conn.BeginTransaction();
                cmd.Transaction = tx;
                try
                {
                    int count = 0;
                    for (int n = 0; n < SQLStringList.Count; n++)
                    {
                        string strsql = SQLStringList[n];
                        Log.xierizhi2(strsql);
                        if (strsql.Trim().Length > 1)
                        {
                            cmd.CommandText = strsql;
                            count += cmd.ExecuteNonQuery();
                        }
                        System.Threading.Thread.Sleep(20);
                    }
                    tx.Commit();
                    return count;
                }
                catch
                {
                    tx.Rollback();
                    return 0;
                }
            }
        }

这样可以吗？

站长苏飞

循环，你的循环中间是没有时间间隔了，如果太多，相当于死循环了。循环每执行一次加个时间暂时一下

站长苏飞

Thread.Sleep(1000); 放到循环最下面这表示 一秒执行一次。当然时间上你自己控制，

lanxuan

站长苏飞 发表于 2016-12-27 10:53
Thread.Sleep(1000); 放到循环最下面这表示 一秒执行一次。当然时间上你自己控制，

飞哥，还是没搞懂，是拼接 SQLStringList 的循环里面加个间隔，还是在执行 ExecuteSqlTran方法中的循环里面加个间隔？@站长苏飞

站长苏飞

执行的时候，你循环执行每秒可能发送上百个Sql语句这样会给数据库造成很大的压力，所以加个一个Thread.Sleep(1000);是控制一下发送的频率，