在 asp.net-mvc 网页中关闭 OdbcConnection 的正确模式是什么

Question

我有一个网站，并且有一个具有 OdbcConnection 的数据访问类。在构造函数中，我有这样的代码：

    public MySybaseProvider()
    {
        _conn = GetConn();        
    }

    public OdbcConnection GetConn()
    {
        string connString = "DSN=SybaseIQ;Eng=SYSERVER;Links=tcpip(Host=" + _connectionInfo.Host + ";Port=" + _connectionInfo.Port + ");UID=" + _connectionInfo.User + ";PWD=" + _connectionInfo.Pwd + ";";

        return new OdbcConnection(connString);
    }

在整个类中，我有以下代码：

    private OdbcQuery GetQuery(string sql)
    {
        return new OdbcQuery(_conn, sql);
    }

我试图找出确保正确关闭连接的最佳方法。我应该实施 IDisposable 吗？我应该打开和关闭每个查询的连接吗？这里还有其他最佳实践吗？

编辑：

澄清一下，我的用例是我正在加载一个网页，但需要大约 10 个查询才能获取该页面所需的所有数据。

Answer 1

检查ODBC 连接池是否已启用，并创建并打开连接对象对于每次访问数据库都使用 using 语句：（

using (var conn = new OdbcConnection(_connString)) {
  conn.Open();

  // do a database command ...
}

请注意，我更改了设计以将您的 _connString 存储在字段中）

这是最惯用的方式为此，您无需担心在构造函数中创建连接、将其存储在字段中，甚至无需实现 IDisposable。

Answer 2

所以我提交了另一个答案，因为我意识到我通常使用企业库进行数据访问，而不是纯粹的.net，这会改变一些事情。其中之一是 DataAccessApplicationBlock (DAAB) 负责打开和关闭到数据库的连接，所以我没有太多，因此我像下面一样使用它。此外，在这种情况下，执行所描述的操作的一种更简单、更有效的方法是执行 1 个存储过程中的所有 SELECT 语句，然后一次读取并解析每个结果集。

示例：

存储过程

Create PROCEDURE [dbo].[GetUserByEmail]
    @UserID int,
    @AccountID int,
AS
BEGIN
    SET NOCOUNT OFF;

    SELECT * FROM Users WHERE UserID = @UserID and AccountID = @AccountID

    SELECT * FROM UserAddresses WHERE UserID = @UserID
END

C# 代码：

public class DataAccessLayer
{
    private static DataAccessLayer me = new DataAccessLayer();

    private DataAccessLayer() { }

    public static DataAccessLayer GetInstance()
    {
        return me;
    }

    public Database GetDatabase(string connectionString, string provider)
    {
       DbProviderFactory providerFactory = DbProviderFactories.GetFactory(provider);

        return new GenericDatabase(connectionString,
                                   providerFactory);
    }
}

public class Repository
{
    protected Database curDatabase;

    public Repository()
    {
        curDatabase = DataAccessLayer.GetInstance().GetDatabase("ConnectionString", "System.Data.Odbc");
    }

    public Database CurrentDatabase
    {
        get { return curDatabase; }
    }
}

public class UserRepository : Repository
{
    public User GetUser(int urserID, int accountID)
    {
        using (IDataReader dataReader = CurrentDatabase.ExecuteReader("StroedProcName", urserID, accountID))
        {
            //User Details
            while (dataReader.Read())
            {
                //Parse Data
            }

            dataReader.NextResult();

            //User Address(es)
            while (dataReader.Read())
            {
                //Parse Data
            }
        }
    }
}

Answer 3

我认为你绝对应该实现 IDisposable 并遵循 处置模式 请记住，垃圾收集器对 IDisposable 一无所知。作为最后的手段，您应该有一个终结器，它会在需要时终止数据库连接，但尽可能抑制终结器（请参阅我添加的链接以了解所有血淋淋的详细信息）。

至于您是想为每个命令创建一个新连接还是在对象的整个生命周期中保留一个连接，这更加主观，并且取决于您的场景、环境和许多其他因素。我建议探索两者并看看你的想法。如果这是一个“真正的”数据库，也许可以与 DBA 交谈并看看他们的想法。

无论哪种方式，实现 IDisposable 都是您的朋友，因为您可以以所有 .NET 开发人员都熟悉的模式拥有漂亮、干净、定义良好的处理代码。

Answer 4

您可以使用以下模式来确保连接已释放。即使发生错误，也会被处理掉。为了更安全的编码，您应该使用连接字符串生成器来构建连接。您应该仅在计划执行操作时创建/使用连接。它不应该在您的应用程序的整个生命周期中徘徊。您可以使用连接字符串/属性来代替。

    public OdbcConnection GetConn()
    {
        OdbcConnectionStringBuilder sb = new OdbcConnectionStringBuilder();
        sb.Driver = "Microsoft Access Driver (*.mdb)";
        sb.Add("Dbq", "C:\\info.mdb");
        sb.Add("Uid", "Admin");
        sb.Add("Pwd", "pass!word1");
        OdbcConnection con = new OdbcConnection(sb.ConnectionString);
        return con;
    }


    public void DoSomeWork()
    {
        using(OdbcConnection connection = GetConn())
        {
            // Do stuff with your connection here


        }
    }