实体框架:更新用户失败

时间:2016-10-23 20:51:41

标签: c# entity-framework asp.net-core

我尝试使用用户存储库服务更新用户。我仍在使用IdentityUser,只是将其修改为使用int作为主键而不是字符串/ guid。

一切正常,直到我的Update抛出异常。

所以这是我的BaseRepository课程中的更新方法:

public virtual void Update(T entity)
{
    EntityEntry dbEntityEntry = _context.Entry<T>(entity);
    dbEntityEntry.State = EntityState.Modified;
}

public virtual void Commit()
{
    _context.SaveChanges();
}

电话本身就是:

[HttpPost]
[Route("home/account/edit")]
public ActionResult Edit(User user)
{
    if (!ModelState.IsValid)
        return View(user);

    string id = _userManager.GetUserId(User);
    user.Id = id.To<int>();

    _userService.Update(user);
    _userService.Commit();

    return RedirectToAction("Index");
}

(我觉得没什么特别的)

现在,当Commit保存我的更改时,它会引发异常:

  

Microsoft.EntityFrameworkCore.DbUpdateConcurrencyException:数据库操作预计会影响1行,但实际上会影响0行。自加载实体以来,数据可能已被修改或删除。有关理解和处理乐观并发异常的信息,请参阅http://go.microsoft.com/fwlink/?LinkId=527962

我想,好吧,我检查一下探查器,不是那么棘手。 Profiler向我展示了声明,实体框架发给他:

exec sp_executesql N'SET NOCOUNT ON;
UPDATE [AspNetUsers] SET [AccessFailedCount] = @p0, [Birthday] = @p1, [ConcurrencyStamp] = @p2, [Email] = @p3, [EmailConfirmed] = @p4, [FirstName] = @p5, [Language] = @p6, [LastName] = @p7, [LockoutEnabled] = @p8, [LockoutEnd] = @p9, [MiddleName] = @p10, [NormalizedEmail] = @p11, [NormalizedUserName] = @p12, [PasswordHash] = @p13, [PhoneNumber] = @p14, [PhoneNumberConfirmed] = @p15, [SecurityStamp] = @p16, [TwoFactorEnabled] = @p17, [UserName] = @p18, [VerificationToken] = @p19
WHERE [Id] = @p20 AND [ConcurrencyStamp] = @p21;
SELECT @@ROWCOUNT;
',N'@p20 int,@p0 int,@p1 datetime2(7),@p2 nvarchar(4000),@p21 nvarchar(4000),@p3 nvarchar(256),@p4 bit,@p5 nvarchar(4000),@p6 nvarchar(4000),@p7 nvarchar(4000),@p8 bit,@p9 nvarchar(4000),@p10 nvarchar(4000),@p11 nvarchar(256),@p12 nvarchar(256),@p13 nvarchar(4000),@p14 nvarchar(4000),@p15 bit,@p16 nvarchar(4000),@p17 bit,@p18 nvarchar(256),@p19 nvarchar(4000)',@p20=1,@p0=0,@p1='1991-06-10 00:00:00',@p2=N'24fe11c5-a831-45ce-8960-16aa78b280df',@p21=N'24fe11c5-a831-45ce-8960-16aa78b280df',@p3=N'my@email.ch',@p4=0,@p5=N'Matthias',@p6=N'de-DE',@p7=N'Burger',@p8=0,@p9=NULL,@p10=NULL,@p11=NULL,@p12=NULL,@p13=NULL,@p14=NULL,@p15=0,@p16=NULL,@p17=0,@p18=NULL,@p19=NULL;

所以,[ConcurrencyStamp] = @p21@p21=N'24fe11c5-a831-45ce-8960-16aa78b280df'

检查了该用户的记录......

ConcurrencyStamp22fa7d03-1382-4eee-8c17-878253fdf1c4

所以我的问题是:

  • 那是ConcurrencyStamp
  • 为什么Entity Framework会为它过滤?他获得了我的用户名。那还不够吗?
  • 为什么用户对象是另一个ConcurrencyStamp而不是数据库中的对象?当实体框架想要对其进行过滤时,它们不应该相同吗?
  • 有人有想法吗?怎么可能/我应该修复它?或者我做错了什么?

更新

模型(现在也是viewmodel):

public class User : IdentityUser<int>, IEntity
{
    public string FirstName { get; set; }
    public string MiddleName { get; set; }
    public string LastName { get; set; }
    public DateTime Birthday { get; set; }
    public string Language { get; set; }

    public ICollection<MailBox> MailBoxes { get; set; } = new List<MailBox>();
    public string VerificationToken { get; set; }
}

1 个答案:

答案 0 :(得分:3)

ConcurrencyStamp是为了防止两个人对同一个用户进行两次不同的更改,而不会发现彼此的更改。

    /// <summary>
    /// A random value that must change whenever a user is persisted to the store
    /// </summary>
    public virtual string ConcurrencyStamp { get; set; } = Guid.NewGuid().ToString();

原则上,这应该在创建用户时分配一次,并且(我假设)数据库中应该存在类似触发器的机制,只要数据库中的条目更新,就会导致值发生更改

来自this answer

  

如果你在OnModelCreating()方法中查看IdentityDbContext的实现,你会发现:

builder.Entity<TUser>(b =>
{
....
    b.Property(u => u.ConcurrencyStamp).IsConcurrencyToken();
....

因此,实体框架意识到这应该被视为并发令牌。它会检查属性的值以及ID,以确保自用户对象从数据库加载后用户已更改时更新失败。

至于为什么价值不相同,你可能需要通过调试来解决这个问题。以下是您可能会检查的一些理论:

  1. 也许用户对象在加载后确实被更改了?如果用户对象以其原始值加载两次,然后独立更改两次,则会发生这种情况。
  2. 也许您正在根据传递给Web请求的值构建用户对象,但是您没有根据从数据库中提取的值来映射ConcurrencyStamp令牌,因此它和&# #39;保留随机生成的GUID,它是在第一次构建时得到的?