Question

我正在为使用n层架构的旧asp.net应用程序添加新功能。一个基本的例子可以作为

给出

对象

public class Test
{
    public int ID{get;set;}
    public int name{get;set;}
}

数据访问层

public static List<Test> GetTests()
{
    List<Test> list = new List<Test>();
    try
    {
        //codes
        SqlDataReader dr  = com.ExecuteReader();
        while(dr.Read())
            list.Add(FillTestRecord(dr))
        //codes
    }
    catch{}
    return list;
}

private static Test FillTestRecord(IDataRecord dr)
{
    Test test = new Test();
    try{test.ID = Convert.ToInt32(dr["ID"]);}
    catch{}
    try{test.Name = Convert.ToInt32(dr["Name"]);}
    catch{}
    return test;
}

开发需要我向对象类添加新字段，并且为了可重用性，我对每种类型的对象只使用一个Fill * Record方法。许多其他DAL方法可以调用此方法，其IDataRecord可能不包含对象的所有列。因此我分别为每个属性设置了try-catch块。这可以确保正确解析IDataRecord中的所有可用列。

我的问题是，还有更好的方法吗？这种架构中最好的做法是什么？

更新

在阅读David L和Anup的评论/回答后，我尝试了另一种使用扩展方法的方法。方法如下

public static bool TryGetOrdinal(this IDataRecord dr, string column, out int ordinal)
{
    try
    {
        ordinal = dr.GetOrdinal(column);
    }
    catch(Exception ex)
    {
        ordinal = -1; //Just setting a value that GetOrdinal doesn't return
        return false;
    }
    return true;
}

所以FillTestRecord方法将是

private static Test FillTestRecord(IDataRecord dr)
{
    Test test = new Test();

    int ordinal = default(int);

    if(dr.TryGetOrdinal("ID",out ordinal))
        test.ID = Convert.ToInt32(dr.GetValue(ordinal));
    if(dr.TryGetOrdinal("Name",out ordinal))
        test.Name = Convert.ToString(dr.GetValue(ordinal));

    return test;
}

对此的任何建议都表示高度赞赏。

更新03-02-2016

在调试期间，如果try-catch在GetOrdinal中找不到提供的列名称时DataRecord抛出错误，我发现DataReader会对性能造成很大影响。所以我写了一个新方法，它在GetOrdinal中获取列名，并用Array.IndexOf替换public static bool TryGetOrdinal(this IDataRecord dr, string[] columnNames, string column, out int ordinal) { ordinal = Array.IndexOf(columnNames, column); return ordinal >= 0; }。

FillTestRecord

我的private static Test FillTestRecord(IDataRecord dr, string[] columnNames) { Test test = new Test(); int ordinal = default(int); if(dr.TryGetOrdinal(columnNames, "id",out ordinal)) test.ID = Convert.ToInt32(dr.GetValue(ordinal)); if(dr.TryGetOrdinal(columnNames, "name",out ordinal)) test.Name = Convert.ToString(dr.GetValue(ordinal)); return test; }变成了 -

using (var dr = com.ExecuteReader())
{
    string[] colNames = dr.GetColumnNames();
    while (dr.Read())
        list.Add(FillTestRecord(dr, colNames));
}

列名称传递给Fill方法，如下所示 -

public static string[] GetColumnNames(this IDataReader dr)
{
    string[] columnNames = new string[dr.FieldCount];
    for (int i = 0; i < dr.FieldCount; i++)
    {
        columnNames[i] = dr.GetName(i).ToLower();
    }
    return columnNames;
}

'GetColumnNames'是新的扩展方法 -

{{1}}

Answer 1

我觉得你的方向正确。
只要解析是在所有高级类重用的集中位置完成的，这看起来就像是一个很好的解决方案。

我唯一要改变的是替换try-catch语句，检查列中是否存在数据。肯定有办法告诉（列不存在？DB-Null值？）您可以使用与TryParse methods类似的内容来实现它。

    private static Test FillTestRecord(IDataRecord dr)
    {
        Test test = new Test();
        int tempId;
        if (TryParseDataRow<int>(dr, "ID", out tempId))
        {
            test.Id = tempId;
        }
        return test;
    }

    private static bool TryParseDataRow<T>(IDataRecord record, string column, out T value)
    {
        value = default(T);
        bool success = true;

        if (record == null)
        {
            //nothing you can do with a null object
            success = false;
        }
        else if (!record.HasColumn(column)) //not sure if this will throw exeption or return null. you can check in your project
        {
            success = false;
        }
        else if (record[column] != typeof(T))
        {
            //object was of an unexpected type
            success = false;
        }           
        else
        {
            //cast the value into the output parameter
            value = (T)record[column];
        }

        return success;
    }

当然，您必须实现HasColumn方法（此处实现为扩展名）：

    /// <summary>
    /// Determines whether the specified record has column.
    /// </summary>
    /// <param name="record">The record.</param>
    /// <param name="columnName">Name of the column.</param>
    /// <returns>true if column exist, false otherwise</returns>
    public static bool HasColumn(this IDataRecord record, string columnName)
    {
        for (int i = 0; i < record.FieldCount; i++)
        {
            if (record.GetName(i).Equals(columnName, StringComparison.InvariantCultureIgnoreCase))
                return true;
        }
        return false;
    }

Answer 2

这里列出了一些代码，用于将IDataRecord映射到属性

    public static T ParseRecord<T>(this IDataRecord reader) where T : new()
    {
        var model = new T();
        var type = typeof(T);

        for (int i = 0; i < reader.FieldCount; i++) {
            var fieldType = reader.GetFieldType(i);
            var fieldName = reader.GetName(i);
            var val = reader.GetValue(i);
            var prop = type.GetProperty(fieldName);

            // handle or throw instead here if needed
            if (prop == null)
                continue;

            var propType = prop.PropertyType;

            // HACK: remove this if you don't want to coerce to strings
            if (propType == typeof(string))
                prop.SetValue(model, val.ToString());
            else if (fieldType != propType)
                throw new Exception($"Type mismatch field {fieldType} != prop {propType}");
            else
                prop.SetValue(model, val);
        }

        return model;
    }

Answer 3

我使用以下代码来映射不同对象的属性。它使用反射来获取源对象和目标对象的属性，但您可以轻松地将其更改为使用IDataRecord：

public static T MapDTO<T>(object dto) where T : new()
{
    T Result = new T();

    if (dto == null)
        return Result;

    dto.GetType().GetProperties().ToList().ForEach(p =>
        {
            PropertyInfo prop = Result.GetType().GetProperty(p.Name);

            if (prop != null && prop.CanWrite)
            {
                try
                {
                    var convertedVal = Convert.ChangeType(p.GetValue(dto, null), prop.PropertyType);
                    prop.SetValue(Result, convertedVal, null);
                }
                catch (Exception ex)
                {
                    try
                    {
                        prop.SetValue(Result, p.GetValue(dto, null), null);
                    }
                    catch (Exception ex1) { }
                }
            }
        });
    return Result;
}

这里的关键是source和dest属性应该具有相同的名称。

将数据解析为数据访问层中的对象有哪些最佳实践？

更新

更新03-02-2016

3 个答案: