Discussion :

[FACBEST]

Bonjour,

je cherche à récupérer le dernier tirage du kéno sur un listView par l'intermédiaire de HtmlAgility. Mon problème c'est que par le fait que j'appelle les balise <p> je me retrouve avec 40 numéros soit les deux tirages du jour au lieu de 20 que je voudrais et là je bloque complètement. Il serais sympa de me donner une piste pour limiter le foreach à 20 lignes. voir 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
string url = "https://www.fdj.fr/jeux/jeux-de-tirage/keno/resultats";
            HtmlWeb web = new HtmlWeb();
            HtmlAgilityPack.HtmlDocument doc = web.Load(url);
 
            var ensemble = (from node in doc.DocumentNode.Descendants("p")
                            where node.Attributes.Contains("class")
                            where node.Attributes["class"].Value == "keno_num"
                            select node);
 
            foreach (HtmlAgilityPack.HtmlNode p in ensemble)
            {
 
                relever_numeros.Items.Add("" + p.InnerText);
 
            }

Merci par avance

[katkiller]

Tu as deux solutions dont une qui me semble plus adaptée a ton besoin :
Une boucle "for", sinon tu peux garder ton "foreach" avec un compteur.

Exemple :

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
Console.WriteLine("For exemple");
            for (int i = 0; i < 20; i++)
            {
                Console.WriteLine(i);
            }
 
            Console.WriteLine("Foreach exemple");
            List<int> elements = new List<int> { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20 };
            int maxElement = 10;
            int compteur = 0;
            foreach (var element in elements)
            {
                compteur++;
                Console.WriteLine(element);
                if (compteur == maxElement)
                    break;
            }

[Jabbal'H]

Un simple Take devrait fonctionner non ?

Code :

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

1
2
3
4
5
6
7
 
var skipList = ensemble.Take(10);
 
foreach(var item in skipList)
{
	relever_numeros.Items.Add("" + p.InnerText);
}

[FACBEST]

Merci pour la solution le code pour tous.

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
relever_numeros.Items.Clear();
 
            int compteur = 0;
            int maxElement = 21;
 
            string url = "https://www.fdj.fr/jeux/jeux-de-tirage/keno/resultats";
            HtmlWeb web = new HtmlWeb();
            HtmlAgilityPack.HtmlDocument doc = web.Load(url);
 
 
             var ensemble = (from node in doc.DocumentNode.Descendants("p")
                            where node.Attributes.Contains("class")
                            where node.Attributes["class"].Value == "keno_num"
                            select node);
 
 
 
             foreach (HtmlAgilityPack.HtmlNode p in ensemble)
             {
 
                compteur++;
                if (compteur == maxElement) break;
                relever_numeros.Items.Add("" + p.InnerText);
 
             }

[Jabbal'H]

Euh le Take ne fonctionne pas ?
Car cette solution est quand même moyenne, tu fais une condition et un break juste pour tester un compteur !

Au pire si le take ne fonctionne pas, fait le avec un for, ce sera plus performant et surtout plus lisible !

[FACBEST]

Je vais voir Merci

[katkiller]

Je confirme que le take est certainement la meilleur solution.

Et la moins bonne est bien le foreach break;

[meziantou]

Je confirme que le take est certainement la meilleur solution.
Et la moins bonne est bien le foreach break;

Meilleur ou moins bonne dans quel sens ?

Car cette solution est quand même moyenne, tu fais une condition et un break juste pour tester un compteur !

Pour info Take fait exactement un foreach break. Donc à par réécrire une méthode qui existe déjà, ça ne change pas grand chose.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
    private static IEnumerable<TSource> TakeIterator<TSource>(IEnumerable<TSource> source, int count)
    {
      if (count > 0)
      {
        foreach (TSource item in source)
        {
          yield return item;
          if (--count == 0)
            break;
        }
      }
    }

[katkiller]

Pour info Take fait exactement un foreach break. Donc à par réécrire une méthode qui existe déjà, ça ne change pas grand chose.

Merci pour l'info

[Jabbal'H]

Autant pour moi, je n'avais pas regarder l'implémentation du Take

Du coup le Take (pour ne pas réécrire la méthode) doit être mieux que le for étant donné qu'il s'agit d'un Enumerable (yield) ?
Après je n'ai pas regardé le source du HtmlAgilityPack

[meziantou]

Du coup le Take (pour ne pas réécrire la méthode) doit être mieux que le for étant donné qu'il s'agit d'un Enumerable (yield) ?

Mieux dans quel sens?

[Jabbal'H]

Chargement différé non ? => le yield ?
L'iterateur n'est pas entièrement chargé, contrairement à un for où il faudrait passer par une liste, ce qui force le chargement complet.
Non ?

[meziantou]

A partir du moment où la source est de type IEnumerable<T>, je ne vois pas trop pourquoi tu le fais avec une boucle for mais si tu insistes :

Code :

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

1
2
3
4
5
6
7
8
 
            using (var enumerator = source.GetEnumerator())
            {
                for (int i = 0; enumerator.MoveNext() && i < 10; i++)
                {
                    Console.WriteLine(enumerator.Current);
                }
            }

[Jabbal'H]

^^ Non c'est pas ce que je voulais dire
Lorsque je disais forcer le chargement, c'est faire çà : enumerateur.ToList()
On charge l'intégralité de la liste, ce qui est contre performant dans ce cas.
Je pense que c'était la solution du for indiqué dans la 1ère réponse

[meziantou]

C'est sûr qu'il faut éviter de faire n'importe quoi. A moins de vouloir itérer plusieurs fois sur les éléments, il faut éviter le ToList().