提高DAL性能

Question

我目前填充业务对象的方式是使用类似于下面的代码段.

using (SqlConnection conn = new SqlConnection(Properties.Settings.Default.CDRDatabase))
{
    using (SqlCommand comm = new SqlCommand(SELECT, conn))
    {
        conn.Open();

        using (SqlDataReader r = comm.ExecuteReader(CommandBehavior.CloseConnection))
        {
            while (r.Read())
            {
                Ailias ailias = PopulateFromReader(r);
                tmpList.Add(ailias);
            }
        }
    }
}

private static Ailias PopulateFromReader(IDataReader reader)
{
    Ailias ailias = new Ailias();

    if (!reader.IsDBNull(reader.GetOrdinal("AiliasId")))
    {
        ailias.AiliasId = reader.GetInt32(reader.GetOrdinal("AiliasId"));
    }

    if (!reader.IsDBNull(reader.GetOrdinal("TenantId")))
    {
        ailias.TenantId = reader.GetInt32(reader.GetOrdinal("TenantId"));
    }

    if (!reader.IsDBNull(reader.GetOrdinal("Name")))
    {
        ailias.Name = reader.GetString(reader.GetOrdinal("Name"));
    }

    if (!reader.IsDBNull(reader.GetOrdinal("Extention")))
    {
        ailias.Extention = reader.GetString(reader.GetOrdinal("Extention"));
    }

    return ailias;
}

有没有人有任何关于如何改善这样的表现的建议？请记住,对于某些类型,PopulateFromReader包含更多数据库查找,以便完全填充对象.

Answer 1

一个明显的变化是替换这种语句:ailias.AiliasId = reader.GetInt32(reader.GetOrdinal("AiliasId"));

同

ailias.AiliasId = reader.GetInt32(constAiliasId);

其中constAiliasId是一个常数,持有AiliasId字段的序数.

这避免了循环的每次迭代中的序数查找.

Answer 2

如果数据量很高,那么构建庞大列表的开销可能会成为瓶颈; 在这种情况下,使用流对象模型可能更有效; 即

public IEnumerable<YourType> SomeMethod(...args...) {
    using(connection+reader) {
        while(reader.Read()) {
            YourType item = BuildObj(reader);
            yield return item;
        }
    }
}

消费代码(通过foreach等)然后只有一个对象(一次)处理.如果他们想获得一个列表,他们可以(使用新的List<SomeType>(sequence),或在.NET 3.5中sequence.ToList()).

这涉及到更多的方法调用(附加MoveNext()/ Current每个序列项,隐藏在其后面foreach),但是当您有来自数据库的进程外数据时,您将永远不会注意到这一点.