Comment supprimer des données dans deux Entités (Tables) qui sont liées

**jérémyp8** · 30/10/2014, 11h14

Bonjour à tous

J'ai une table "User" et une table "Proximity" (Une capture d’écran de mon model EDMX pour mieux voir les colonnes de chaque table ainsi leurs relations)

Question

Pourquoi cette méthode supprime juste les utilisateurs associé à la table "Proximity" et PAS aussi dans la table "User" sachant que y a un lien entre les deux tables que je fais dans le code avec Include ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
 public static void DeleteUser_Proximity(int prxIdx)
        {
            try
            {
                using (EDM edm = new EDM())
                {
                    Proximity proximity = edm.Proximity.Include("User").Where(p => p.PrxIdx == prxIdx).FirstOrDefault();
                    if (proximity != null)
                    {
                        while (proximity.User.Count > 0)
                        {
                            User user = proximity.User.First();
                            proximity.User.Remove(user);
                        }
                    }
                    edm.SaveChanges();
                }
            }
            catch (Exception ex)
            {
                Logging.Log(ex);
                throw;
            }
        }

Ce que je veux moi c'est que quand je supprime un User dans la table Proximity il sera supprimé aussi dans la Table "User"

Ps: ma source de donnée est SQL SERVER 2008

Merci à tous

**Alexandre le Grand** · 09/02/2015, 10h47

Envoyé par jérémyp8

Bonjour à tous

J'ai une table "User" et une table "Proximity" (Une capture d’écran de mon model EDMX pour mieux voir les colonnes de chaque table ainsi leurs relations)

Question

Pourquoi cette méthode supprime juste les utilisateurs associé à la table "Proximity" et PAS aussi dans la table "User" sachant que y a un lien entre les deux tables que je fais dans le code avec Include ?

Ce que je veux moi c'est que quand je supprime un User dans la table Proximity il sera supprimé aussi dans la Table "User"

Ps: ma source de donnée est SQL SERVER 2008

Merci à tous

Bonjour,

Il te faudra activer les "delete cascade" dans ta Base de Données, n'oublie de le faire pour les Update aussi."

Cordialement.

**jérémyp8** · 09/02/2015, 16h23

Bonjour Alexandre le Grand

Grace à tes conseils, j'ai pu régler mon problème

Merci énormément

**DotNetMatt** · 09/02/2015, 17h39

Attention avec les opérations en Cascade car même en-dehors d'Entity Framework ce comportement sera présent (ca peut créer de mauvaises surprises en production lorsqu'on a oublié ce détail, et Dieu sait qu'on a vite fait de l'oublier) : "tiens je vais supprimer des infos dans cette table ... ah ben mince une partie de mes campagnes a aussi disparu ..."

Et puis bien sûr le dernier backup remonte à plusieurs jours voire semaines, quand il y en a un...

My 2 cents

**jérémyp8** · 09/02/2015, 17h44

Merci énormément pour tes conseils DotNetMatt

**dfiad77pro** · 09/02/2015, 20h05

Envoyé par DotNetMatt

Attention avec les opérations en Cascade car même en-dehors d'Entity Framework ce comportement sera présent (ca peut créer de mauvaises surprises en production lorsqu'on a oublié ce détail, et Dieu sait qu'on a vite fait de l'oublier) : "tiens je vais supprimer des infos dans cette table ... ah ben mince une partie de mes campagnes a aussi disparu ..."

Et puis bien sûr le dernier backup remonte à plusieurs jours voire semaines, quand il y en a un...

My 2 cents

Oui, sans compter les problèmes de performance avec EF, si tu lance un delete sur 10 000 lignes

**jérémyp8** · 10/02/2015, 08h48

Donc si je comprends bien cette pratique "les opérations en Cascade" ont plus d’inconvenants que d'avantages !

**Alexandre le Grand** · 10/02/2015, 11h14

