How to show htmlagilitypack scraped data in labels in C#?

Question

0.00/5 (No votes)

See more:

i want to scrape titles of movies from a site. in foreach when i use

messagebox.show(item.InnerText);

it works and it shows the titles but first it won't open the windows form and second it shows a lot of messages.

the code ( i want to make this application in windows form ):

public Form1()
        {
            InitializeComponent();
            

            HtmlAgilityPack.HtmlWeb web = new HtmlAgilityPack.HtmlWeb();
            HtmlAgilityPack.HtmlDocument doc = web.Load("https://yts.mx/");
            foreach (var item in doc.DocumentNode.SelectNodes("//a[@class='browse-movie-title']"))
            {
                label1.Show(item.InnerText);
            }

        }

What I have tried:

i tired this code:

label1.Show(item.InnerText);

it said "no overload for method show takes 1 arguments"
i don't understand this, what does it mean and what should i do to make titles show in like 10-15 labels instead of 10 message boxes?

Posted 30-Jun-21 6:25am

brandon1999

Updated 30-Jun-21 7:08am

Add a Solution

Comments

[no name] 30-Jun-21 15:52pm

Too much "noise" for a constructor; should be using a Window / Form Loaded event; async task; etc.

2 solutions

Solution 1

If you do not understand an issue with a .NET control then you should first check the documentation: Control.Show Method (System.Windows.Forms) | Microsoft Docs[^].

Posted 30-Jun-21 6:35am

Richard MacCutchan

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Dave Kreskowiak · Accepted Answer · 2021-06-30T07:08:00

Solution 2

Read up on the documentation for the Label control.

You change the text a label shows by setting it's Text property to a string.

You would normally NEVER call Show() on a label.

Posted 30-Jun-21 7:08am

Dave Kreskowiak

Comments

brandon1999 30-Jun-21 13:28pm

thank you, i replaced the code with label1.Text = item.InnerText;
but now the problem is that, the code above gets 16 titles and this code only shows 1.
is it possible to make 16 labels that show all the 16 different titles? if yes, how can i do it?

Dave Kreskowiak 30-Jun-21 13:30pm

That's the wrong way to think about this. You have a single string that contains more than what you're expecting. You have to figure out what all that content really is and possibly come up with a method to get what you need and not all the extra junk.

brandon1999 30-Jun-21 13:34pm

the code only gets titles and i want all the titles. i'm looking at htmlagilitypack documentation but there's only two codes, i can either get all of them at once, or i can get only the first title!

Dave Kreskowiak 30-Jun-21 13:35pm

You are getting a single string with all the content. Again, you have to parse that up to get what you want.