将datarow提取到c#对象

Question

我有一个类Item表示列表中的项目.我在其函数中调用返回datatable的存储过程,我需要将数据表转换为项目数组.这是我做的:

public class Item
{
    private string _ItemIdDataName = "item_id";
    private string _ItemNameDataName = "item_name";
    private string _PriceDataName = "price";

    public long ItemId { get; set; }
    public string ItemName { get; set; }
    public float Price { get; set; }

    private Item(DataRow row)
    {
        if (row != null)
        {
            ItemId = long.Parse(row[_ItemIdDataName].ToString());
            ItemName = row[_ItemNameDataName].ToString();
            Price = float.Parse(row[_PriceDataName].ToString());
        }
    }

    public Item[] load()
    {
        DataTable dt=DBHandler.GetItems();//Stored procedure that returns DataTable 
        Item[] items = new Item[dt.Rows.Count];
        for (int i = 0; i < dt.Rows.Count; i++)
        {
            items[i] = new Item(dt.Rows[i]);
        }
        return items;
    }
}

我做得对吗？我怎样才能改善这个？

Answer 1

如果你只是在它可能会好的时候使用它,但如果你会做很多,你应该尝试做一些更通用的东西.我写了一篇关于如何编写扩展方法的博客文章DataTable,创建了一个对象列表.它的惯例是,对象中的属性应该与存储过程中的列具有相同的名称(如果可以的话,我会在存储过程中更改名称):

public static class DataTableExtensions
{
    public static IList<T> ToList<T>(this DataTable table) where T : new()
    {
        IList<PropertyInfo> properties = typeof(T).GetProperties().ToList();
        IList<T> result = new List<T>();

        foreach (var row in table.Rows)
        {
            var item = CreateItemFromRow<T>((DataRow)row, properties);
            result.Add(item);
        }

        return result;
    }

    public static IList<T> ToList<T>(this DataTable table, Dictionary<string, string> mappings) where T : new()
    {
        IList<PropertyInfo> properties = typeof(T).GetProperties().ToList();
        IList<T> result = new List<T>();

        foreach (var row in table.Rows)
        {
            var item = CreateItemFromRow<T>((DataRow)row, properties, mappings);
            result.Add(item);
        }

        return result;
    }

    private static T CreateItemFromRow<T>(DataRow row, IList<PropertyInfo> properties) where T : new()
    {
        T item = new T();
        foreach (var property in properties)
        {
            property.SetValue(item, row[property.Name], null);
        }
        return item;
    }

    private static T CreateItemFromRow<T>(DataRow row, IList<PropertyInfo> properties, Dictionary<string, string> mappings) where T : new()
    {
        T item = new T();
        foreach (var property in properties)
        {
            if(mappings.ContainsKey(property.Name))
                property.SetValue(item, row[mappings[property.Name]], null);
        }
        return item;
    }
}

现在你可以打电话了

var items = dt.ToList<Item>();

要么

var mappings = new Dictionary<string,string>();
mappings.Add("ItemId", "item_id");
mappings.Add("ItemName ", "item_name");
mappings.Add("Price ", "price);
var items = dt.ToList<Item>(mappings);

博客文章在这里:http://blog.tomasjansson.com/2010/11/convert-datatable-to-generic-list-extension

有许多方法可以扩展它,你可以包含某种映射字典,告诉扩展如何映射列,这样名称就不需要匹配了.或者,您可以添加要在映射中排除的属性名称列表.

更新:您Item正在创建的对象()必须具有默认构造函数,否则私有方法将无法创建它.由于解决方案的工作方式首先是创建对象,而不是使用从反射中获取的属性来设置对象的值.

更新2:我添加了带有映射字典的部分,但我自己没有尝试过,所以它可能无法编译.但是,这个概念是存在的,我认为它有效.

Answer 2

很好,但我有一些建议:

• 不要将ToString转换为要解析回另一种类型的东西.这可能会导致数据类型损坏,并且速度慢/效率低.

• 期待并检查来自SQL Server的null.

所以,而不是:

ItemId = long.Parse(row[_ItemIdDataName].ToString());

尝试:

 ItemId = row.Field<long?>(_ItemIdDataName) ?? value_if_null; 

(添加对System.Data.DatasetExtensions的引用以获取Field扩展名)