接続が閉じられませんでした。接続の現在の状態は開いています。エラー

Question

私は

接続が閉じられていなかったと私にエラーを与えているtry..catchブロック内のコードのセットを持っています。それはブロックをキャッチするために行くと述べたエラーをスローします

try 
    { 
     MainEnqMkey = InsertUpdateDelete.InsertUpdateDeleteCls.InsertUpdateDelete_oracle(HidMode.Value, Convert.ToInt32(HidMKey.Value), "XXCUS.XXACL_PN_AGRMNT_MST", "MKEY", "AM", sb.ToString()); 

     if (Request.QueryString["Mode"] != "M") 
     { 
      ObjPriCon.Open(); 
      OracleCommand ObjPriCmd_seq2 = new OracleCommand("select XXCUS.XXACL_PN_AGRMNT_MST_SEQ.nextval from dual", ObjPriCon); // generating ref no here. 
      string strRefNo = Convert.ToString(ObjPriCmd_seq2.ExecuteOracleScalar()); 

      Response.Write("<script language=javascript> alert('Your Document No is : " + strRefNo.ToString() + "')</script>"); 

      OracleCommand cmd2 = new OracleCommand("Update XXCUS.XXACL_PN_AGRMNT_MST set DOC_NO = '" + strRefNo.ToString() + "' where MKEY = '" + MainEnqMkey + "'", ObjPriCon); // Updating ref no here 

      OracleCommand cmd3 = new OracleCommand("Update xxcus.xxacl_pn_farming_mst set LAST_DOC_NO = '" + txtdocno.Value + "' where project_id = '" + ddlProject.SelectedValue + "' and VILLAGE = '" + ddlVillage.SelectedValue + "' and SURVEY_AREA_7_12 = '" + ddl712.SelectedValue + "' and LEGAL_ENTITY = '" + txtCompName.Value + "' " + 
                " and TALUKA = '" + ddlTaluka.SelectedValue + "'"); // updating doc as last doc no 

      cmd2.ExecuteNonQuery(); 

      cmd3.ExecuteNonQuery(); 
      ObjPriCon.Close(); 
     } 

     ScriptManager.RegisterStartupScript(this, this.GetType(), "alert", "alert('Record Saved Successfully');window.location ='Frm_Agreement_Master.aspx?TranType=AM&PView=N&Mode=A&Redirect=oracle&Key=0&Redirect=" + Request.QueryString["Redirect"] + "&userid=" + Request.QueryString["userid"].ToString() + "';", true); 
    } 
    catch (Exception ex) 
    { 
     ex.Message.ToString(); 
    } 

ObjPriCon.Open();した後、接続の現在の状態が開かれています。

Answer 1

代わりのオープン接続を開き、あなたはそれがあなたの接続状態を使用してのハック方法でこれを修正することができますが、一つの接続を共有するために良いことではありません

if (ObjPriCon.State != ConnectionState.Open) 
{ 
    ObjPriCon.Open(); 
} 

Answer 2

のような状態だ確認することができます。接続プールはあなたの友人です。だから、再コードを書くことができます。

using(OracleConnection conn = new OracleConnection("connectionString")) 
{ 
    conn.Open(); 
    //your code 
    //using block will automatically close your connection when you are done. 
} 

はまた、現在、あなたはSQLインジェクションに開放されています。使用は

OracleCommand cmd2 = new OracleCommand("Update XXCUS.XXACL_PN_AGRMNT_MST set DOC_NO = @Doc_No where MKEY =@MKey , conn); 

cmd2.Parameters.AddWithValue("@Doc_No", strRefNo.ToString()); 
cmd2.Parameters.AddWithValue("@Mkey", MainEnqMkey); 

cmd2.ExecuteNonQuery(); 

はあなたのクエリと同じ順序でそれらを定義する必要がありますので、OracleCommandのは、名前付きパラメータをサポートしていません注意してくださいクエリをparamatrized。

Answer 3

ObjPriCon.Close（）が必要なことがあります。最終的にブロックに入れます。それがキャッチになると、それは閉じられていない可能性があります。あなたは、そのヒンジを、それをはぎ取るよ -

Answer 4

より良いあなたはそれがすでに開いている場合は、接続を開くことができませんObjPriCon.close()

try 
{ 
    ObjPriCon.Open(); 

}  
catch (Exception ex) 
{ 
    throw ex; 
} 
finally 
{ 
    ObjPriCon.close(); 
}