asp.net-mvc-3 – C#lock关键字,我想我用错了

我最近在ASP.NET MVC应用程序中发布了多个表单的问题.情况基本上是,如果有人故意敲定提交按钮,他们可以强制数据被多次发布,尽管验证逻辑(服务器和客户端)都是为了禁止这一点.发生这种情况是因为在Transaction.Commit()方法可以在初始请求上运行之前它们的帖子会经过(这都是在nHibernate中完成的)

MVC ActionMethod看起来有点像这样……

public ActionResult Create(ViewModelObject model)
{
 if(ModelState.IsValid)
 {
  // ...

  var member = membershipRepository.GetMember(User.Identity.Name);
  // do stuff with member
  // update member
 }
}

提出了很多解决方案,但是我找到了C#lock语句,并尝试了一下,所以我改变了我的代码看起来像这样……

public ActionResult Create(ViewModelObject model)
{
 if(ModelState.IsValid)
 {
  // ...
  var member = membershipRepository.GetMember(User.Identity.Name);
  lock(member) {     
     // do stuff with member
     // update member
  }
 }
}

有效！我的测试人员都不能重现这个bug了！我们一直在抨击它超过一天,没有人能找到任何缺陷.但我对这个关键字并不是那么经验.我再次查看以澄清……

The lock keyword marks a statement block as a critical section by obtaining the mutual-exclusion lock for a given object,executing a statement,and then releasing the lock

好的,这是有道理的.这是我的问题.

这太简单了

这个解决方案看似简单,直接,清晰,高效和干净.这太简单了.我知道比认为复杂的东西更简单的解决方案更好.所以我想问更有经验的程序员……

有什么不好的事情我应该知道吗？

public IActionResult Submit(MyModel model)
{
    lock (model)
    {
       //will not block since each post generates it's own instance
    }
}

$(document).ready(function(){
    $(':submit').click(function() {
        var $this = $(this);
        if ($this.hasClass('clicked')) {
            alert('You have already clicked on submit,please be patient..');
            return false;
        }
        $this.addClass('clicked');
    });
});