急！向数据库中插入数据出错

Question

protected void SureBtn_Click(object sender, EventArgs e)
    {
        SqlConnection con = new SqlConnection(ConfigurationManager.AppSettings["conn"]);
        con.Open();
        string sql = "insert into store (name,number,amount) values('"+name+"','"+number+"','"+amount+"')";
        SqlCommand cmd = new SqlCommand(sql,con);
        try
        {
            cmd.ExecuteNonQuery;
            Response.Write("<script language=javascript>alert('" + 添加数据成功 + "')</script>");              con.Close();
        }
调试后出错，什么原因啊？

Answer 1

cmd.ExecuteNonQuery()这是方法不是属性; 

Answer 2

cmd.ExecuteNonQuery;
改成
cmd.ExecuteNonQuery();

Answer 3

name是关键字，对于关键字应该加上[]

Answer 4

还有就是
string sql = "insert into store (name,number,amount) values('"+name+"','"+number+"','"+amount+"')"; 
这种写法很容易出问题的，而且也不安全，改成输入参数的。否则单引号替换有的时候很头疼，很容易出错的

Answer 5

呵呵。你这个代码怎么编译通过的？我很好奇。还有不建议你那样拼接SQL语句，使用SqlPara...会确保数据类型的正确性。

Answer 6

protected void SureBtn_Click(object sender, EventArgs e) 
    { 
        SqlConnection con = new SqlConnection(ConfigurationManager.AppSettings["conn"]); 
        con.Open(); 
        string sql = "insert into store ([name],[number],[amount]) values('"+name+"','"+number+"','"+amount+"')"; 
        SqlCommand cmd = new SqlCommand(sql,con); 
        try 
        { 
            cmd.ExecuteNonQuery(); 
            Response.Write(" <script language=javascript>alert('" + 添加数据成功 + "') </script>");              con.Close(); 
        } 

Answer 7

name+"','"+number+"','"+amount+"')"; 
你的这几个值在哪里定义的？

Answer 8

protected void SureBtn_Click(object sender, EventArgs e) 
    { 
        SqlConnection con = new SqlConnection(ConfigurationManager.AppSettings["conn"]); 
        con.Open(); 
        string sql = "insert into store ([name],[number],[amount]) values('"+name+"','"+number+"','"+amount+"')"; 
        SqlCommand cmd = new SqlCommand(sql,con); 
        try 
        { 
            cmd.ExecuteNonQuery(); 
            Response.Write(" <script language=javascript>alert('" + 添加数据成功 + "') </script>");              con.Close(); 
        }