3

我已经使用 THP 实现了具有简单继承的代码优先数据库架构： 数据库图

我需要查询所有类型的所有通知。 TargetUser表中的属性NotificationUser是一个关联。我正在尝试执行下一个代码：

var notifications = _context.Notifications;
foreach (var notification in notifications)
{
    Debug.WriteLine((notification is NotificationUser)? ((NotificationUser) notification).TargetUser?.Name : "-");
}

在数据库TargetUser中属性设置为正确的外键，但在代码中我没有得到任何结果。延迟加载已启用。

是否可以用户急切加载？我已经尝试过写_context.Notifications.Include('TargetUser')它会引发异常。

更新。例外是：

A specified Include path is not valid. The EntityType 'Core.Concrete.NotificationBase' does not declare a navigation property with the name 'TargetUser'.

试图将此答案修改为：

var notifications = _context.Notifications.OfType<NotificationUser>()
                .Include(n => n.TargetUser)
                .Cast<NotificationBase>()
                .Union(_context.Notifications.OfType<NotificationPlace>()

但仍然抛出相同的异常。

4

3 回答 3

2

我知道这是一个旧线程，但我仍然想为寻求相同解决方案的人发布一些改进。

1.网络冗余

选择 Ids 然后运行一个查询，加载带有 Ids 的项目是多余的，只需运行这个就可以达到相同的效果

解决方案：

var userNotifications = _context.Notifications
    .OrderByDescending(n => n.DateTime)
    .Skip(offset)
    .Take(limit)
    .OfType<NotificationUser>()
    .Include(n => n.TargetUser)
    .Include(n => n.TargetUser.Images)
    .ToList();

这样，您无需等待 2 个 DB 连接，而只需等待一个。你也节省了一些流量。

2. 对忽略的实体进行分页？

有人会假设，此特定方法仅用于查看继承类型的实体，因此我希望 Skip 和 Take 仅直接在所述类型的实体上工作。例如，我想跳过 10 个 NotificationUser，而不是 10 个用户（例如，其中只有 4 个是 NotificationUser）。

解决方案：将 ofType 向上移动到查询中

var userNotifications = _context.Notifications
    .OfType<NotificationUser>()
    .OrderByDescending(n => n.DateTime)
    .Skip(offset)
    .Take(limit)
    .Include(n => n.TargetUser)
    .Include(n => n.TargetUser.Images)
    .ToList();

3. 异步/等待

在编写 API 时，您应该考虑使用 async/await，因为它不会阻塞线程，从而浪费更少的资源（如果您还没有使用它，这可能需要您重写很多现有代码）。

请研究 async/await 的优点，并在等待结果等场景中使用它们。

解决方案：改变这个

private List<NotificationUser> GetNotificationUsers(int offset, int limit)
    {
        return _context.Notifications
                .OfType<NotificationUser>()
                .OrderByDescending(n => n.DateTime)
                .Skip(offset)
                .Take(limit)
                .Include(n => n.TargetUser)
                .Include(n => n.TargetUser.Images)
                .ToList();
    }

进入这个

private async Task<List<NotificationUser>> GetNotificationUsersAsync(int offset, int limit)
    {
        return await _context.Notifications
                .OfType<NotificationUser>()
                .OrderByDescending(n => n.DateTime)
                .Skip(offset)
                .Take(limit)
                .Include(n => n.TargetUser)
                .Include(n => n.TargetUser.Images)
                .ToListAsync();
    }

注意： 您还必须更改使用此方法的任何地方

var x = GetNotificationUsers(skip, take);

到

var x = await GetNotificationUsersAsync(skip, take);

并使该方法异步并返回任务

于 2017-11-06T13:55:28.533 回答
1

我不知道您将使用的实体数量。如果可能的话，我会尝试不在数据库服务器上进行联合：

var userNotifications = _context.Notifications.OfType<NotificationUser>()
                                .Include(n => n.TargetUser).ToList();
var placeNotifications = _context.Notifications.OfType<NotificationPlace>().ToList();
var notifications = userNotifications.Union(placeNotifications);

见https://stackoverflow.com/a/27643393/2342504

于 2016-05-06T12:26:09.683 回答
0

已经尝试了很多不同的解决方案，但没有一个符合我的要求，因为我正在处理 API，并且查询必须支持分页并对数据库进行持续请求（而不是将所有实体加载到内存中）。

终于找到了一个解决方案，也许不是最好的，但现在已经足够了。首先，我请求一部分有序数据（分页逻辑）：

var notifications = _context.Notifications
            .OrderByDescending(n => n.DateTime)
            .Skip(offset)
            .Take(limit);

（此时我对任何属性都不感兴趣）接下来，我将获取每种实体类型的已加载项的 ID：

var ids = notifications.OfType<NotificationUser>().Select(n => n.Id).ToList();

最后加载包括所有属性的特定实体：

var userNotifications = _context.Notifications.OfType<NotificationUser>()
             .Include(n => n.TargetUser)
             .Include(n => n.TargetUser.Images)
             .Where(n => ids.Contains(n.Id))
             .ToList();

所有实体都进入列表并再次排序。

这里有很多不好的东西，希望有人能提供更好的解决方案。

于 2016-05-06T13:51:35.413 回答