Question

在写ASP.NET时出现这样的错误：

代码： string settings = Convert.ToString(ConfigurationManager.ConnectionStrings["MySqlConnection"]);
//创建数据库连接
SqlConnection myconn = new SqlConnection(settings);
//打开数据库连接
myconn.Open();
string n = TextBox1.Text.Trim();
string p = TextBox2.Text.Trim();
string sql;
int i = 2;
sql = "insert into products(userName,userPwd,RoleID)";
sql = sql + "values (" + "'" + n + "'," + "'" + p + "','"+i + "')";
SqlCommand mycmd = new SqlCommand(sql, myconn);
mycmd.ExecuteNonQuery();
myconn.Close();
Response.Redirect("logo.aspx");

Answer 1

values后面的引号有问题，写得也不规范，这样不容易查错。

追问

那要怎么写呢。

追答

你这种插入数据的写法很特别啊，我们都是这么写的:
sql="insert into products (userName,userPwd,RoleID) values(' "+n+" ',' "+p+" ',' +i+')";
如果你嫌语句太长的话，可以用StringBuilder类的Append（）方法拼接sql语句.

追问

这样写了。又出现这样的错误了。

追答

呵呵 忘记了，你前面定义的n和p以及i都是变量，在字符串（这里是指sql）里应用这些变量，系统只会把他们也当成字符串，这种数据插入的方式是错误的。你直接把两个文本框的值
插进去皆可以了。不过，我还是建议你不要把操作数据库的代码放到cs文件里，这样代码不利于维护（当然，小型的功能就不必些三层了）。
改后是这样的：
sql="insert into products (userName,userPwd,RoleID) values(' "+TextBox1.Text.Trim()+" ',' "+TextBox2.Text.Trim()+" ',' 2 ')";
这次应该不错了，你再试下。

追问

可能别处有问题。还是列名无效。呜呜。

Answer 2

应该是字符串拼接有问题