使用数据库 最佳实践？

Question

我有一个 C# 数据库层，它具有在后台计时器中每秒调用一次的静态读取方法。

目前我创建 SqlCommand、SqlConnection 一次作为类成员。

在每个方法调用中我都执行命令来获取结果，我这样做是为了避免每秒创建连接和命令，但我担心这个方法会发生异常，会中断连接或将对象放入无效状态。

这是我当前的实现（定时器处理程序）

    static void GetBarTime(object state)
    {
        lock (_staticConnection)
        {
            SqlDataReader dataReader = null;
            try
            {
                dataReader = _getMaxTimeCommand.ExecuteReader();
                dataReader.Read();
                _currentTick = dataReader.GetInt32(0);
            }
            catch (Exception ex)
            {
                //Log the error
            }
            finally
            {
                dataReader.Dispose();
            }
        }
    }

最好的做法是什么？

更多详情：

我在计时器中执行此操作，因为有另一个进程每秒更新我的表，并且一组客户端使用另一个公开的方法并每秒调用一次以获取最新值。

因此，我不是每秒为每个客户端执行 select 语句，而是在计时器中执行此操作并更新客户端使用的全局变量。

Answer 1

SqlConnection 内置了池化；如果你使用，你会发现几乎没有区别：

using(SqlConnection conn = new SqlConnection(connectionString)) {
    conn.Open();
    // your code
}

每次。这可以自动对死（底层）连接做出反应。

目前你有一个错误，顺便说一句；如果命令失败，阅读器仍将为空……在调用Dispose()之前检查null：

if(dataReader !=null) {dataReader.Dispose();}

或者直接使用using:

 try
 {
     using(SqlDataReader dataReader = _getMaxTimeCommand.ExecuteReader())
     {
         dataReader.Read();
         _currentTick = dataReader.GetInt32(0);
     }
 }
 catch (Exception ex)
 {
    //Log the error
 }

Answer 2

很难确定一个执行是否意味着连接是一个死鸭子。为了安全起见，您可以在遇到异常时关闭并重新打开 SqlConnection 和 SqlCommand，以防万一。当一切正常时，这不会造成任何开销。