内存泄漏/代码写得不好或两者兼而有之 - 正确处置 sqlconnection？

Question

我正在调查 .net 1.1 Web 应用程序的一个问题，其中似乎有很多“内部连接致命错误”异常，可能是连接对象保持打开状态或未正确处理。 Web 服务器最终在重负载下崩溃。我检查了代码，实际上我们在 try catch finally 的所有地方都调用了 sqlconnection.close()

该项目实现了以下模式。有人可以告诉我这是否会导致内存泄漏？

aspx网页在Private Sub Page_Load中进行以下调用

Dim oDictionary As New dbDictionary
tagVal = oDictionary.GetTag(41)

，其中dbDictionary用于从DB获取sql表

Public Class dbDictionary
    Inherits DBInteractionBase

Public Function GetTag(ByVal tagId)

       'uses the  _connection connection object and Executes sql which sometimes throws a sql exception, but closes the connection in finally

        GetDictConnectionString()

         'new sqlconnection object for another sqlcommand
        Dim dictConnection As New SqlConnection
        dictConnection = _connection  'At this point the _connection is closed.

        'I think the dictConnection is a reference to _connection

        Dim cmdToExecute As SqlCommand = New SqlCommand

        ' // Use base class' connection object
        cmdToExecute.Connection = dictConnection

        Try
            ' // Open connection.
            dictConnection.Open()

            ' // Execute query.
            adapter.Fill(toReturn)


        Catch ex As Exception
            Throw New Exception("Error occured.", ex)
        Finally
            ' // Close connection.
            dictConnection.Close()
            cmdToExecute.Dispose()
            adapter.Dispose()
        End Try


    End Function

    Private Function GetDictConnectionString()


        ' initialize SqlCommand... Use base class _connection object
        cmdToExecute.Connection = _connection

        Try
            _connection.Open()
            adapter.Fill(toReturn)

        Catch ex As Exception
            Return "Error" ' Could this be the issue?  The original exception isn't released?
        Finally
            _connection.Close()
            cmdToExecute.Dispose()
            adapter.Dispose()
        End Try

End Class

这是它继承的DBInteractionBase

Public MustInherit Class DBInteractionBase
    Inherits System.Web.UI.Page

Protected _connection As SqlConnection

    Public Sub New()

         _connection = New SqlConnection
         _connection.ConnectionString = "some conn string"
            End Sub

    Public Overloads Sub Dispose()
        Dispose(True)
        GC.SuppressFinalize(Me)
    End Sub


    Protected Overridable Overloads Sub Dispose(ByVal isDisposing As Boolean)
        ' // Check to see if Dispose has already been called.
        If Not _isDisposed Then
            If isDisposing Then
                ' // Dispose managed resources.
                _connection.Dispose()
                _connection = Nothing
            End If
        End If
        _isDisposed = True
    End Sub

现在，当代码执行时，调用网页永远不会调用Dispose。我想知道 GC 是否执行过处理代码？我可能看到的另一个问题是，如果 GetDictConnectionString 有异常，它永远不会重新抛出原始的 sql 异常。这会以某种方式将 sql 对象留在内存中吗？请记住，这是一个 .NET 1.1 应用程序，并且（.NET 1 中的 GC 效率不是很高）

另外，我想知道我可以使用 perfmon 在 Web 服务器上监视什么来指示内存泄漏。我计划修改此代码，并希望有一个指示问题已解决的指标。我看到 SqlClient 中的趋势：当前 # 连接池 - 它每天稳定增长 1000（这是与进程关联的当前池数。）所以我想知道它是否应该随着会话的减少而下降。我正在查看 (\ASP.NET Apps v1.1.4322(Total)\Sessions Active) 以查看服务器负载情况。

Answer 1

此代码：

'new sqlconnection object for another sqlcommand
Dim dictConnection As New SqlConnection
dictConnection = _connection  'At this point the _connection is closed.

正在创建两个 SqlConnection（As New SqlConnection 正在创建一个），第一个永远不会被释放。这肯定是连接泄漏的根源。

您还将处置从未创建过的连接_connection（看不到它是在哪里创建的）。假设它是按 1:1 创建的，这应该不是一个大问题，但如果在 Try 块之前发生异常，则该连接也不会被释放。如果 _connection 没有为此代码以 1:1 的方式创建，那么当其他东西试图使用它时，您最终可能会得到 ObjectDisposeException 。