Discussion :

[SIC83]

Bonjour à tous !

Je reprends la suite du post combobox-dynamique-mvvm/ avec une petite question complémentaire. Mais puisqu'elle n'est pas en rapport direct, j'ai présumé qu'il fallait ouvrir une autre discussion.

Pour des soucis d'ergonomie, je souhaite désactiver la combobox ville si la liste retournée après sélection du département est vide.

J'ai donc créé une propriété bool EnableComboVille et je teste la liste retournée.

Code :

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

1
2
3
4
5
6
if (_selectedDepartment != null)
        {
	EnableComboVille = (_cities.Where(c => c.Department == _selectedDepartment).ToList() >0);
            return _cities.Where(c => c.Department == _selectedDepartment).ToList();
        }
        return _allCities;

Y a-t-il plus simple comme approche ?

Merci d'avance,

Frédéric

[micka132]

En terme de performance j'aurais stocker le resultat de "_cities.Where(....)" dans une liste temporaire parceque tu te retapes deux fois l'ensemble de tes Cities.
Bon rien de dramatique m'enfin potentiellement si y a d'autre endroit comme ca tu grattes les centiemes de secondes par-ci par là .

Sinon moi j'aurais plutôt fait un Converter qui prend en parametre Cities et qui test si null ou Count ==0 ou Count >0;

Code Xaml :

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

1
2
3
4
5
6
<ComboBox ItemsSource="{Binding Departments}"
                DisplayMemberPath="Name"
                SelectedItem="{Binding SelectedDepartment, Mode=TwoWay}" />
 
<ComboBox ItemsSource="{Binding Cities}" IsEnabled="{Binding Cities,Converter={StaticResources TonConverter}}"
                DisplayMemberPath="Name" />

Mais ca fait faire un converter en plus... Je crois pas que ca soit plus clair .

[SIC83]

Merci Micka132 !
C'est bien ce que je pensais aussi ... Questionner la base 2 fois, c'est vraiment pas optimisé!

Si personne n'a mieux, je vais regarder ton approche. Un converter me semble effectivement plus approprié qu'une variable booleène.

[BenoitM]

perso j'ai crée une propriété aussi part contre j'ai pas bien compris le code que tu as envoyer

Normalement tu as déjà dans ton modèle la liste des départements donc suffit de tester celle-ci

Code :

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

1
2
3
4
5
6
7
8
9
10
public bool ComboBoxCitiesIsEnabled
        {
            get
            {
                if (IsReadOnly) return false;
                if (listDepartements == null || listDepartements.Count<1
                       || listCities == null || listCities.Count<1) return false;
                return true;
            }
        }

[SIC83]

perso j'ai crée une propriété aussi part contre j'ai pas bien compris le code que tu as envoyer

Normalement tu as déjà dans ton modèle la liste des départements donc suffit de tester celle-ci
public bool ComboBoxCitiesIsEnabled

Code :

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

1
2
3
4
5
6
7
8
9
        {
            get
            {
                if (IsReadOnly) return false;
                if (listDepartements == null || listDepartements.Count<1
                       || listCities == null || listCities.Count<1) return false;
                return true;
            }
        }

En fait, on doit tester les villes ... Si dans un département il n'y a pas de villes, la combo reste disabled. Etant donné que la liste des villes reste entière, c'est la requête linq qui va nous donner le résultat (ça évite de requéter la base).
Celà dit, on peut également faire la requête dans le get de la propriété bool ... Très bonne idée !!

[micka132]

Le problème c'est qu'il manque un notifyPropertyChanged derriere .
Ce qui va revenir à faire un test de nullité pour voir si tu appeles le OnPropertyChanged("EnableComboVille "); on se mord la queue^^

[SIC83]

Le problème c'est qu'il manque un notifyPropertyChanged derriere .
Ce qui va revenir à faire un test de nullité pour voir si tu appeles le OnPropertyChanged("EnableComboVille "); on se mord la queue^^

oups !!!

Merci pour le suivi Micka !

[BenoitM]

Code :

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

1
2
3
4
5
6
7
8
9
public List<Cities> ListCities
{
            get { return _listCities; }
            set
            {
                listCities= value;
                NotifyPropertyChange("ComboBoxCitiesIsEnabled");
            }
}

et tu as surement un events qui quand on change de départements ca reload ListCities :p

[micka132]

Code :

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

1
2
3
4
5
6
7
8
9
public List<Cities> ListCities
{
            get { return _listCities; }
            set
            {
                listCities= value;
                NotifyPropertyChange("ComboBoxCitiesIsEnabled");
            }
}

Oui c'est possible, mais philosphiquement parlant ca n'a pas de sens!