如何在同一路由控制器中使用和删除

Question

我可以在同一路由控制器中添加和删除实体吗？我尝试了这种方法，添加了作品，但删除实体给了我一个错误。

[HttpPost]
public async Task<ActionResult<Upvote>> AddRemoveUpvote(CreateUpvoteDTO upvoteDtO)
{
    if (_context.Upvotes == null)
    {
        return Problem("Entity set 'ApplicationDbContext.Upvotes'  is null.");
    }
    var upvote = _mapper.Map<Upvote>(upvoteDtO);
    foreach (var item in _context.Upvotes)
    {
        if (item.UserId == upvote.UserId)
        {
            _context.Upvotes.Remove(item);
            await _context.SaveChangesAsync();
            return Ok("Topic unliked");
        }
    }
    _context.Upvotes.Add(upvote);
    await _context.SaveChangesAsync();
    
    return Ok("Topic Liked");
}

这是试图删除的错误：

system.invalidoperationException：已经有一个与此连接关联的开放数据标准，必须首先关闭。 在Microsoft.data.sqlclient.sqlineternconnectiontds.validateconnectionforeforexecute（sqlcommand命令） 在microsoft.data.sqlclient.sqlineternconnection.beginsqltransaction（iNALATIONLEVEL ISO，字符串TransActionName，boolean shordReconnect）

Answer 1

您会遇到该错误，因为您仍在尝试保存更改时upvotes dbset。实体框架DBContext使用与数据库的单个连接。我看到的第二个问题是，您在upvotes dbset中的每个呼叫中​​的每个记录中都循环循环。这是效率低下的，随着时间的流逝，随着更多记录添加到该表中，随着时间的流逝，这会变得越来越慢。而是让EF仅将值过滤到相关记录。

您的代码的整洁版本看起来像这样：

// See if an existing upvote exists...
var existingUpvote = await _context.Upvotes
    .Where(uv => uv.UserId == upvote.UserId)
    .SingleOrDefaultAsync();

// If it exists, then remove it
if(existingUpvote != null)
{
    _context.Upvotes.Remove(item);
    await _context.SaveChangesAsync();
    return Ok("Topic unlike");
}

// since we reached here there was no existing vote so we can add a new one
_context.Upvotes.Add(upvote);
await _context.SaveChangesAsync();

return Ok("Topic Liked");

此外，我怀疑您还缺少对该主题的支票，因此您的条款可能看起来更像是这样：

var existingUpvote = await _context.Upvotes
    .Where(uv => uv.UserId == upvote.UserId && uv.TopicId == upvote.TopicId)
    .SingleOrDefaultAsync();