Discussion :

[Immobilis]

Salut,

J'ai une classe de base et des dérivées. EF:

• créé une table pour chaque objet;
• me permet de mettre à jour les entités à partir de la table de base.

Constat: le code est le même pour réaliser les opérations du CRUD sur les classes enfant seul le Type à traiter change.

Question: comment réaliser un manager générique acceptant n'importe lequel des objets?

Le model:

Le code:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.UI;
using System.Web.UI.WebControls;
 
namespace WebApplication1
{
    public class VehiculeManager : IDisposable
    {
        private Model1Container m = new Model1Container();
 
        public void Create(Volant volant)
        {
            m.VehiculeJeu.AddObject(volant);
            m.SaveChanges();
        }
 
        public Volant Read(int id)
        {
            return m.VehiculeJeu.OfType<Volant>().Where(x => x.ID == id).FirstOrDefault();
        }
 
        public void Dispose()
        {
            m.Dispose();
        }
    }
 
    public partial class _Default : System.Web.UI.Page
    {
        protected void Page_Load(object sender, EventArgs e)
        {
            using (var mgr = new VehiculeManager())
            {
                Volant v1 = new Volant();
                v1.Nom = "747";
                v1.ModePropulsion = "Réacteur";
                mgr.Create(v1);
 
                var v2 = mgr.Read(v1.ID);
 
                Response.Write(v2.ModePropulsion + "<br />");
            }
        }
    }
}

Merci d'avance pour vos idées.

Petite précision suplémentaire, le manager doit implémenter une interface ICrud comme celle-ci: http://immobilis.developpez.com/tuto...work/#LIII-A-2

[rad_hass]

Bonjour Immobilis,

Quand tu dis manager ici, c'est un peu le pattern repository non ? Si c'est le cas une solution courante est d'avoir un Repository générique, dont voici un exemple :

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
 
    public abstract class Repository<T> where T : class
    {
        protected DbContext Context;
        public Repository(DbContext context)
        {
            Context = context;
        }
 
        public virtual IEnumerable<T> GetAll()
        {
            return Context.Set<T>();
        }
 
        public abstract T GetById(int id);
 
        public virtual IEnumerable<T> Query(Expression<Func<T, bool>> filter)
        {
            return Context.Set<T>().Where(filter);
        }
 
        public void Add(T entity)
        {
            Context.Set<T>().Add(entity);
        }
 
        public void Remove(T entity)
        {
            Context.Set<T>().Remove(entity);
        }
    }

[Immobilis]

Salut,

Merci de ta réponse c'est une possibilité. Cependant, j'aimerai éviter l'utilisation d'une classe abstraite au profit d'une interface.

En utilisant la classe de base ça passe, mais cela m'oblige à la convertir lors de la lecture:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
public interface ICrud<T> : IDisposable
{
    void Create(T volant);
    T Read(int id);
}
 
public class VehiculeManager : ICrud<Vehicule>
{
    private Model1Container m = new Model1Container();
 
    public void Create(Vehicule vehicule)
    {
        m.VehiculeJeu.AddObject(vehicule);
        m.SaveChanges();
    }
 
    public Vehicule Read(int id)
    {
        return m.VehiculeJeu.Where(x => x.ID == id).FirstOrDefault();
    }
 
    public void Dispose()
    {
        m.Dispose();
    }
}
 
public partial class _Default : System.Web.UI.Page
{
    protected void Page_Load(object sender, EventArgs e)
    {
        using (var mgr = new VehiculeManager())
        {
            Volant v1 = new Volant();
            v1.Nom = "Cessna";
            v1.ModePropulsion = "Hélice";
            mgr.Create(v1);
 
            var v2 = mgr.Read(v1.ID);
 
            Response.Write((v2 as Volant).ModePropulsion + "<br />");
        }
    }
}

Je préférerai manipuler uniquement des interfaces.

[rad_hass]

Salut,

Merci de ta réponse c'est une possibilité. Cependant, j'aimerai éviter l'utilisation d'une classe abstraite au profit d'une interface.

En utilisant la classe de base ça passe, mais cela m'oblige à la convertir lors de la lecture:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
public interface ICrud<T> : IDisposable
{
    void Create(T volant);
    T Read(int id);
}
 
public class VehiculeManager : ICrud<Vehicule>
{
    private Model1Container m = new Model1Container();
 
    public void Create(Vehicule vehicule)
    {
        m.VehiculeJeu.AddObject(vehicule);
        m.SaveChanges();
    }
 
    public Vehicule Read(int id)
    {
        return m.VehiculeJeu.Where(x => x.ID == id).FirstOrDefault();
    }
 
    public void Dispose()
    {
        m.Dispose();
    }
}
 
