Dois-je vérifier si une ligne existe avant la suppression?

Question

J'utilise Mssql comme db et EF4 comme ORM/DAL.
Ma question concerne le code suivant:

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

Si la ligne existe cela fonctionne très bien, sinon je reçois une exception (mise à jour Store, insertion ou de suppression affecté un nombre inattendu de lignes . (0) les entités peuvent avoir été modifiés ou supprimés car les entités ont été chargées Actualisez entrées ObjectStateManager)
Dois-je faire un aller-retour à la base de données pour vérifier si la ligne existe comme ceci:..

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

Je pense que l'aller-retour supplémentaire est un peu inutile car sans l'EF, avec SQL pur je pourrais simplement supprimer la ligne avec une seule commande DELETE.

Quelle serait une meilleure pratique?

Answer 1

Cette erreur est un effet secondaire de la concurrence optimiste d'Entity Framework. Fondamentalement, quelqu'un d'autre a peut-être supprimé l'enregistrement entre vous l'avoir récupéré. Ou peut-être que vous avez fait quelque chose avec l'entité avant ce code. Essayez de l'exécuter dans un environnement isolé (par exemple un test unitaire) pour voir si le problème persiste.

Oui, vous pourriez jouer en toute sécurité et obtenir à nouveau l'enregistrement, ou vous pouvez utiliser 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(); 
    } 
} 

Sur une note côté - peut-être est parce que votre méthode est statique? Comment instanciez-vous votre contexte? J'espère que vous n'utilisez pas un singleton. :(

Il y a une excellente article sur EF optimiste here concurrency, ce qui explique plus en détail pourquoi votre obtenir cette erreur, et quelles mesures peuvent être prises pour le contrer.