关于事务,我这么写对不对.

[blackhero]

public static bool ExcuteNonQueryTrans(string[] sqls, params SqlParameter[][] pas)
{
bool success=true;
string strConn=System.Configuration.ConfigurationSettings.AppSettings["connstr"].ToString();
SqlConnection cn = new SqlConnection(strConn);
cn.Open();
SqlCommand cmd =cn.CreateCommand();
SqlTransaction ta=cn.BeginTransaction();
cmd.Transaction=ta;
try
{
for(int i=0;i<sqls.Length;i++)
{
cmd.CommandText = sqls[i];
cmd.CommandType =System.Data.CommandType.StoredProcedure;
cmd.Parameters.Clear();
foreach(SqlParameter pa in pas[i])
{
cmd.Parameters.Add(pa);
}
cmd.ExecuteNonQuery();
}
ta.Commit();
}
catch
{
ta.Rollback();
success= false;
}
finally
{
cn.Close();
}
return success;
}

[jijl2001]

可以的

[blackhero]

sqls,是存储过程,第个存储过程,都有参数pas

[xhmbldsh]

根本没必要写事物

[blackhero]

我往七个表里插数据

[copico]

可以，我到是觉得你可以把SQL放在一个连成一个字符串，传给一个存储过程。然后在里面分解

再执行

这样的效率和安全率都相对比页面的高，因为必毕是在数据库里操作

[gui0605]

好象是对的啊~

[cancerser]

遇到什么问题了吗?

[Ivony]

ta.Commit();

如果你这里失败呢？

还有，变量名用transaction，ta是什么？他？


如果你Commit失败，那么触发异常，被catch到，再执行ta.RollBack，肯定又是失败，这样异常就跑出去了，你没有屏蔽异常反而还篡改了异常信息。

也不应该屏蔽异常，
建议把Commit放到try外面去，connection用using包住，catch里面加个throw;
非要屏蔽异常的话，把RollBack加个try

[鼻涕蛋蛋]

mark 楼上说的，我貌似就遇见过。。

[baffling]

gz