HtmlAgilityPack problème de code

Version imprimable

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:

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

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:

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

Un simple Take devrait fonctionner non ?

Code:

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

Merci pour la solution le code pour tous.

Code:

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

26/11/2013, 13h14
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 !
26/11/2013, 14h29
FACBEST

Je vais voir Merci
26/11/2013, 14h37
katkiller

Je confirme que le take est certainement la meilleur solution.

Et la moins bonne est bien le foreach break;

Citation:

Envoyé par katkiller

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 ?

Citation:

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:

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

26/11/2013, 15h08
katkiller

Citation:

Envoyé par meziantou

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 :P
26/11/2013, 15h47
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
26/11/2013, 15h53
meziantou

Citation:

Envoyé par Jabbal'H

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?
26/11/2013, 16h12
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 ?

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:

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

26/11/2013, 17h00
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
26/11/2013, 17h05
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().