如何使用多个条件在数据库中选择一个条目？

Question

我正在做一个有趣的项目，我正在保存用户当前的 GuildId、UserId、他们的等级和当前的 Exp。它可以部分工作，但它似乎只抓住了一个条件的第一个结果。

我尝试了多种从数据库中选择项目的不同方法，但由于这是一种爱好，我觉得我遗漏了一些非常明显的东西。我已经记录了我获取 UserId 和 GuildId 的方式，它们返回了正确的值。

如果新条目不存在，它当前会添加一个新条目，并且会正确更新 Xp，但仅限于一个条目。它似乎忽略了 GuildId，只找到与 UserId 匹配的第一个条目并将其保存在那里。

    public static async Task SaveExp(ulong guildId, ulong userId, int level, uint xp, DateTime cooldown)
    {
        using (var dbContext = new DatabaseHandler())
        {
            if (dbContext.Exps.Any(x => x.GuildId == guildId && x.UserId == userId))
            {
                var current = dbContext.Exps.FirstOrDefault(x => x.GuildId == guildId && x.UserId == userId);
                if (current != null)
                {
                    current.Xp += xp;
                    current.Level += level;
                    dbContext.Exps.Update(current);
                }
            }
            else
            {
                dbContext.Exps.Add(new Exp
                {
                    GuildId = guildId,
                    UserId = userId,
                    Level = level,
                    Xp = xp,
                    Cooldown = cooldown
                });
            }
            await dbContext.SaveChangesAsync();
        }
    }

我希望它将 Xp 保存到同时满足 GuildId 和 UserId 条件的条目中。

Answer 1

您可以像这样稍微重构代码以提高效率。您的代码确实执行您所描述的操作。如果您应该只返回一个/null 结果，那么您应该使用SingleOrDefault/SingleOrDefaultAsync，这样可以确保更多结果会导致InvalidOperationException。

public static async Task SaveExp(ulong guildId, ulong userId, int level, uint xp, DateTime cooldown)
{
    using (var dbContext = new DatabaseHandler())
    {
        var current = await dbContext.Exps.FirstOrDefaultAsync(x => x.GuildId == guildId && x.UserId == userId);
        if (current != null)
        {
            current.Xp += xp;
            current.Level += level;
            // you should not need this next line in EF6
            // the entity is already attached as you just retrieved it and should be tracked
            dbContext.Exps.Update(current);
        }
        else
        {
            dbContext.Exps.Add(new Exp
            {
                GuildId = guildId,
                UserId = userId,
                Level = level,
                Xp = xp,
                Cooldown = cooldown
            });
        }
        await dbContext.SaveChangesAsync();
    }
}