Discussion :

[sp2308]

Bonjour

Je cree une liste de pays avec un bout de code recuperer sur le net...Pas de pb sauf qu il y a des doublons
Je teste alors si le dernier nom de pays est identique au precedent... mais le test ne donne rien !

Où me suis-je planté ?

Merci de votre aide

Mon 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
 List<RegionInfo> countries = new List<RegionInfo>();
             string Last = "";
            foreach (CultureInfo culture in CultureInfo.GetCultures(CultureTypes.SpecificCultures))
            {
                RegionInfo country = new RegionInfo(culture.LCID);
                if (countries.Where(p => p.Name == country.Name).Count() == 0)
                {
                    if (country.EnglishName != Last)
                    {
                        cbx_pays.Items.Add(country.EnglishName);
                    }
 
 
                }
               Last = country.EnglishName;
            }

[sp2308]

Bon j ai solutionné le pb avec ce code :

Code :

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

1
2
3
4
5
 
if (!cbx_pays.Items.Contains(country.EnglishName))
                    {
                        cbx_pays.Items.Add(country.EnglishName);
                    }

mais si l un d entre vous pouvait me dire où je me suis planté dans le code precedent, ce serait sympa .

[Invité]

Bonjour,

J'ai regardé ton code et je n'ai vu aucun endroit où ta liste "countries" était renseignée. Du coup ton test

Code :

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

if (countries.Where(p => p.Name == country.Name).Count() == 0)

sera toujours vrai!

Deuxième point : se baser sur l'entrée précédente pour vérifier s'il n'y a pas de doublons est faux car la documentation MSDN de la fonction "GetCultures" précise bien que la liste n'est pas triée.

Dernier point : puisque tu utilises Linq, utilises-le à fond, notamment pour vérifier s'il n'y a pas de doublon.
De plus, utilises aussi Linq pour avoir directement ton résultat, c'est plus simple.
Regarde :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
            IEnumerable<string> englishNames =
                   CultureInfo.GetCultures(CultureTypes.SpecificCultures)
                   .Select(culture => new RegionInfo(culture.LCID).EnglishName)
                   .Distinct();
 
            foreach (string englishName in englishNames)
            {
                cbx_pays.Items.Add(englishName);
            }

Avec cela tu as tous les "EnglishNames" distincts directement et tu n'as plus qu'à remplir ta combo. N'est-ce pas génial Linq?

[sp2308]

Merci
C est vrai c est plus simple