WPF C# 来回移动 b/w 时出错 两个组合框项目选择

Question

我有 02 组合框，如图所示 [组合框] 当我来回移动以选择黑白组合框（类别和型号）时，出现以下错误下面给出了我的 C# 清晰代码        try        {            con.Open();            string CmdString = "select ProductID from Product where ModelNo='" + comboModel.SelectedItem.ToString() + "'";            SqlCommand cmd = new SqlCommand(CmdString, con);            SqlDataAdapter sda = new SqlDataAdapter(cmd);            DataTable dt1 = new DataTable("Product");            sda.Fill(dt1);            foreach (DataRow dr in dt1.Rows)            {                txtProductID.Text = dr["ProductID"].ToString();            }            con.Close();        }        catch (Exception exp)        {            MessageBox.Show(exp.ToString());            con.Close();        }

Answer 1

您的连接已在其他地方打开。尝试将其放入 using 块，这样它将被自动处理并且是连接的最佳实践：

using (SqlConnection connection = new SqlConnection(connectionString))

{

    connection.Open();

    string CmdString = "select ProductID from Product where ModelNo='" + 

    comboModel.SelectedItem.ToString() + "'";

    SqlCommand cmd = new SqlCommand(CmdString, connection);

    SqlDataAdapter sda = new SqlDataAdapter(cmd);

    DataTable dt1 = new DataTable("Product");

    sda.Fill(dt1);

    foreach (DataRow dr in dt1.Rows)

    {

        txtProductID.Text = dr["ProductID"].ToString();

    }

}

Answer 2

摆脱con.Open();. 显然，连接已经从其他地方的较早操作或此代码的先前运行中打开。

话虽如此，除非您需要它们，否则您可能不应该保持连接打开，因为您冒着它们从未正确关闭的风险。您应该使用一次性模式，如下所示：

using (var con = new SqlConnection(myConnStr))

{

    // do your query, etc.

}

这将自动关闭连接并处理资源。