Discussion :

[yagaam]

Salut les poneys !

Voici donc un petit soucis, j'ai l'impression que mon DataContext ne se met pas à jour lors de la sélection d'un combobox qui doit alimenter en alimenter un autre !

Code :

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

1
2
3
 
<ComboBox Width="150" ItemsSource="{Binding Clients}" SelectedIndex="-1" SelectedItem="{Binding SelectedClient}" />
<ComboBox Width="150" ItemsSource="{Binding Databases}" SelectedIndex="-1" />

Le code C# associé est:

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
 
class ClientVM
{
ClientsDataContext dc;
        private List<string> _clients;
        public List<string> Clients
        {
            get { return _clients; }
            set { _clients = value; OnPropertyChanged("Clients"); }
        }
 
        private List<string> _databases;
        public List<string> Databases
        {
            get { return _databases; }
            set { _databases = value; OnPropertyChanged("Databases"); }
        }
 
        private string _selectedClient;
        public string SelectedClient
        {
            get { return _selectedClient; }
            set { _selectedClient = value; GetClientDatabases(); }
        }
public List<string> GetClients()
        {
            _clients = (from c in dc.Clients orderby c.Name ascending select c.Name).ToList();
            return _clients;
        }
 public void GetClientDatabases()
        {
            var clientID = (from c in dc.Clients where c.Name == _selectedClient select c.Id).First();
            _databases = (from d in dc.Databases orderby d.Name ascending where d.Client_ID == clientID select d.Name).ToList();
        }

A noter qu'en dehors que ma combobox "Databases" ne se met pas à jour, la propriété "Databases/_databases" se met bel et bien à jour
Voili-voilou, je suis preneur de toute bonne idée :p

Dans l'attente de vos nouvelles, Bonne journée

[micka132]

C'est parceque ta source Databases est une List<T>, qui ne procure aucune information de changement (INotifyCollectionChanged).
Utilise à la place une ObservableCollection<T>, et au lieux d'en affecter une nouvelle, tu fais un clear et tu ajoutes les elements issues de ta nouvelle liste.
Utilise cette extension:

Code :

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

1
2
3
4
5
6
7
8
9
10
class RangeEnabledObservableCollection<T> : ObservableCollection<T>
{
    public void InsertRange(IEnumerable<T> items) 
    {
        this.CheckReentrancy();
        foreach(var item in items)
            this.Items.Add(item);
        this.OnCollectionChanged(new NotifyCollectionChangedEventArgs(NotifyCollectionChangedAction.Reset));
    }
}

si tu ne veux pas generer l'evenement de changement de collection à chaque Add (en meme temps si tu as des petites collections c'est pas génant).

[yagaam]

Merci à toi ! ; mon code actuel du coup est:

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
 
private ObservableCollection<string> _clients;
        public ObservableCollection<string> Clients
        {
            get { return _clients; }
            set { _clients = value; OnPropertyChanged("Clients"); OnPropertyChanged("Databases"); }
        }
 
        private RangeEnabledObservableCollection<string> _databases;
        public RangeEnabledObservableCollection<string> Databases
        {
            get { return _databases; }
            set { _databases = value; OnPropertyChanged("Databases"); }
        }
 
        private string _selectedClient;
        public string SelectedClient
        {
            get { return _selectedClient; }
            set { _selectedClient = value; GetClientDatabases(); }
        }
 
 public void GetClients()
        {
            var query = from c in dc.Clients orderby c.Name ascending select c.Name;
            _clients = new ObservableCollection<string>(query);
        }
 
 public void GetClientDatabases()
        {
            var clientID = (from c in dc.Clients where c.Name == _selectedClient select c.Id).First();
            var query = from d in dc.Databases orderby d.Name ascending where d.Client_ID == clientID select d.Name;
            _databases = new RangeEnabledObservableCollection<string>();
            _databases.Clear();
 
 
            foreach (var data in query)
                _databases.Add(data);
        }

Code :

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

1
2
3
4
5
6
7
8
9
 
public partial class DBExplorerView : UserControl
    {
        public DBExplorerView()
        {
            InitializeComponent();
            DataContext = new ClientVM();
        }
    }

Code :

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

1
2
3
 
<ComboBox Width="150" ItemsSource="{Binding Clients}" SelectedItem="{Binding SelectedClient, UpdateSourceTrigger=PropertyChanged}" />
<ComboBox Width="150" ItemsSource="{Binding Databases}" />

