在 ASP.Net 中将数据更新到 SQL Server 的最佳实践是什么

Question

我正在 ASP.Net 中将数据更新到 SQL Server。

而且我只有一个 cs 文件，没有 aspx/ascx 文件，所以我不会在这里使用 SqlDataSource 控件。

下面是我的代码：

string connStr = ConfigurationManager.ConnectionStrings["XXConnString"].ConnectionString;

SqlConnection conn = new SqlConnection(connStr);

if (conn.State == ConnectionState.Closed)
{
    conn.Open();
}

string query = @"exec dbo.XX_Insert_Announcement @AnnID ='" + id +
                                            "', @AnnTitle ='" + title +
                                            "', @AnnSubmitDateTime ='" + startDate + 
                                            "', @AnnProcessDateTime ='" + endDate + "'";

SqlCommand cmd = new SqlCommand(query, conn);
cmd.ExecuteNonQuery();
conn.Close();

if (conn.State == ConnectionState.Open)
{
    conn.Close();
}

我认为我的代码对于错误处理来说不够好，如果查询执行失败，则不会抛出错误并且代码会继续运行而不更新到数据库。

也许使用Using语句可以解决这个问题，代码如下：

using (SqlConnection connection = new SqlConnection(connectionString))
{
    SqlCommand command = new SqlCommand(queryString, connection);
    command.Connection.Open();
    command.ExecuteNonQuery();
}

请评论并建议什么是最佳实践。

先感谢您。

Answer 1

您的第二种方法更好，但如果您使用参数化查询来更好地防止 SQL 注入攻击，效果会更好。

using (SqlConnection connection = new SqlConnection(connectionString))
{
 using(SqlCommand command = connection.GetCommand(queryString, CommandType.Text))
 {
  command.Parameters.Add(new SqlParameter("AnnID", id));
  command.Parameters.Add(new SqlParameter("AnnTitle", title));
  ............
  ..............
  command.Connection.Open();
  command.ExecuteNonQuery();
 }
}