InvalidOperationException Kết nối không được đóng. Trạng thái hiện tại của kết nối đang mở

Question

Tại sao mã này ném một ngoại lệ hoạt động không hợp lệ?

private SqlCommand cmd; // initialized in the class constructor 

public void End(string spSendEventNotificationEmail) { 
    try { 
    cmd.CommandText = spSendEventNotificationEmail; 
    cmd.Parameters.Clear(); 
    cmd.Parameters.Add("@packetID", SqlDbType.Int).Value = _packetID; 
    cmd.Parameters.Add("@statusID", SqlDbType.Int).Value = _statusID; 
    cmd.Parameters.Add("@website", SqlDbType.NVarChar, 100).Value = Tools.NextStep; 
    cmd.Connection.Open(); 
    cmd.ExecuteNonQuery(); 
    } finally { 
    cmd.Connection.Close(); 
    cmd.Parameters.Clear(); 
    cmd.Dispose(); 
    } 
    endCall = true; 
} 

Answer 1

Bạn đang cố gắng để mở một kết nối đã được mở, kết quả này trong ngoại lệ.

Giải pháp 1 (đề nghị):

Kiểm tra mã của bạn, rà soát tất cả các phần nơi cmd.Connection kết nối được mở ra và đảm bảo rằng nó luôn luôn đóng đúng cách.

Giải pháp 2 (quick'n'dirty sửa chữa):

trước dòng

cmd.Connection.Open(); 

thêm mã kiểm tra/dọn dẹp sau:

if (cmd.Connection.State == ConnectionState.Open) 
{ 
    cmd.Connection.Close(); 
} 

Answer 2

Có rất ít nhu cầu giữ các đối tượng Sql * ở cấp lớp, đặc biệt dựa trên những gì bạn đang hiển thị. Bạn cũng sẽ mất những lợi ích của việc kết nối tổng hợp bằng cách cố gắng tự làm điều đó.

Với phương pháp này, bạn loại bỏ khả năng lỗi của bạn vì bạn không chia sẻ bất kỳ đối tượng

private readonly _connectionString = "..."; 

public void End(string spSendEventNotificationEmail) { 
    using(var conn = new SqlConnection(_connectionString)) 
    using(var cmd = conn.CreateCommand()) 
    { 
    cmd.CommandText = spSendEventNotificationEmail; 
    cmd.Parameters.Add("@packetID", SqlDbType.Int).Value = _packetID; 
    cmd.Parameters.Add("@statusID", SqlDbType.Int).Value = _statusID; 
    cmd.Parameters.Add("@website", SqlDbType.NVarChar, 100).Value = Tools.NextStep; 
    conn.Open(); 
    cmd.ExecuteNonQuery(); 
    } 
    endCall = true; 
}