Perso, je ne comprends pas trop ce qu'ils disent. Pour des tables liées par une clé étrangère, lorsqu'on efface une clé, il faudrait aussi effacer les colonnes de clé(s) étrangère(s) liée à celle ci. Sur SQL SERVER, ça évite les messages d'erreurs.

**dfiad77pro** · 10/02/2015, 11h31

En fait si la fonction de cascade est super pratique niveau codage.
Sur les grosses bases de production, on évite cela, on préfère maîtriser la suppression via des proc stockées, pour des raisons de maintenance et d'optimisations.

en gros si t'a 100 000 000 de lignes ( avec 2-3 liaisons) ça peut te prendre plus 20h pour supprimer en cascades , et le calcul asynchrone des indexes fait monter en flèche la charge de la base.

Enfin je parle surtout sur ORACLE, mais je suppose que ça doit être pareil sur SQL Server, surtout avec l'Entity framework.

Bref, sur un projet perso tu peut l'utiliser sans soucis, mais en entreprise , sur des bases de moyenne à forte volumétrie, je le dé-conseil fortement.

**Alexandre le Grand** · 10/02/2015, 11h38

Merci pour l'éclaircissement.

cordialement.

**DotNetMatt** · 10/02/2015, 11h51

Envoyé par dfiad77pro

Enfin je parle surtout sur ORACLE, mais je suppose que ça doit être pareil sur SQL Server, surtout avec l'Entity framework.

Oui c'est pareil sur tout SGBDR disposant de cette fonctionnalité, peu importe qu'on utilise Entity Framework ou non. Et le pire, c'est si la clé étrangère n'est pas indexée, alors c'est full scan sur toute la table pour trouver les enregistrements à supprimer, puis suppression. Ca peut faire mal

**dfiad77pro** · 10/02/2015, 13h25

Envoyé par DotNetMatt

Oui c'est pareil sur tout SGBDR disposant de cette fonctionnalité, peu importe qu'on utilise Entity Framework ou non. Et le pire, c'est si la clé étrangère n'est pas indexée, alors c'est full scan sur toute la table pour trouver les enregistrements à supprimer, puis suppression. Ca peut faire mal

Quand j'ai débuté avec EF (époque du 4) il balançait une requête par delete donc ça faisait encore plus mal

C'est toujours le cas mais bon c'est justifié,
il y'a plein de raisons pour lesquelles ils n'ont pas encore pris la décision d'implémenter le Batch Delete/update ( y'a tout de même un sujet de travail en cours pour EF7 il me semble).

**Alexandre le Grand** · 10/02/2015, 14h05

Pourrais-je avoir un exemple de procédure delete pour de grandes bases de productions. Je compte insérer pas moins de 1,4 millions de données dans une table lors des tests. J'ai utilisé EF pour l'application que je développe actuellement.

**dfiad77pro** · 10/02/2015, 14h22

Tu ne peut pas insérer 1 millions de lignes directement avec EF, c'est bien trop long et très gourmand en mémoire,
personnellement, je limite à une centaine de lignes ( sur les applications non critiques)

