C# 中的 ADO.NET 对象无法正常工作

Question

SqlConnection con = new SqlConnection("Data Source=.\\SQLEXPRESS;AttachDbFilename=|DataDirectory|\\CustomersDB.mdf;Integrated Security=True;User Instance=True");

SqlCommand cmd = new SqlCommand("INSERT INTO Customers (ID, Date, GUIA, SName, SAddress, SCity, SState, SZipCode, SPhone, SEmail, RName, RAddress, RCity, RState, RZipCode, RPhone, REmail) VALUES (1,'"+textBox1.Text + "','" + textBox2.Text+"','" + textBox3.Text + "','" + textBox4.Text + "','" + textBox5.Text + "','" + textBox6.Text + "','" + textBox7.Text + "','" + textBox8.Text + "','" + textBox9.Text + "','" + textBox16.Text + "','" + textBox15.Text + "','" + textBox14.Text + "','" + textBox13.Text + "','" + textBox12.Text + "','" + textBox11.Text + "','" + textBox10.Text +"')" , con);
cmd.CommandType = System.Data.CommandType.Text;

con.Open();
cmd.ExecuteNonQuery();
con.Close();

MessageBox.Show("Data added successfully!");

如您所见，我正在尝试向数据库中添加一些数据，这些数据是在 C# Windows 窗体应用程序中创建的。

但是，执行代码后，我没有收到任何错误，但是当我查看表格数据时，没有任何变化。

换句话说，即使代码执行正确，也不会添加任何数据。

这里有什么缺陷？任何帮助表示赞赏。

Answer 1

首先，我想指出你有一个巨大的 SQL 注入。其次，看看Rows not being updated，看看是不是和你一样的问题。

Answer 2

主要缺陷是整个 User Instance 和 AttachDbFileName= 方法。 Visual Studio 将复制 .mdf 文件，很可能，您的 INSERT 工作正常 - 但您只是查看 错误的 .mdf 文件结束！

如果您想坚持使用这种方法，请尝试在 myConnection.Close() 调用上设置断点 - 然后使用 SQL Server Mgmt Studio Express 检查 .mdf 文件 - 我几乎可以肯定您的数据在那里。

在我看来，真正的解决方案是

1. 安装 SQL Server Express（反正你已经完成了）

2. 安装 SQL Server Management Studio Express

3. 在 SSMS Express 中创建您的数据库，并为其命名（例如 CustomersDB）

4. 使用它的逻辑数据库名称（在服务器上创建它时给出）连接到它——不要乱用物理数据库文件和用户实例。在这种情况下，您的连接字符串将类似于：

   Data Source=.\\SQLEXPRESS;Database=CustomersDB;Integrated Security=True
   

   其他一切都一模一样和以前一样......

Answer 3

1 您的查询将创建SQL Injection，请尝试使用 SP 或 LINQ 以更安全地执行。

[2] 首先尝试用你的 sql server 数据库表执行你的长查询返回字符串值，因为这里你没有显示你的表结构，所以任何单引号都不会执行正确的查询。

string sqlstr = "INSERT INTO Customers (ID, Date, GUIA, SName, SAddress, SCity, SState, SZipCode, SPhone, SEmail, RName, RAddress, RCity, RState, RZipCode, RPhone, REmail) VALUES (1,'"+textBox1.Text + "','" + textBox2.Text+"','" + textBox3.Text + "','" + textBox4.Text + "','" + textBox5.Text + "','" + textBox6.Text + "','" + textBox7.Text + "','" + textBox8.Text + "','" + textBox9.Text + "','" + textBox16.Text + "','" + textBox15.Text + "','" + textBox14.Text + "','" + textBox13.Text + "','" + textBox12.Text + "','" + textBox11.Text + "','" + textBox10.Text +"')"

[3] 最后一点更好的命名对编码很重要。

Answer 4

cn.ConnectionString = @"Data Source=.\\SQLEXPRESS;AttachDbFilename=|DataDirectory|\\CustomersDB.mdf;Integrated Security=True;User Instance=True";
            cn.Open();
            SqlCommand com = new SqlCommand();
            com.Connection = cn;
            com.CommandType = CommandType.Text;

            com.CommandText = "INSERT INTO Customers (ID, Date, GUIA, SName, SAddress,               SCity, SState, SZipCode, SPhone, SEmail, RName, RAddress, RCity, RState, RZipCode, RPhone, REmail) VALUES (1,'"+textBox1.Text + "','" + textBox2.Text+"','" + textBox3.Text + "','" + textBox4.Text + "','" + textBox5.Text + "','" + textBox6.Text + "','" + textBox7.Text + "','" + textBox8.Text + "','" + textBox9.Text + "','" + textBox16.Text + "','" + textBox15.Text + "','" + textBox14.Text + "','" + textBox13.Text + "','" + textBox12.Text + "','" + textBox11.Text + "','" + textBox10.Text +"')" ;
          com.ExecuteNonQuery();
          MessageBox.Show("Saving is done!");

试试这个代码，看看它是否工作我认为这应该工作.. ;)