Sollte ich vor dem Löschen prüfen, ob eine Zeile existiert?

Question

Ich benutze Mssql als db und EF4 als ORM/DAL.
Meine Frage bezieht sich auf den folgenden Code:

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(); 
} 

Wenn die Zeile dieses sehr gut existiert funktioniert, wenn nicht ich eine Ausnahme erhalten (Shop aktualisieren, einfügen oder Anweisung löschen eine unerwartete Anzahl der betroffenen Zeilen . (0) Entities modifiziert worden sein oder gestrichen, da Einheiten geladen wurden aktualisieren Object Einträge)
Soll ich eine Rundreise an der Datenbank vornehmen zu überprüfen, ob die Zeile wie folgt vorhanden:..

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(); 
    } 
} 

Ich denke, dass extra Rundreise irgendwie unnötig ist, denn ohne die EF, mit einfachen SQL konnte ich einfach die Zeile mit einem einzigen DELETE-Befehl löschen.

Was wäre eine bessere Praxis?

Answer 1

Dieser Fehler ist ein Nebeneffekt der optimistischen Parallelität von Entity Framework.

Im Grunde könnte jemand anderes den Datensatz zwischen dem Abrufen gelöscht haben. Oder vielleicht hast du etwas mit der Entität vor diesem Code getan.

Versuchen Sie es in einer isolierten Umgebung auszuführen (z. B. einen Komponententest), um zu sehen, ob das Problem weiterhin auftritt.

Ja, Sie es sicher und erhalten den Datensatz wieder spielen konnte, oder man könnte ObjectContext.Refresh verwenden:

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(); 
    } 
} 

Auf einer seitlichen Anmerkung - vielleicht ist es, weil Ihre Methode statische ist? Wie instanzist du deinen Kontext? Ich hoffe, du verwendest kein Singleton. :(

Es gibt einen ausgezeichneten Artikel über EF Parallelität here, die weiter im Detail erklärt, warum Sie diesen Fehler bekommen, und was kann Schritte getan werden, um das entgegenzuwirken.