求大哥给我看看怎么写这段程序

[itit54]

下面是完整的数据查询语言，但是我发现执行的效率很低，希望高手帮我看看有没有其他高效率的解决办法？
希望给出实例代码，谢谢~~~


SqlConnection con = new SqlConnection("server=911COM\\SQLEXPRESS;uid=p2p;pwd=123456;database=p2p");
con.Open();
string cmdtxt="insert into users(yonghuming,pwd,email,tuijiangren) values('" + TextBox1.Text + "','"+ TextBox2.Text +" ','" + TextBox3.Text + "','" + TextBox5.Text + "')";
SqlCommand cmd = new SqlCommand(cmdtxt, con);
int count = Convert.ToInt32(cmd.ExecuteScalar());
cmd.CommandText = "select id,yonghuming from users where yonghuming='" + TextBox1.Text + "'";
cmd.ExecuteNonQuery();
SqlDataReader sdr = cmd.ExecuteReader();
sdr.Read();
string id = (sdr["id"]).ToString();
sdr.Close();
cmd.CommandText = "insert into checkmail(users_id,keys) values(" + id + ",'" + srtHtmlTemplate4 + "')";
cmd.ExecuteNonQuery();
con.Close();

[king198866]

不会提高效率改写，帮顶

[追寻风的自由]

提高效率不会，不过要小心SQL注入

[itit54]

up

[wuyq11]

使用数据库操作类sqlhelper
调用存储过程，批操作

using(SqlConnection con = new SqlConnection(""))
{
con.Open();


con.Close();
}

[maxiaoyuanXM]

改成存储过程，多条一起执行，效率肯定高些的，
再者，返回插入ID的时候，用这个Select @@Identity不更好嘛

[kssys]

你的代码需要三次往返，其中的第二次和第三次可以节省为1次。另建议用SQL参数的形式插入资料，否则会有安全性问题。以下为代码：
SqlConnection con = new SqlConnection("server=911COM\\SQLEXPRESS;uid=p2p;pwd=123456;database=p2p");
con.Open();
string cmdtxt="insert into users(yonghuming,pwd,email,tuijiangren) values('" + TextBox1.Text + "','"+ TextBox2.Text +" ','" + TextBox3.Text + "','" + TextBox5.Text + "')";
SqlCommand cmd = new SqlCommand(cmdtxt, con);
cmd.ExecuteNonQuery();
int count = Convert.ToInt32(cmd.ExecuteScalar());
cmd.CommandText = "select id,yonghuming from users where yonghuming='" + TextBox1.Text + "'";
cmd.ExecuteNonQuery();
SqlDataReader sdr = cmd.ExecuteReader();
sdr.Read();
string id = (sdr["id"]).ToString();
sdr.Close();
cmd.CommandText = "insert into checkmail(users_id,keys) values(" + id + ",'" + srtHtmlTemplate4 + "')";
cmd.ExecuteNonQuery();
con.Close();
cmd.CommandText="insert into checkmail(users_id,keys)"
cmd.CommandText+="select top 1 id,'" + srtHtmlTemplate4 + "' ";
cmd.CommandText+="FROM users ";
cmd.CommandText+="where yonghuming='" + TextBox1.Text + "'";
cmd.ExecuteNonQuery();

[kssys]

con.close();

[itit54]

非常感谢kssys对我等菜鸟的照顾，非常感谢

[xugan666]

111