首先,我无法修改表格的设计以启用主键的身份规范。
我的问题是如何处理EF,其中用户A和用户B在添加新记录时主键没有冲突。
我正在开发Asp.Net MVC5项目。
我目前的代码:
public void NewFooMember(FooMember fooMember)
{
fooMember.NullGuard("fooMember");
// This is the primary key
fooMember.MemberKey = NextFooMemberKey();
Repo.Add(fooMember);
UnitOfWork.Commit();
}
public int NextFooMemberKey()
{
int? maxFooMemberKey = Repo.Data.Max(fm => (int?)fm.MemberKey);
if (!maxFooMemberKey.HasValue)
{
return 1;
}
return maxFooMemberKey.Value + 1;
}
答案 0 :(得分:2)
您当前的想法有一个缺点,即在插入新记录之前,可以同时对NextFooMemberKey
进行两次不同的调用,从而导致使用相同的密钥将两个记录插入到Db中。
为了解决这个问题,你应该确保这个序列是线程安全的,并在密钥生成和插入功能周围添加lock
调用,以确保一次只有一个线程可以运行该部分代码(确保唯一密钥。)
您还可以对密钥生成功能进行一些优化 - 假设您没有处理多个服务器,您可以缓存最后一个密钥并且每次都增加它 - 这样您就不必进行舍入 - 每次生成密钥时都会访问数据库(您需要解决插入失败的情况,在这种情况下,您需要减少静态保存的密钥)。
private object newFooLock = new object();
public void NewFooMember(FooMember fooMember)
{
fooMember.NullGuard("fooMember");
lock (newFooLock) { // now only one thread can get in here at a time
try
{
fooMember.MemberKey = NextFooMemberKey();
Repo.Add(fooMember);
UnitOfWork.Commit();
}
catch (Exception ex)
{
lastFooKey--; // decrement saved value
}
}
}
private object newFooKeyLock = new object();
private static int? lastFooKey = null;
public int NextFooMemberKey()
{
// also lock the key generation, just to be safe
lock (newFooKeyLock) {
if (lastFooKey == null)
{
// only get from the db if the local value is not yet populated
lastFooKey = Repo.Data.Max(fm => (int?)fm.MemberKey) ?? 1;
}
lastFooKey++;
return lastFooKey.Value;
}
}