0

我正在查看一个较旧的应用程序的冗长数据访问代码。每个函数都在调用一个存储过程来从 Oracle DB 中选择一些东西。每个函数或多或少看起来像下面的代码：

public List<SomeObject> GetMeSomethingFromDB(string myParam, int anotherParam)
{
    OracleConnection conn = null;
    OracleDataReader dataReader = null;
    try
    {
        conn = new OracleConnection(Settings.ConnectionString);
        conn.Open();

        var cmd = new OracleCommand("STORED_PROC_NAME", conn);
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.Parameters.Add(new OracleParameter("IN_MY_PARAM", OracleDbType.Varchar2)).Value = myParam;
        cmd.Parameters.Add(new OracleParameter("IN_ANOTHER_PARAM", OracleDbType.Int32)).Value = anotherParam;
        cmd.Parameters.Add(new OracleParameter("OUT_REF_CURSOR", OracleDbType.RefCursor)).Direction = ParameterDirection.Output;
        dataReader = cmd.ExecuteReader();

        List<SomeObject> result = new List<SomeObject>();
        if (dataReader == null || !dataReader.HasRows) return result;

        while (dataReader.Read())
        {
            SomeObject someObject = new SomeObject
            {
                SomeId = (int)dataReader["SOME_ID"],
                SomeStringValue = dataReader["SOME_STRING_VALUE"].ToString()
            };

            result.Add(someObject);
        }

        return result;
    }
    catch (Exception e)
    {
        throw e;
    }
    finally
    {
        if (dataReader != null)
        {
            dataReader.Close();
            dataReader.Dispose();
        }
        if (conn != null)
        {
            if (conn.State == ConnectionState.Open) conn.Close();
            conn.Dispose();
        }
    }
}

我的问题是：

  1. 一些函数使用类级别的 OracleConnection 变量。什么是首选 - 功能级别或类级别变量？
  2. 检查是否dataReader == null必要？cmd.ExecuteReader()通话后它会为NULL吗？
  3. 连接关闭/处置和阅读器关闭/处置的功能不同。关闭/处置的正确方式/顺序是什么？如果连接被释放，阅读器不会自动关闭/释放吗？
  4. 我希望在不久的将来将 Oracle.ManagedDataAccess.Client 连接到这个项目。此代码中的任何内容是否会更改以与托管数据访问客户端一起使用？
  5. 除此之外，欢迎任何最佳实践/建议。

谢谢你。

4

1 回答 1

4

using 语句将大大简化您的代码。

public List<SomeObject> GetMeSomethingFromDB(string myParam, int anotherParam)
{
    using (OracleConnection conn = new OracleConnection(Settings.ConnectionString))
    using (OracleCommand cmd = new OracleCommand("STORED_PROC_NAME", conn))
    {   
        conn.Open();
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.Parameters.Add(new OracleParameter("IN_MY_PARAM", OracleDbType.Varchar2)).Value = myParam;
        cmd.Parameters.Add(new OracleParameter("IN_ANOTHER_PARAM", OracleDbType.Int32)).Value = anotherParam;
        cmd.Parameters.Add(new OracleParameter("OUT_REF_CURSOR", OracleDbType.RefCursor)).Direction = ParameterDirection.Output;
        using (OracleDataReader dataReader = cmd.ExecuteReader())
        {
            while (dataReader.Read())
            {
                 SomeObject someObject = new SomeObject
                 {
                     SomeId = (int)dataReader["SOME_ID"],
                     SomeStringValue = dataReader["SOME_STRING_VALUE"].ToString()
                 };
                 result.Add(someObject);
            }
        }
    }
    return result;
}
  1. 始终使用本地连接对象并将其包含在 using 语句中以正确关闭和处置对象（OracleDataReader 和 OracleCommand 也是如此）。这将使您的服务器从与您的代码保持连接所需的内存和线程中释放出来，并且性能由 ADO.NET 提供程序启用的连接池保证
  2. 不，如果您打算遍历结果，则不需要调用，也不需要调用HasRows 。如果没有行或到达数据集的末尾，则读取器返回 false
  3. 请参阅有关 using 语句的要点。正确的 using 语句将从您的负担中消除这个问题。
  4. 如果您使用 Oracle 的 ODP 提供程序，则此代码应该没有任何问题
  5. 如果您只想重新抛出异常，则无需尝试捕获。只需让它冒泡到上层而不用 a 中断堆栈跟踪，throw e并且 finally 语句中所需的所有代码都由编译器隐式添加到 using 右花括号中。
于 2017-10-02T20:31:40.093 回答