Discussion :

[Papy214]

Bonjour tlm,

J'ai une question sur le Databinding.

J'ai un formulaire qui présente un Treeview sur sa partie gauche et un UserControl sur sa partie droite. Le Treeview est rempli avec une liste d'objets User, chaque node contenant un objet dans son Tag.

Le UserControl a une propriété _User avec un getter et un setter.
Quand l'utilisateur sélectionne un node, j'assigne au UserControl._User l'objet contenu dans le Tag. Jusque là, rien de sorcier.

Le setter fait un "bind" des propriétés de l'objet sélectionné sur différents contrôles TextBox. Je pensais faire ce binding une seule fois mais ça ne fonctionne pas. La première fois lie les contrôles à mon premier objet sélectionné mais pas la seconde fois.

Pour contourner ce "problème", je fais maintenant un Databindings.Clear() à chaque sélection puis je refais le binding sur les propriétés.

Trouvant ça un peu lourdingue, je me demandais s'il n'y avait un moyen plus simple pour faire ça et je venais vous demander votre avis.

Papy !

[tomlev]

Si j'ai bien compris, c'est le setter de la propriété User qui ne rafraichit pas les bindings liés à cette propriété ? En fait c'est assez normal : le binding n'a rien de "magique", il ne peut pas deviner que la valeur de la propriété a changé si on ne lui dit pas... Pour cela il faut que ton contrôle implémente l'interface INotifyPropertyChanged, et que le setter de ta propriété déclenche l'évènement PropertyChanged :

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
using System.ComponentModel;
using System.Windows.Forms;
 
public class MonControle : UserControl, INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;
 
    protected virtual void OnPropertyChanged(string propertyName)
    {
        PropertyChangedEventHandler handler = PropertyChanged;
        if (handler != null)
            handler(this, new PropertyChangedEventArgs(propertyName));
    }
 
    private User _user;
    public User User
    {
        get { return _user; }
        set
        {
            _user = value;
            OnPropertyChanged("User");
        }
    }
 
}

[Papy214]

J'ai pourtant bien implémenté INotifyPropertyChange dans ma classe. D'ailleurs, quand la valeur d'une propriété de l'objet est modifiée, le changement de valeur est bien reflété par le contrôle visuel. Apparemment, ce qui ne passe pas, c'est le changement d'objet, même pointé par la même variable, qui pose problème.

Code :

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

1
2
3
4
5
6
7
8
9
this.user = Load(1);
 
this.txtFirstName.DataBindings.Add("Text", this.user, "First_name");
 
// txtFirstName affiche bien le First_name de this.user
 
this.user = Load(2);
 
// txtFirstName affiche toujours le First_name du user chargé par la première ligne.

Je vais quand refaire un test avec un code plus simple pour voir si le bug ne viendrait pas d'ailleurs. Le temps de l'écrire et je reviens.

[Papy214]

ah mais attends ... j'ai parlé trop vite :-(

Toi tu parles de mettre un INotifyPropertyChange au niveau du usercontrol et pas de la classe affichée ...

Ok ... je teste ...

[Papy214]

J'ai modifié mon code mais je dois être un peu dans la lune.

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
 
 
FrameUser.PropertyChanged += new System.ComponentModel.PropertyChangedEventHandler(FrameUser_PropertyChanged);
 
void FrameUser_PropertyChanged(object sender, System.ComponentModel.PropertyChangedEventArgs e)
{
	throw new NotImplementedException();
}
 
private void accounts_AfterSelect(object sender, TreeViewEventArgs e)
{
  UsersEntity user = e.Node.Tag as UsersEntity;
  FrameUser.User = user;
  FrameUser.Visible = true;
  FrameProfile.Visible = false;
}

L'event Propertychanged, je le lie à une fonction dans la code de ma form principale. Mais je ne vois pas où ça me mène. Qu'est-ce qu'il faudrait que je code dans cette procédure ? C'est déjà mon formulaire qui définit ou modifie la valeur de la propriété user de mon UserControl. A quoi ça me sert d'avoir ce retour disant que le user a changé ?

[tomlev]

Tu n'as pas besoin de t'abonner à l'évènement, il faut juste que tu le déclenches. C'est le binding qui va s'abonner à l'évènement

[Papy214]

Je dois pas être très doué mais rien n'y fait. Voilà un code d'exemple que je viens de faire en arrivant ce matin. Si tu vois un truc qui manque ....

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
 
using System.ComponentModel;
using System.Windows.Forms;
 
namespace WindowsFormsApplication1
{
	public partial class ucUser : UserControl, INotifyPropertyChanged
	{
 
		Personne employe;
 
		public Personne Employe
		{
			get { return this.employe; }
			set { this.employe = value; this.OnPropertyChanged("Employe"); }
		}
 
		public void DoBinding()
		{
			this.textBox1.DataBindings.Add("Text", this.Employe, "Nom");
		}
 
		public ucUser()
		{
			InitializeComponent();
		}
 
		#region INotifyPropertyChanged Membres
 
