0

这是我当前的代码

[HttpPost]
    public void AddMenuItem(DateTime MenuDate, string MenuItemIds_String)
    {
        if (string.IsNullOrEmpty(MenuItemIds_String)) { return; }
        var MenuItemIds = MenuItemIds_String.Split(',').Select(x=>int.Parse(x));

        int JamaatId = 1;

        object thisLcok = new object();
        lock (thisLcok)
        {
            bool ParentPresent = db.DailyMenus.SingleOrDefault(x => x.JamaatId == JamaatId && x.MenuDate == MenuDate) != null;
            if (!ParentPresent)
            {
                db.DailyMenus.Add(new DailyMenu { JamaatId = JamaatId, MenuDate = MenuDate, MenuNoteText = string.Empty });
                db.SaveChanges();
            }
        }
        DailyMenu parentMenu = db.DailyMenus.Single(x => x.JamaatId == JamaatId && x.MenuDate == MenuDate);

        foreach(int id in MenuItemIds){
            bool AlreadyExits = parentMenu.DailyMenuItems.SingleOrDefault(x => x.MenuItem == db.MenuItems.Find(id)) != null;
            if (!AlreadyExits)
            {
                db.DailyMenuItems.Add(new DailyMenuItem { MenuItem = db.MenuItems.Find(id), DailyMenu = parentMenu });
                db.SaveChanges();
            }
        }
    }

如果不存在，我只想创建父记录。因为这是一个http post方法。许多人可以同时发布到它，并且可以在同一日期创建多个项目。

放锁并没有解决它。仍在为同一日期创建多个项目。我把锁放对了吗？

4

1 回答 1

3

我把锁放对了吗？

不，让我们看看你的代码：

object thisLcok = new object();
lock (thisLcok)
{
    ...
}

每次都会创建一个新锁 - 所以另一个请求不可能已经拥有锁。

您的锁需要在所有请求之间共享。这意味着它需要是一个字段......要么是静态字段（ick），要么是某个对象中的实例字段，这些字段已经在您的所有请求之间共享。但是，这仍然只能在单个 AppDomain 中、在单个盒子上工作——它不能很好地扩展，或者在 AppDomain 回收的情况下工作。

在我看来，您可能应该使用数据库约束来处理这个问题 - 所以检查记录是否已经存在，如果不存在，请尝试插入它 - 理解如果另一个请求在两者之间出现，您将失败。

于 2013-01-24T06:52:05.507 回答