我只看出一个问题,连接的close最好放在finally里面,不然很多连接打开后会由于执行出错导致连接关闭不了.

解决方案 »

  1.   

    这段代码也太粗糙了,首先建议你在数据库操作部分最好try catch 下,然后跟一下看看有没有异常。
    如你所说 如果插入到数据库有错误,那么你就要针对错误做相应处理(关闭连接或其他逻辑操作)。
    用适配器Fill的方式不是很好,虽然比较简单。
      

  2.   

    我曾经因为没有关闭连接,造成数据连接100个以上.你可以查看一下连接数.多的话可能会死.
    另外服务器提供者说代码优化,页面ViewState是不是太大了?
      

  3.   

    using(sqlconnection.......)
    {
        //代码
    }
      

  4.   

    using (SqlConnection connection = new SqlConnection(Conn.ConnectionString))
            {            using (SqlCommand cmd = new SqlCommand(strSQL, connection))
                {
                    try
                    {
                        connection.Open();
                        int rows = cmd.ExecuteNonQuery();
                        return rows;
                    }
                    catch (System.Data.SqlClient.SqlException e)
                    {
                        connection.Close();
                        throw e;
                    }
                }
            }
    试试这种么