si tu ne peut/veut pas faire ça en procédure stockée, la solution (C#) suivante peut s'appliquer.
Avec cette solution on arrive grosso-modo 100 000 lignes par secondes sur une table de 10 colonnes et 3 indexes ( estimation à la louche

)

Note : j'écris pour oracle , mais avec SQL Server la SYNTAXE varie peu ( @ pour les bind).
C'est sensiblement plus simple pour les deletes, mais j'ai préféré mettre un exemple d'insert

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
        /// <summary>
        /// insère une ligne composée de listes en base
        /// (note mieux vaut utiliser un vrai objet en paramètre)
        /// </summary>
        /// <param name="cols1">colonne 1</param>
        /// <param name="cols2">colonne 2</param>
  public static void INSERT(List<int> cols1, List<decimal> cols2)
        {
  using (var connexion = new OracleConnection()) // changer le provider dans le cadre SQL_SERVER
            {
                string sql = "INSERT INTO TABLE (COL1,COL2 ) VALUES (:VAL_COL1,:VAL_COL2)";
 
                connexion.Open();
 
                OracleCommand cmd = connexion.CreateCommand(); // changer l'objet de commande dans le cadre SQL_SERVER
                cmd.CommandText = sql;
                cmd.CommandType = CommandType.Text;
                cmd.BindByName = true;
                cmd.ArrayBindCount = cols1.Count; // on compte le nombre de ligne à insérer
 
                // Binding des valeurs des lignes à insérer
                cmd.Parameters.Add(":VAL_COL1", OracleDbType.Int32, cols1.ToArray(), ParameterDirection.Input);
                cmd.Parameters.Add(":VAL_COL2", OracleDbType.Decimal, cols2.ToArray(), ParameterDirection.Input);             
 
                try
                {
                    cmd.ExecuteNonQuery();
                  //  CustomLogs.Write(cmd.ArrayBindCount + " BulkInserts  réussi");
                }
                catch
                {
                   //  CustomLogs.Write(cmd.ArrayBindCount + " [ERREUR] BulkInserts  ratés");
                }
 
                connexion.Close();
            }
}

Bon j'ai écris ça très vite, ce n'est pas parfait et paternisé mais ça peut débloquer.
Attention à ne pas envoyer des listes de 1 000 000 de valeurs en une seul fois, il faut envoyer par blocs.

dans le cadre du delete , Tu vide les tables dans leur ordre Hiérarchique :
Petit Fils -->Fils --> Parent , soit 3 requêtes ou le tout dans une proc stockée

au cas ou un peu d'oracle :

Code PL/SQL :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
   PROCEDURE delete_table_test (p_SEQ  IN NUMBER, p_commit  IN VARCHAR2)
   IS
   BEGIN
      -- On efface du fils au parent
      DELETE FROM PETIT_FILS
            WHERE SEQ = p_SEQ;
 
      DELETE FROM FILS
            WHERE SEQF = p_SEQ;
 
      DELETE FROM PERE
            WHERE SEQ= p_SEQ;
 
 
      IF (p_commit = 'O')
      THEN
         COMMIT;
      END IF;
   END delete_table_test ;

ensuite il y a plusieurs solutions de Tuning :
- si c'est une table entière : il faut utiliser les fonctions de vidage de table (TRUNCATE,ETC)
- si c'est plusieurs millions de lignes, on peut désactiver l'indexation et la recalculer plus tard (à voir avec un DBA)
- bref y'a plein de petits trucs à étudier pour trouver la solution adaptée

Important :
- pense à détacher les entités liées ou à utiliser un AsNoTracking() avant de charger les éléments concernés (si tu le peux, dans ton contexte), sinon t'aura une exception de Désynchronisation du contexte.
- J'ai conscience que faire du SQL avec entity c'est chiant et moche, mais pour les opérations en masse , il n'y a pas le choix.

**Alexandre le Grand** · 11/02/2015, 00h03

Pour les insertions de masse , je le fais depuis la base de données SQL SERVER, seulement pour les delete que je souhaiterais tester sur EF.
Merci pour le partage.

Cordialement.

**dfiad77pro** · 11/02/2015, 08h29

ok 1 dernier bout de code alors :

1) détacher une liste d'entitée sur EF 5,6 (DbContext)

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
  foreach (ENTIY e in listEntityTracked)
            {
                context.Entry(e).State = EntityState.Detached;
            }
 
            //Appel de la proc stocké

Après pour SQL server il existe des librairies d'extension pour gérer les batch DELETE, mais comme ça marche pas avec oracle, je ne les utilises pas..

https://github.com/loresoft/EntityFramework.Extended
https://www.nuget.org/packages/EntityFramework.Extended

Comment supprimer des données dans deux Entités (Tables) qui sont liées

Entity Framework

Discussions similaires

Partager

Partager