C# 泛型效率，一种更好的方法

Question

好的，假设我有如下类：

public class KPIObject<T> //<--This class where T is the following classes
{
    public List<T> Data { get; set; }
    public string Caption { get; set; }
}

public class KPICycleCountAccuracyData //<--There are 20 of these with different names and values
{
    public string Facility { get; set; }
    public string CCAdjustedCases { get; set; }
    public string TotalCases { get; set; }
    public string CCAdjustedPercent { get; set; }
}

然后我有：

public List<ReportData>> ProcessAccountReport(GetAccountReport request)
{
    var data = new List<ReportData>();
    ProcessKPI(data, request.KPICycleCountAccuracy, "KPICycleCountAccuracy"); //<-- 20 of these
    return data;
}

这是 ProcessKPI 方法：

private static void ProcessKPI<T>(List<ReportData> data, ICollection<KPIObject<T>> items, string name)
{
    if (items == null || items.Count <= 0) return;
    foreach (var item in items)
    {
        if (item.Data == null || item.Data.Count <= 0) continue;
        var temp = new List<object>();
        temp.AddRange((IEnumerable<object>)item.Data);
        data.Add(new ReportData { Data = temp, Name = name, Title = item.Caption });
    }
}

所有这些都可以正常工作并编译，我只是想知道这是否是最有效的方法。

谢谢。

编辑

我将流程 KPI 更改为：

private static void ProcessKPI<T>(ICollection<ReportData> data, ICollection<KPIObject<T>> items, string name)
        {
            if (items == null || items.Count <= 0) return;
            foreach (var item in items.Where(item => item.Data != null && item.Data.Count > 0))
            {
                data.Add(new ReportData { Data = (IEnumerable<object>)item.Data, Name = name, Title = item.Caption });
            }
        }

Answer 1

注释

• 无需在 ProcessKPI 中将 data 设为 ref 参数。如果您实际分配给 C# 中的 class 类型，ref 参数仅对它有意义。在这里，您只是修改对象，因此除了尴尬的调用语法之外， ref 不会对您进行任何操作。
• 即使 Count 已签名，它也不会返回负值。
• 我更喜欢 (IEnumerable

Answer 2

速度
假设您正在谈论计算效率（即速度），您可能可以改进两个操作：

首先，您在 temp 中创建 item.Data 的副本代码>变量。当您知道生成的 ReportData 永远不会被修改时，您可以直接使用 item.Data，从而放弃昂贵的复制操作。

<代码>
data.Add(新的报表数据{
数据 = (IEnumerable

其次，转换为 IEnumerable

内存
通过使用延续来实现您的解决方案，您可以一次处理一个 ReportData 元素，而不是一次处理所有元素，从而减少内存占用。查看 yield 语句看看如何实施这种方法。

其他
对于进一步提高代码质量，JaredPar 的回答提供了一些很好的建议。