[Débutant] Eliminer doublons a la creation de la liste

Eliminer doublons a la creation de la liste

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:

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;
            }

Bon j ai solutionné le pb avec ce code :
Code:

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 .

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:

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:

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?

Merci
C est vrai c est plus simple