1

有人可以告诉我我做错了什么吗?我试图在页面加载时自动调用该方法,但是它不起作用。

protected void Page_Load(object sender, EventArgs e)
{
    if (!Page.IsPostBack)
    {
        customerInformation();

    }
}

protected void ddNames_SelectedIndexChanged(object sender, EventArgs e)
{
    customerInformation();
}

private void customerInformation()
{
    string dbString = ConfigurationManager.ConnectionStrings["TechSupportDBConString"].ConnectionString;
    string query = "SELECT * FROM Customers WHERE Name='" + ddNames.Text + "'";

    SqlConnection Connection = new SqlConnection(dbString);
    Connection.Open();

    SqlCommand Com = new SqlCommand(query, Connection);
    SqlDataReader reader = Com.ExecuteReader();
    if (reader.Read())
    {
        lblName.Text = reader["Name"].ToString() + "'s Personal Information";
        lblAddress.Text = reader["Address"].ToString() + "\n" + reader["City"].ToString() + " " + reader["State"].ToString() + " " + reader["ZipCode"].ToString();
        lblPhone.Text = reader["Phone"].ToString();
        lblEmail.Text = reader["Email"].ToString();

        reader.Close();
        Connection.Close();
    }
}
4

1 回答 1

2

好的,通过您的代码,我可以看到阅读器和数据库的连接本身是封闭的,不应该这样做。

SqlDataReader reader = command.ExecuteReader();    
if (reader.HasRows)
{
     while (reader.Read())
     {
                //populate fields from reader
     }
}
else
{
     lbl.Text = "No records found.";
}
reader.Close();
Connection.Close();

接下来我注意到下拉选择的文本/值应该是ddNames.SelectedItem.Text或者在值传递的情况下ddNames.SelectedValue

接下来确保每个阅读器名称应与表列名称完全匹配。

最后但并非最不重要的一点是,在应用之前始终进行适当的 null 检查。ToString()如果任何列具有 null 值ToString()将无法转换。

希望这可以帮助。

于 2016-03-02T09:08:42.540 回答