MoveNext 在实现 IEnumerable的类中表现糟糕

Question

我有一个在 MVC 应用程序中使用的包装类，该类旨在循环访问集合中的项目，并在返回该项目进行显示之前检查当前用户是否有权访问该项目。在大多数情况下，它就像一个魅力。然而，对于一种特定的对象类型，它运行得像一只绝对的狗，我不明白为什么。

接口类是这样写的：

    private readonly IEnumerable<T> _ViewData;

    public IEnumerator<T> GetEnumerator()
    {
        foreach (T item in _viewData)
        {
            if (item.UserCanEdit || item.UserCanView)
                yield return item;
        }
    }


    System.Collections.IEnumerator System.Collections.IEnumerable.GetEnumerator()
    {
         return GetEnumerator();
    }

现在我的第一个问题是检查我对这里发生的事情的理解。我可能是错的，但我的理解是，当我尝试对类型对象集合运行 foreach 循环时，它永远不需要使用 System.Collections.IEnumerable.GetEnumerator() - 当然该代码永远不会被命中？

问题在于，当我尝试在日志文件条目上使用此包装器时，日志文件条目是通过 LogEntryViewDaya 类中的实体框架传递的。当代码到达 GetEnumerator 内的 foreach 循环时，它会停止整整 6-8 秒，即使枚举中只有 20 个项目！

这是 LogEntryView 的代码

public class LogEntryViewData:BaseViewData
{
    private LogEntry _entry;
    public LogEntryViewData(LogEntry entry, ICustomerSecurityContext securityContext) : base(securityContext)
    {
        _entry = entry;
    }

    public string Application { get { return _entry.Application; } }
    public string CurrentUser { get { return _entry.CurrentUser; } }
    public string CustomerName { get { return _entry.CustomerName; } }
    public DateTime Date { get { return _entry.Date; } }
    public string Exception { get { return _entry.Exception; } }
    public string HostName { get { return _entry.HostName; } }
    public long Id { get { return _entry.Id; } }
    public string Level { get { return _entry.Level; } }
    public string Message { get { return _entry.Message; } }
    public int? ProcessId { get { return _entry.ProcessId; } }
    public int? ServiceId { get { return _entry.ServiceId; } }
    public string ServiceName { get { return _entry.ServiceName; } }
    public int? TaskId { get { return _entry.TaskId; } }
    public int? TaskName { get { return _entry.TaskName; } }
    public string Thread { get { return _entry.Thread; } }
}

据我所知，实例化这些类没有明显的性能 - 在构造函数中放置一个断点并通过 F5 似乎很顺利。

那么为什么这些特定对象的集合的迭代速度如此之慢呢？我不知道：非常感谢建议。

Answer 1

您还没有向我们展示班级是如何填充的。我的猜测是，数据库查询的时间正在流逝 - 特别是，根据所获取的内容，可能有一个查询来获取骨架实体，然后对每个实体进行一个查询 UserCanEdit 和 UserCanView。这似乎不太可能（我希望该条目在初始查询中填充这些属性），但只是有可能。

基本上，观察它如何与数据库交互。

我建议你尝试编写一个使用相同类的控制台应用程序，这样你就可以比在网络应用程序中更轻松地使用它、添加计时日志等。

编辑：除了评论等中的其他内容之外，您是否有任何理由不将其作为 LINQ 查询的一部分来执行？例如：

var query = db.LogEntries.Where(x => x.UserCanEdit || x.UserCanView);

我很欣赏它可能不会那么简单，因为 UserCanEdit 将取决于当前用户等 - 但它可能应该作为数据库完成查询而不是客户端。