通用方法。将属性从 DataReader 分配给通用对象

Question

早上好，我从一堆 Google 搜索中创建了我的第一个通用方法。我希望有人检查一下，如果我违反了任何主要规则，或者是否有办法改进这种方法，请告诉我。

该方法调用 sql 中的存储过程，然后根据从 DataReader 架构中读取的值使用反射来分配属性。对存储过程进行了编码，以便它们返回类所期望的确切属性名称。代码如下：

       public static List<T> GetList<T>(string SQLServer, string DBName,
        string ProcedureName, Dictionary<string, string> Parameters )
         where T : new()
    {
        List<T> list = new List<T>();

        //Setup connection to SQL
        SqlConnection SqlConn = new SqlConnection(ConnectionString(SQLServer, DBName));
        SqlCommand SqlCmd = new SqlCommand(ProcedureName, SqlConn);
        SqlCmd.CommandType = System.Data.CommandType.StoredProcedure;
        SqlDataReader reader;

        //Process Parameters if there are any
        foreach (KeyValuePair<string, string> param in Parameters)
        {
            SqlCmd.Parameters.AddWithValue(param.Key, param.Value);
        }

        SqlConn.Open();
        reader = SqlCmd.ExecuteReader();

        //Get The Schema from the Reader
        //The stored procedure has code to return
        //the exact names expected by the properties of T
        DataTable schemaTable = reader.GetSchemaTable();
        List<string> fields = new List<string>();
        foreach (DataRow r in schemaTable.Rows)
        {
            fields.Add(r[0].ToString());
        }


        while (reader.Read())
        {
            T record = new T();

            foreach (string field in fields)
            {
                //Assign the properties using reflection
                record.GetType().GetProperty(field).SetValue(
                    record, reader[field],
                    System.Reflection.BindingFlags.Default,
                    null,null,null);
            }

            list.Add(record);
        }
        return list;
    }

Answer 1

我不知道我是否会这样做。我可能会先使用 ORM，例如实体框架。不过，我过去也做过类似的事情，但也有一些缺点：

• 反射可能比只指定所有内容要慢，具体取决于多少，这对您来说可能根本不是问题。
• 对我来说，更大的问题只是你冒着更多运行时错误而不是编译时错误的风险，这样可以节省前期时间，但从长远来看可能会引入烦人的错误。

关于我唯一肯定要说的就是确保使用 try/catch/finally、try/finally，或者将 SqlConnection、SqlCommand 和 SqlDataReader 包装在 using() 中。我只花了2天时间重构，因为以前的开发者没有关闭任何连接或数据读取器，并且连接池正在炸毁并拒绝连接。

Answer 2

虽然这种方法确实有效，但您肯定希望添加一些错误处理。

还有一些现有的库可以为您执行此操作，例如 AutoMapper。您还可以查看其他 ORM，例如 SubSonic、Linq2SQL、EntityFramework、NHibernate 等...

还要注意反射非常慢，尤其是像这样一遍又一遍地进行。如果这将在大型企业重负载系统中，您最好在第一次遇到映射时生成 ta 动态方法和 IT 代码来进行映射，然后一遍又一遍地重新运行相同的动态方法，而不是依赖反射。