可能的连接泄漏导致“System.Data.SqlClient.SqlException：超时已过期” SQL Server 中出现错误？

Question

我的应用程序要求用户登录并允许他们编辑事物列表。但是，似乎如果同一用户总是登录和注销并编辑列表，则该用户将遇到“System.Data.SqlClient.SqlException：超时已过期”。错误。我读过有关它的评论，可能是由未提交的事务引起的。我确实有一个正在申请中。

我将提供我正在使用的代码，其中有一个 IF 语句，我对此有点犹豫，但这似乎是一个合理的事情。

我将回顾一下这里发生的情况，有一个要更新或添加到数据库中的对象列表。应用程序中创建的新对象的 ID 为 0，而现有对象则拥有从数据库生成的自己的 ID。如果用户选择删除某些对象，它们的 ID 将存储在单独的整数列表中。一旦用户准备好保存更改，两个列表就会传递到此方法中。通过使用 IF 语句，添加 ID 为 0 的对象（使用 Add 存储过程），并更新那些 ID 非零的对象（使用 Update 存储过程）。之后，FOR 循环将遍历“删除”列表中的所有整数，并使用删除存储过程来删除它们。所有这一切都使用事务。

Public Shared Sub UpdateSomethings(ByVal SomethingList As List(Of Something), ByVal RemovalList As List(Of Integer))
Using DBConnection As New SqlConnection(conn)
    DBConnection.Open()
    Dim MyTransaction As SqlTransaction
    MyTransaction = DBConnection.BeginTransaction()
    Try
        Using MyCommand As New SqlCommand()
            MyCommand.Transaction = MyTransaction
            MyCommand.CommandType = CommandType.StoredProcedure

            For Each SomethingItem As Something In SomethingList
                MyCommand.Connection = DBConnection
                If SomethingItem.ID > 0 Then
                    MyCommand.CommandText = "UpdateSomething"
                Else
                    MyCommand.CommandText = "AddSomething"
                End If
                MyCommand.Parameters.Clear()
                With MyCommand.Parameters
                    If MyCommand.CommandText = "UpdateSomething" Then
                        .Add("@id", SqlDbType.Int).Value = SomethingItem.ID
                    End If
                    .Add("@stuff", SqlDbType.Varchar).Value = SomethingItem.Stuff
                End With
                MyCommand.ExecuteNonQuery()
            Next

            MyCommand.CommandText = "DeleteSomething"
            For Each ID As Integer In RemovalList
                MyCommand.Parameters.Clear()
                With MyCommand.Parameters
                    .Add("@id", SqlDbType.Int).Value = ID
                End With
                MyCommand.ExecuteNonQuery()

            Next
        End Using
        MyTransaction.Commit()
    Catch ex As Exception
        MyTransaction.Rollback()
        'Exception handling goes here   '
    End Try

End Using
End Sub

这里使用了三个存储过程以及一些循环，因此我可以看到如果列表足够大，某些东西如何能够支撑一切。

我使用 Visual Studio 2008 进行调试，并使用 SQL Server 2000 作为数据库。

编辑：我似乎仍然收到此错误。我什至已经删除了整个交易的事情，但我仍然遇到它。此时，我假设这里发生了某种泄漏。我尝试不使用 USING 语句并明确告诉命令和连接自行处置，但没有骰子。如果短时间内多次调用此方法，SQL Server 的内存使用量也会增加很多。

我读到增加 SQLCommand 的 CommandTimeout 属性会有所帮助。我想知道这样做是否有什么大的缺点或后果。

Answer 1

如果您有大量命令，您可能需要在打开连接之前构建所有命令。启动事务并打开连接后，旋转并执行它们。

您可能希望使用 TransactionScope

Using _tx as New System.Transactions.TransactionScope(<add your own timeout here>)

 'Do all your sql work' 

 If _noErrors Then
  _tx.Complete()
 End If

End Using 

与事务范围，您可以设置最多 20 分钟的超时，而无需修改服务器设置。

Answer 2

我相信我已经成功解决了这个问题。我修改了应用程序，以便不会对数据库进行不必要的调用（即，未更改的对象不需要再次更新），并增加了 SQLCommand 对象的 CommandTimeout 属性。到目前为止，没有问题。

也非常感谢您的建议。

Answer 3

我建议使用以下内容，这样 Dispose 将始终被调用并在每个未提交的情况下回滚。

using (SqlConnection sqlCn = new SqlConnection())
{
   using (SqlTransaction myTrans = sqlCn.BeginTransaction())
   {
   ...
   myTrans.Commit();
   }
}

另外，我不认为您需要为每次执行创建一个新的 SqlCommand。只需保持相同并更新 CommandText 和参数即可。