		public event PropertyChangedEventHandler PropertyChanged;
 
		public virtual void OnPropertyChanged(string propertyName)
		{
			PropertyChangedEventHandler handler = PropertyChanged;
			if (handler != null)
				handler(this, new PropertyChangedEventArgs(propertyName));
		}
 
		#endregion
	}
}

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
 
using System;
using System.ComponentModel;
using System.Windows.Forms;
 
namespace WindowsFormsApplication1
{
	public partial class Form1 : Form
	{
 
		Personne[] personnes = { new Personne() { Nom = "riri" }, new Personne() { Nom = "loulou" } };
 
		public Form1()
		{
			InitializeComponent();
		}
 
		private void button1_Click(object sender, EventArgs e)
		{
			ucUser1.Employe = this.personnes[0];
			ucUser1.DoBinding();
		}
 
		private void button2_Click(object sender, EventArgs e)
		{
			ucUser1.Employe = this.personnes[1];
		}
	}
 
	public class Personne : INotifyPropertyChanged
	{
		string nom;
		public string Nom
		{
			get { return this.nom; }
			set
			{
				this.nom = value;
				this.OnPropertyChanged("Nom");
			}
		}
 
		#region INotifyPropertyChanged Membres
 
		public event PropertyChangedEventHandler PropertyChanged;
 
		protected virtual void OnPropertyChanged(string propertyName)
		{
			PropertyChangedEventHandler handler = PropertyChanged;
			if (handler != null)
				handler(this, new PropertyChangedEventArgs(propertyName));
		}
 
		#endregion
	}
}

Le click sur le bouton1 affiche bien riri mais le bouton2 ne change rien dans le textbox.

[tomlev]

ah ok...

est-ce que par hasard ça marcherait si tu remplaçais ça :

Code :

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

this.textBox1.DataBindings.Add("Text", this.Employe, "Nom");

par ça :

Code :

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

this.textBox1.DataBindings.Add("Text", this, "Employe.Nom");

C'est juste une idée comme ça, je suis pas sûr que ça marche...

EDIT: sinon, au pire tu mets des propriétés supplémentaires dans ton UserControl :

Code :

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

1
2
3
4
public string NomEmploye
{
    get { return this.Employe.Nom; }
}

Et dans le setter de Employee tu lèves aussi l'évènement OnPropertyChanged pour "NomEmployee" :

Code :

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

1
2
3
4
5
6
7
8
9
10
public Personne Employe
{
    get { return this.employe; }
    set
    {
        this.employe = value;
        this.OnPropertyChanged("Employe");
        this.OnPropertyChanged("NomEmploye");
    }
}

Et tu fais le binding comme ça :

Code :

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

this.textBox1.DataBindings.Add("Text", this, "NomEmploye");

[Papy214]

this.textBox1.DataBindings.Add("Text", this, "Employe.Nom");

me renvoie :

Impossible de créer une liste enfant pour le champ Employe.

au moment de l' exécution. Je ne savais pas que c'est le moyen pour faire ça.

Bon, tout ça finit par être un peu compliqué. Je vais rester sur mon Databinding.Clear() dans mon programme principal et je vais tester ton idée sur mon exemple.

Merci

[tomlev]

tu as vu mon EDIT dans le post précédent ? c'est un peu plus ch***t, mais je suis presque sûr que ça marche...

[Papy214]

tu as vu mon EDIT dans le post précédent ? c'est un peu plus ch***t, mais je suis presque sûr que ça marche...

Tiens, j'ai pas eu d'email pour m'avertir de ton post ...

Effectivement, comme ça, ça fonctionne. Mais pourquoi ça ne fonctionne pas sur mon objet ?

à y réfléchir, l'objet n'est jamais qu'un pointeur. Donc, en assignant un autre objet à ma variable, le compilo détecterait un changement de valeur du pointeur, mais comme ce pointeur n'est "bindé" à rien du tout visuellement, il ne se passe rien pour les propriétés de l'objet. Si je "bindais" l'adresse de l'objet à un contrôle visuel, je pense que je verrai la modification.

Qu'en dis-tu ?

[tomlev]

Effectivement, comme ça, ça fonctionne. Mais pourquoi ça ne fonctionne pas sur mon objet ?

Quand tu mets this.Employe comme source, le binding s'abonne à l'évènement PropertyChanged de l'objet Employe. Mais si tu remplaces this.Employe, le binding ne le sait pas, parce qu'il n'écoute pas le PropertyChanged de this mais de this.Employee. En gros, tu passes au binding un Employe comme source, mais il ne sait pas que cet Employe est affecté à la propriété Employe du UserControl, car il ne récupère pas "this.Employee" mais juste la valeur correspondante.

Je sais pas si c'est très clair mes explications

[Papy214]

Je sais pas si c'est très clair mes explications

En fait, si ! :-) Il s'agit juste d'un décalage de niveau (level) dans le "chemin d'accès" à l'objet pointé.

Merci à toi grand chef !