public partial class _Default : System.Web.UI.Page
{
    protected void Page_Load(object sender, EventArgs e)
    {
        using (var mgr = new VehiculeManager())
        {
            Volant v1 = new Volant();
            v1.Nom = "Cessna";
            v1.ModePropulsion = "Hélice";
            mgr.Create(v1);
 
            var v2 = mgr.Read(v1.ID);
 
            Response.Write((v2 as Volant).ModePropulsion + "<br />");
        }
    }
}

Je préférerai manipuler uniquement des interfaces.

La mettre abstraite c'était un exemple

Dans mon projet, on a une interface IRepository<T> (l'équivalent de ICrud) et une classe Repository<T> qui fais la même chose

[Edit] Après une rapide recherche voici un exemple pour illustrer mon propos :
http://elegantcode.com/2009/12/15/en...ork-prototype/

A+

[Immobilis]

OK, nous ne sommes pas tout à fait sur les mêmes bases. Il faut déjà que j'installe cette version 4.1 du Framework 4. Je teste et te dit.

[Immobilis]

D'ailleurs, si tu as une idée pour implémenter cela avec le FW 4.0 ça me va.

[rad_hass]

C'est un peu la même chose, il faut juste modifier les api's :

Au lieu de DbContext -> ObjectContext
Au lieu de Set<> -> ObjectSet<>
Au lieu de Add -> AddObject

Pour créer un ObjectSet il faut passer par

Code :

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

_objectSet = context.CreateObjectSet<T>();

(le code dans le site c EF 4.0 me semble-t-il)

Ex tirer de la msdn :

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
public class SqlRepository<T> : IRepository<T>
                                where T : class, IEntity {
    public SqlRepository(ObjectContext context) {
        _objectSet = context.CreateObjectSet<T>();
    }
    public IQueryable<T> FindAll() {
        return _objectSet;
    }
    public IQueryable<T> FindWhere(
                           Expression<Func<T, bool>> predicate) {
        return _objectSet.Where(predicate);
    }
    public T FindById(int id) {
        return _objectSet.Single(o => o.Id == id);
    }
    public void Add(T newEntity) {
        _objectSet.AddObject(newEntity);
    }
    public void Remove(T entity) {
        _objectSet.DeleteObject(entity);
    }
    protected ObjectSet<T> _objectSet;
}

http://msdn.microsoft.com/en-us/library/ff714955.aspx

[Immobilis]

Merki!

Voici ton code (un poil modifié) + le mien:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
using System;
using System.Data.Objects;
using System.Linq;
using System.Linq.Expressions;
 
namespace WebApplication1
{
    public interface IEntity
    {
        int ID { get; set; }
    }
 
    public interface ICrud<T> : IDisposable where T : class
    {
        IQueryable<T> FindAll();
        IQueryable<T> FindWhere(Expression<Func<T, bool>> predicate);
        T FindById(int id);
        void Add(T newEntity);
        void Remove(T entity);
    }
 
    public class Repository<T> : ICrud<T> where T : class, IEntity
    {
        private ObjectSet<T> _objectSet;
        private ObjectContext _context = new Model1Container();
        private bool disposed = false;
 
        public Repository()
        {
            _objectSet = _context.CreateObjectSet<T>();
        }
        public IQueryable<T> FindAll()
        {
            return _objectSet;
        }
        public IQueryable<T> FindWhere(Expression<Func<T, bool>> predicate)
        {
            return _objectSet.Where(predicate);
        }
        public T FindById(int id)
        {
            return _objectSet.Single(o => o.ID == id);
        }
        public void Add(T newEntity)
        {
            _objectSet.AddObject(newEntity);
        }
        public void Remove(T entity)
        {
            _objectSet.DeleteObject(entity);
        }
        public void Save()
        {
            _context.SaveChanges();
        }
        #region IDisposable
        public void Dispose()
        {
            if (!disposed)
            {
                Dispose(true);
                GC.SuppressFinalize(this);
            }
            disposed = true;
        }
        ~Repository()
        {
            Dispose(false);
        }
        private void Dispose(bool disposing)
        {
            if (disposing)
            {
                _objectSet = null;
                _context.Dispose();
                _context = null;
            }
        }
        #endregion
    }
 
    partial class Vehicule : IEntity
    { }
 
    public partial class _Default : System.Web.UI.Page
    {
        protected void Page_Load(object sender, EventArgs e)
        {
            using (var repository = new Repository<Vehicule>())
            {
                Volant v1 = new Volant();
                v1.Nom = "Parachute ascensionnel";
                v1.ModePropulsion = "Bateau";
                repository.Add(v1);
                repository.Save();
 
                var v2 = repository.FindById(v1.ID);
                Response.Write((v2 as Volant).ModePropulsion + "<br />");
            }
        }
    }
}

Faudra que je teste les perfs de tout ça.

A+