Question

更新：谢谢大家，代码不是问题，虽然有关SQL注入的信息很有用，我的问题是我使用的是我的数据库的旧版本，它没有相应的产品ID，所以它使用的是第一个产品。能找到。现在感觉非常愚蠢，但感谢您的建议。

我目前有以下代码：

SqlConnection connection = new SqlConnection(@"Data Source=(LocalDB)\v11.0 AttachDbFilename=C:\Users\h8005267\Desktop\Practical Project\Build\System4\System\StockControl.mdf;Integrated Security=True;Connect Timeout=30");
connection.Open();
SqlCommand cmd = new SqlCommand("SELECT * FROM Product WHERE ProductID='" + textBox3.Text + "'", connection); 
SqlDataReader re = cmd.ExecuteReader();

if (re.Read())
{
  textBox4.Text = re["ProductTitle"].ToString(); // only fills using first product in table
  textBox5.Text = re["ProductPublisherArtist"].ToString();
  comboBox1.Text = re["ProductType"].ToString();
  textBox6.Text = re["Price"].ToString();
}
else
{
  MessageBox.Show("Please enter a valid item barcode");
}
re.Close();
connection.Close();

我目前遇到的问题是虽然文本框显示按钮单击的信息，但显示的信息只是数据库中的第一行数据，而不是sql语句中textbox3对应的行

Answer 1

试试这个。避免以您的方式动态构建SQL语句。您正在打开数据库以应对SQL注入的风险。使用的参数是insead。

using (var connection = new SqlConnection("connection string"))
{
    connection.Open();
    using (var cmd = new SqlCommand("SELECT * FROM Product WHERE ProductID=@MYVALUE", connection))
    {
        cmd.Parameters.Add("@MYVALUE", SqlDbType.VarChar).Value = textBox3.Text;
        SqlDataReader re = cmd.ExecuteReader();

        if (re.Read())
        {
            textBox4.Text = re["ProductTitle"].ToString(); // only fills using first product in table
            textBox5.Text = re["ProductPublisherArtist"].ToString();
            comboBox1.Text = re["ProductType"].ToString();
            textBox6.Text = re["Price"].ToString();
        }
        else
        {
            MessageBox.Show("Please enter a valid item barcode");
        }
    }
}

Answer 2

在该行设置断点

SqlDataReader re = cmd.ExecuteReader();

并在textBox3中输入以下内容

'; DROP TABLE Product; SELECT '

'将在您的文本框中输入。现在执行你的方法并仔细阅读生成的sql命令...欢迎使用sql注入;）

@M Patel：感谢你的评论，你是完全正确的

结果将是以下SQL

SELECT * FROM Product WHERE ProductID=''; DROP TABLE Product; SELECT ''

这将允许恶意用户破坏您的数据库。

为了防止你应该使用M Patel在他的回答中提出的准备好的陈述

Answer 3

您有'" + textBox3.Text + "'"

的SQL注入问题

并且您不必像这样命名控件，您必须使用有意义的名称

您可以使用此代码

using (SqlConnection connection = new SqlConnection(@"Data Source=(LocalDB)\v11.0 AttachDbFilename=C:\Users\h8005267\Desktop\Practical Project\Build\System4\System\StockControl.mdf;Integrated Security=True;Connect Timeout=30"))
{
    connection.Open();
    SqlCommand cmd = new SqlCommand("SELECT * FROM Product WHERE ProductID=@ProductID", connection);
    cmd.Parameters.AddWithValue("@ProductID", textBox3.Text);
    SqlDataReader re = cmd.ExecuteReader();
    if (re.Read())
    {
        textBox4.Text = re.GetString(re.GetOrdinal("ProductTitle")); // only fills using first product in table
        textBox5.Text = re.GetString(re.GetOrdinal("ProductPublisherArtist"));
        comboBox1.Text = re.GetString(re.GetOrdinal("ProductType"));
        textBox6.Text = re.GetString(re.GetOrdinal("Price"));
    }
    else
    {
        MessageBox.Show("Please enter a valid item barcode");
    }
    re.Close();
}

如何在WinForm应用程序的文本框中显示特定的数据库条目

3 个答案: