連接未關閉。連接的當前狀態已打開。 C＃錯誤

Question

從gridview保存2個或更多值時出現此錯誤：

連接未關閉。連接的當前狀態是開放的

但是該過程經過並保存並更新數據。我怎樣才能消除這個錯誤？

這裏是我的代碼：

for(int i = 0; i < gvModal.Rows.Count; i++) 
{ 
    string dateA = DateTime.Now.ToString("yyyy-MM-dd"); 

    Utility u = new Utility(); 
    string conn = u.connect(); 

    Label type = (Label)gvModal.Rows[i].Cells[1].FindControl("lbltype"); 
    Label model = (Label)gvModal.Rows[i].Cells[2].FindControl("lblModel"); 
    Label quantity = (Label)gvModal.Rows[i].Cells[3].FindControl("lblQuan"); 
    Label unit = (Label)gvModal.Rows[i].Cells[4].FindControl("lblUnit"); 
    int bal = Convert.ToInt32(gvModal.Rows[i].Cells[4].Text); 
    int forIssue = 0; 
    int forPO = 0; 

    if (bal != 0) 
    { 
     forIssue = 1; 
     forPO = 0; 
    } 
    else 
    { 
     forIssue = 0; 
     forPO = 1; 
    } 

    SqlConnection connUser = new SqlConnection(conn); 
    SqlCommand read = connUser.CreateCommand(); 

    string query = "INSERT INTO Mosef_Alert values (@Mosef_No, @Branch, @BU, @Dept, @Section, @Requisitioner, @Accepted, @Date_Accepted, @Reason, @MOSEF_Date, @type, @model, @quantity, @unit)"; 

    connUser.Open(); 
    read.CommandText = query; 

    read.Parameters.Add(new SqlParameter("@Mosef_No", transIDs)); 
    read.Parameters.Add(new SqlParameter("@Branch", branch)); 
    read.Parameters.Add(new SqlParameter("@BU", bu)); 
    read.Parameters.Add(new SqlParameter("@Dept", dept)); 
    read.Parameters.Add(new SqlParameter("@Section", sec)); 
    read.Parameters.Add(new SqlParameter("@Requisitioner", requisitioner)); 
    read.Parameters.Add(new SqlParameter("@Accepted", accept)); 
    read.Parameters.Add(new SqlParameter("@Date_Accepted", dateA)); 
    read.Parameters.Add(new SqlParameter("@Reason", reason)); 
    read.Parameters.Add(new SqlParameter("@MOSEF_Date", lblDateFiled.Text)); 
    read.Parameters.Add(new SqlParameter("@type", type.Text)); 
    read.Parameters.Add(new SqlParameter("@model", model.Text)); 
    read.Parameters.Add(new SqlParameter("@quantity", quantity.Text)); 
    read.Parameters.Add(new SqlParameter("@unit", unit.Text)); 
    read.Parameters.Add(new SqlParameter("@For_PO", forPO)); 
    read.Parameters.Add(new SqlParameter("@For_Issuance", forIssue)); 

    read.ExecuteNonQuery(); 

    read.Parameters.Clear(); 
} 

ExecuteUpdate(accept); 
UpdateStatus(); 

System.Text.StringBuilder sb = new System.Text.StringBuilder(); 
sb.Append(@"<script type ='text/javascript'>"); 
sb.Append("alert('Records Updated');"); 
sb.Append("$('#editModal').modal('hide');"); 
sb.Append(@"</script>"); 
ScriptManager.RegisterClientScriptBlock(this, this.GetType(), "EditHideModalScript", sb.ToString(), false); 
} 

public void UpdateStatus() 
{ 
     Utility u = new Utility(); 
     string conn = u.connect(); 
     SqlConnection connUser = new SqlConnection(conn); 
     SqlCommand read = connUser.CreateCommand(); 

     for(int i = 0; i < gvModal.Rows.Count; i++) 
     { 
      Label ItemID = (Label)gvModal.Rows[i].Cells[1].FindControl("lblID"); 
      Label stat = (Label)gvModal.Rows[i].Cells[8].FindControl("ItemStatus"); 
      int balance = Convert.ToInt32(gvModal.Rows[i].Cells[4].Text); 
      string status; 

      if(balance != 0) 
      { 
       status = "For Issuance"; 
      } 
      else 
      { 
       status = "For PO"; 
      } 

      string upd = "UPDATE ItemTransaction SET ItemStatus = '" + status +"' WHERE ID = '"+ ItemID.Text +"'"; 

      connUser.Open(); 
      read.CommandText = upd; 

      read.Parameters.Clear(); 
      read.ExecuteNonQuery(); 
     } 
    } 
    public void ExecuteUpdate(int stat) 
    { 
     string upStat = null; 
     if (stat == 1) 
     { 
      upStat = "Accepted"; 
     } 
     else 
     { 
      upStat = "Denied"; 
     } 
     string id = transID.Text; 
     Utility u = new Utility(); 
     string conn = u.connect(); 
     SqlConnection connUser = new SqlConnection(conn); 
     string up = "UPDATE MosefTransaction SET TransStatus = '"+ upStat +"' WHERE TransactionID = '"+ id +"'"; 
     connUser.Open(); 
     SqlCommand cm = new SqlCommand(up, connUser); 
     //cm.Parameters.AddWithValue("@ID", id); 
     //cm.Parameters.AddWithValue("@TransStatus", upStat); 
     cm.Parameters.Clear(); 
     cm.ExecuteNonQuery(); 

     connUser.Close(); 
    } 

Answer 1

你必須注意到的第一件事情，你的純文本查詢打開一個寬門SqlInjection。所以使用參數化查詢。現在讓我來看看你的代碼， 問題出在UpdateStatus方法，在這種方法中，你在迭代過程中打開了連接，並且不關閉它，所以當你在下一次迭代中再次打開連接時，它會拋出錯誤。您可以通過多種方式避免這種情況：

1. 使用connUser.Close()
2. 您可以使用ConnectionState枚舉檢查打開一個新的連接前的連接狀態關閉在每次迭代的連接。並僅在狀態未打開時纔打開它。

這可以通過使用下面的代碼來完成：

if (connUser.State != ConnectionState.Open) 
    connUser.Open(); 

3.打開循環外部連接，並使用相同通過時的循環。執行查詢後清除每次迭代中的參數。

例如考慮代碼：

using (SqlConnection connUser = new SqlConnection(conn)) 
{ 
    string upd = "UPDATE ItemTransaction SET ItemStatus = @status WHERE ID = @id"; 
    connUser.Open(); 
    SqlCommand commandSQL = connUser.CreateCommand(); 
    for (int i = 0; i < gvModal.Rows.Count; i++) 
    { 
     // Get values here using your code 
     commandSQL.Parameters.Add("@status", SqlDbType.VarChar).Value = status; 
     commandSQL.Parameters.Add("@id", SqlDbType.VarChar).Value = ItemID.Text; 
     commandSQL.ExecuteNonQuery(); 
     commandSQL.Parameters.Clear();     
    } 
} 

注：最好的辦法是第三個，我喜歡你遵循，其餘的音符你的知識，這將有助於你在其他情況;