删除前是否应该检查行是否存在？

Question

我使用 Mssql 作为数据库，使用 EF4 作为 ORM/DAL。
我的问题是关于以下代码：

public static void DeleteBuilding(int buildingId, int countryId)
{
    PlayerBuilding playerBuilding = new PlayerBuilding()
    {
        CountryID = countryId,
        BuildingID = buildingId
    };
    Entities.PlayerBuildings.Attach(playerBuilding);
    Entities.PlayerBuildings.DeleteObject(playerBuilding);
    Entities.SaveChanges();
}

如果该行存在，则效果很好，如果不存在，我会得到一个异常（存储更新、插入或删除语句影响了意外数量的行（0 )。自从加载实体后，实体可能已被修改或删除。)
我是否应该像这样往返数据库来检查该行是否存在：

public static void DeleteBuilding(int buildingId, int countryId)
{
    PlayerBuilding playerBuilding = (from p in Entities.PlayerBuildings
                                     where p.BuildingID == buildingId && p.CountryID == countryId
                                     select p).FirstOrDefault();
    if (playerBuilding != null)
    {
        Entities.PlayerBuildings.DeleteObject(playerBuilding);
        Entities.SaveChanges();
    }
}

我认为额外的往返有点不必要，因为如果没有 EF，使用普通 SQL，我可以简单地使用单个 DELETE 命令删除该行。

什么是更好的做法？

Answer 1

该错误是实体框架乐观并发的副作用。

基本上，其他人可能在您检索记录期间删除了该记录。或者也许您在该代码之前已经对实体做了一些操作。

尝试在隔离环境（例如单元测试）中运行它，看看是否仍然遇到问题。

是的，您可以安全起见并再次获取记录，或者您可以使用 ObjectContext.Refresh：

public static void DeleteBuilding(int buildingId, int countryId)
{
    PlayerBuilding playerBuilding = new PlayerBuilding()
    {
        CountryID = countryId,
        BuildingID = buildingId
    };

    try
    {
       Entities.PlayerBuildings.Attach(playerBuilding);
       Entities.PlayerBuildings.DeleteObject(playerBuilding);
       Entities.SaveChanges();
    }
    catch (OptimisticConcurrencyException)
    {
       Entities.Refresh(RefreshMode.ClientWins, playerBuilding);
       Entities.SaveChanges();
    }
}

顺便说一句 - 也许是因为您的方法是静态？你如何实例化你的上下文？我希望你没有使用单例。 :(

有一篇关于 EF 乐观并发的优秀文章 此处，这进一步详细解释了为什么您会得到该错误，以及可以采取哪些步骤来解决该错误。