一个菜鸟请问：为什么总是执行到catch这一句,高分相求

[joyball]

下面一个小程序，
private void btnAccept_Click(object sender, System.EventArgs e)
{
if(Page.IsValid)
{
//save new user to the database
SqlConnection con;
string sql;
SqlCommand cmd;
StringBuilder sb=new StringBuilder();
ArrayList values=new ArrayList();

sb.Append("INSERT INTO[User]");
sb.Append("(UserID,Login,Password,FirstName,LastName,");
sb.Append("CellNumber,DateOfBirth)");
sb.Append("VALUES('{0}','{1}','{2}','{3}','{4}','{5}','{6}','{7}',");
sb.Append("{8},{9},{10})");

//add reqired values to replace
values.Add(Guid.NewGuid().ToString());
values.Add(txtLogin.Text);
values.Add(txtPwd.Text);
values.Add(txtFName.Text);
values.Add(txtLName.Text);
values.Add(txtPhone.Text);
values.Add(txtEmail.Text);
values.Add(0);

//add the optional values or Null
if(txtAddress.Text!=string.Empty) values.Add("'"+txtAddress.Text+"'");
else values.Add("NULL");

if(txtMobile.Text!=string.Empty) values.Add("'"+txtMobile.Text+"'");
else values.Add("NULL");

if(txtBirth.Text!=string.Empty) values.Add("'"+txtBirth.Text+"'");
else values.Add("NULL");

//Format the string with the array of values
sql=String.Format(sb.ToString(),values.ToArray());

//Connect and execute the query
con=new SqlConnection("data source=Joyball;initial catalog=Friends;user id=sa");
cmd=new SqlCommand(sql,con);
con.Open();

bool doredirect=true;

try
{
cmd.ExecuteNonQuery();
}
catch
{
doredirect=false;
this.lblMessage.Visible=true;
this.lblMessage.Text="Insert couldn't be performed,User name may be already taken.";
}
finally
{
con.Close();
}
if(doredirect)
Response.Redirect("Login.aspx");
}//end if
else
{
lblMessage.Text="FIx thefollowing errors and retry:";
}//end else

}
执行过之后，总是出现“Insert couldn't be performed,User name may be already taken.”，而我的数据库中什么数据都没有，请教各位高人

[looner]

是啊，字段和數據的個數怎麼不同呢

[KentYu]

是啊：）改一下sql语句

[joyball]

大哥，如何改阿？？？？？

[joyball]

我真的不会阿

[tonybaobao]

呵呵，估计代码不是你自己写的吧！否则怎么连需求都不知道。你自己设了10个参数，当然应该知道传什么进去的。

[joyball]

hehe,真不是我写的，我哪里会写这么长的东西啊，我照着书上抄的，刚刚开始学习。请各位大哥帮忙

[jin_xiaolu2000]

sb.Append("INSERT INTO[User]");
sb.Append("(UserID,Login,Password,FirstName,LastName,");
sb.Append("CellNumber,DateOfBirth)");//加上你的其他字段2个
sb.Append("VALUES('{0}','{1}','{2}','{3}','{4}','{5}','{6}','{7}',");
sb.Append("{8},{9},{10})");

===================================================================
sb.Append("INSERT INTO[User]");
//要不就这样 自己对齐全部字段，必须是10个才行，
sb.Append("VALUES('{0}','{1}','{2}','{3}','{4}','{5}','{6}','{7}',");
sb.Append("{8},{9},{10})");

这个问题，报错是很详细的啊

[looner]

沒用過sql server，一直在用oracle
要麼你用標準sql來寫吧，看著暈暈：）
還有，你的sql幹嘛不一次寫完，非要Append那麼多次呢？

[wangxt]

帮你顶

[lr2651]

哎，尽信书则不如无书啊！！！

[bluesage]

呵呵。。还是学好好学SQL吧！

[amendajing]

有错误当然被catch住了呀，你的sql写错了，看看sql帮助吧

[lucbesson]

看书去 我晕

[marvelstack]

使用参数化查询或存储过程代替字符的组合。

[marvelstack]

//执行存储过程,PUB_SYS_LOG为存储过程名
public static void InsertGMonitorLog(int sysPrgId,string logType,ing logOperationMsg)
{
SqlConnection myConn = new SqlConnection(strConn);
SqlCommand myComm = new SqlCommand("PUB_SYS_LOG",myConn);
myComm.CommandType = CommandType.StoredProcedure;

SqlParameter myPara = new SqlParameter("@SYS_PRG_ID",SqlDbType.Int);
myPara.Value = sysPrgId;
myComm.Parameters.Add(myPara);

myPara = new SqlParameter("@LOG_TYPE",SqlDbType.VarChar,3);
myPara.Value = logType;
myComm.Parameters.Add(myPara);

myPara = new SqlParameter("@LOG_OPERATION_MSG",SqlDbType.VarChar,50);
myPara.Value = logOperationMsg;
myComm.Parameters.Add(myPara);

try
{
myConn.Open();
myComm.ExecuteNonQuery();
}
catch(SqlException e)
{
e.ToString();
}
finally
{
myConn.Close();
}
}

[WillSmart]

顶吧．．．

[xiaoslong]

帮你顶

[gbbword]

看这个变量名字，真好笑。楼主下会别用这个变量名字了。容易让人误会。

"INSERT INTO[User]
(UserID,Login,Password,FirstName,LastName,CellNumber,DateOfBirth,Lost1,lost2,lost3)
VALUES('1','2','3','4','5','6','7','8','9','10')

[joyball]

真得很不好意思，我真地在努力。

[an_andy]

该说的都说拉...
帮你顶