Discussion :

[ikeas]

Salut à tous,

J'ai fait un article sympa sur mon blog pour faire un ViewModel Abstrait c'est a dire qu'il va absorber les propriétés du modèle sans mettre une ligne de code
http://www.dsw.fr/blogs/index.php?p=...&c=1&tb=1&pb=1

donnez moi vos avis !!!

Merci.

[alexlem]

Mais pourquoi ne pas implémenter INotifyPropertyChanged dans ton Model dès le départ ?

[ikeas]

En général on a pas la main sur le modèle de base
sinon on s'emmerderait pas a mettre un viewmodel
(en dehors des command et des méthodes ajoutés)

[sundown]

Mais pourquoi ne pas implémenter INotifyPropertyChanged dans ton Model dès le départ ?

A condition d'avoir le contrôle sur le Model. Cela n'est pas systématiquement le cas, selon les projets.

Si tu est maître de ton Model, INPC est la meilleure solution, si c'est une autre équipe qui réalise le Model, tu devras peut être faire avec ce qu'on te donne.

Sinon l'article est interessant, cela me rappelle le framework MicroModel

[alexlem]

Ok, je comprends. En tout cas c'est vraiment une bonne idée .

[tomlev]

Sympa, le principe est assez bien vu

Par contre y a un truc qui me gêne un peu... ce n'est généralement pas une bonne chose que les propriétés du ViewModel accèdent directement à celles du modèle. Sauf dans le cas d'une vue en lecture seule, il vaut mieux avoir une copie locale des valeurs, de façon à pouvoir valider ou annuler les changements.


Un peu dans le même esprit, tu seras sans doute intéressé par cet article, qui décrit un autre mécanisme (à base de dynamic) pour faire quelque chose de similaire

[ikeas]

a mon avis on pourrait rajouter ce mécanisme directement dans le typedescriptor

[alexlem]

Ton truc m'intéresse dans le sens où ça permet de rendre "Notifiable" un Model qui à la base ne l'est pas.

Ce qui me gêne (personnellement) c'est que le ViewModelAbstract impose ce fonctionnement à tout le ViewModel, et ajoute une couche intermédiaire.

Ce que j'aurais aimé faire c'est dire que le type Person fonctionne comme s'il implémentait INPC, sans hériter d'un ViewModel spécifique.

Ce qui s'écrirait comme ça :

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
 
 
        private Notifiable<Person> _myPerson;
 
        public Notifiable<Person> MyPerson
        {
            get
            {
                return _myPerson;
            }
 
            set
            {
                _myPerson = value;
                OnPropertyChanged("MyPerson");
            }
        }

Du coup à partir de ton code, et du lien de tomlev basé sur les Dynamic, j'ai essayé de l'implémenter, et ça fonctionne.

J'ai mis un projet en pièce jointe si ça vous intéresse.

[tomlev]

Ce qui s'écrirait comme ça :

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
 
 
        private Notifiable<Person> _myPerson;
 
        public Notifiable<Person> MyPerson
        {
            get
            {
                return _myPerson;
            }
 
            set
            {
                _myPerson = value;
                OnPropertyChanged("MyPerson");
            }
        }

Excellent, bon boulot

[ikeas]

oui en effet c'est pas mal
jai voulu rester simple pour decoupler l'ensemble sur la base de ce qui existait dans le tronc commun
rajoute ton code sur mon blog ...

[alexlem]

oui en effet c'est pas mal
jai voulu rester simple pour decoupler l'ensemble sur la base de ce qui existait dans le tronc commun
rajoute ton code sur mon blog ...

euh.. je colle le tout dans un commentaire ?