Fonctionne pas ; même résultat

[micka132]

Oublie l'observableCollection tu n'en as pas besoin dans ton cas de figure (et en plus utilises incorrectement l'extension )).
En reprenant ton 1er code il faut juste passer par la proprété Databases et non le champs _databases, de ce fait tu passeras automatiquement par le Set (et donc le OnPropertyChanged("Databases")

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
class ClientVM
{
ClientsDataContext dc;
        private List<string> _clients;
        public List<string> Clients
        {
            get { return _clients; }
            set { _clients = value; OnPropertyChanged("Clients"); }
        }

        private List<string> _databases;
        public List<string> Databases
        {
            get { return _databases; }
            set { _databases = value; OnPropertyChanged("Databases"); }
        }

        private string _selectedClient;
        public string SelectedClient
        {
            get { return _selectedClient; }
            set { _selectedClient = value; GetClientDatabases(); }
        }
public List<string> GetClients()
        {
            _clients = (from c in dc.Clients orderby c.Name ascending select c.Name).ToList();
            return _clients;
        }
 public void GetClientDatabases()
        {
            var clientID = (from c in dc.Clients where c.Name == _selectedClient select c.Id).First();
           Databases = (from d in dc.Databases orderby d.Name ascending where d.Client_ID == clientID select d.Name).ToList();
        }

De manière génerale il vaut toujours mieux passer par la propriété que par le champ, justement pour bénéficier du Get/Set.

[yagaam]

Hello,

Le combobox continu de ne pas se remplir ^^ Mais Databases récupère bien les bonnes informations

bizare non ?

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
 
 class ClientVM : RangeEnabledObservableCollection<ClientModel>
    {
        #region properties
        ClientsDataContext dc;
        private List<string> _clients;
        public List<string> Clients
        {
            get { return _clients; }
            set { _clients = value; OnPropertyChanged("Clients"); }
        }
 
        private List<string> _databases;
        public List<string> Databases
        {
            get { return _databases; }
            set { _databases = value; OnPropertyChanged("Databases"); }
        }
 
        private string _selectedClient;
        public string SelectedClient
        {
            get { return _selectedClient; }
            set { _selectedClient = value; GetClientDatabases(); }
        }
 
 
        private RelayCommand _openClientFormCommand;
        public RelayCommand OpenClientFormCommand
        {
            get
            {
                if (_openClientFormCommand == null)
                    _openClientFormCommand = new RelayCommand(OpenClientForm);
                return _openClientFormCommand;
            }
        }
 
        #endregion
 
        #region constructor
        public ClientVM()
        {
            dc = new ClientsDataContext();
            GetClients();
        }
        #endregion
 
        #region methods
 
        public List<string> GetClients()
        {
            _clients = (from c in dc.Clients orderby c.Name ascending select c.Name).ToList();
            return _clients;
        }
        public void GetClientDatabases()
        {
            var clientID = (from c in dc.Clients where c.Name == _selectedClient select c.Id).First();
            Databases = (from d in dc.Databases orderby d.Name ascending where d.Client_ID == clientID select d.Name).ToList();
        }
 
        public void OpenClientForm(object o)
        {
            ClientFormView clientView = new ClientFormView();
            clientView.Show();
        }
        #endregion
 
        #region PropertyChanged
        public event PropertyChangedEventHandler PropertyChanged;
        public void OnPropertyChanged(string propertyName)
        {
            if (PropertyChanged != null)
                PropertyChanged(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
 
 class RangeEnabledObservableCollection<T> : ObservableCollection<T>
    {
        public void InsertRange(IEnumerable<T> items)
        {
            CheckReentrancy();
            foreach (var item in items)
                Items.Add(item);
            OnCollectionChanged(new NotifyCollectionChangedEventArgs(NotifyCollectionChangedAction.Reset));
        }
    }

Code :

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

1
2
3
 
<ComboBox Width="150" ItemsSource="{Binding Clients}" SelectedItem="{Binding SelectedClient, UpdateSourceTrigger=PropertyChanged}" />
<ComboBox Width="150" ItemsSource="{Binding Databases}" />

[micka132]

Mise à part que tu devrais hériter simplement de INotifyPropertyChanged et pas de RangeEnabledObservableCollection, je vois rien d'etrange

[yagaam]

Et TADAAAAM,

Merci à toi

Bonne journée !