Display LINQ result as string

Question

4.00/5 (1 vote)

See more:

Hi, I am trying to write a program that searches a text file for multiple keywords, then lists the matches in a ListBox.

The text file I am testing this on has contents of: this is. not a test

I have the matching right but I am having trouble getting a string to show up from LINQ. With the code I have below, only the last match will show up on the ListBox because the for loop assigns a new value to 'matches' every time.

Anything else I try ends up showing "(Collection)" or something that ends in "[System.String]" for each match.

I apologize if I am unclear; I am new to this. I have searched all over and just want to add the results to a list but I can't find anything that works.

If only "select word.ToString()" worked I could do listBox1.Items.Add(matches);
But, that doesn't work either. I would very much appreciate some help on this.

C#

List<string> matches = new List<string>();

string sourceText = System.IO.File.ReadAllText(fileName);

string[] words = sourceText.Split(new char[] { '.', '?', '!', ',', ' ',
    ';', '\'', ':', '"', '-' }, StringSplitOptions.RemoveEmptyEntries);

string[] wordsToMatch = { "this", "is", "a", "test" };

for (int i = 0; i < wordsToMatch.Count(); i++)
{
    matches = (from word in words
             where word.ToLowerInvariant() == wordsToMatch[i].ToLowerInvariant()
             select word).ToList();
 }

foreach (string s in matches)
{
    listBox1.DataSource = matches;
}

Posted 2-May-11 16:57pm

Member 7891073

Add a Solution

3 solutions

Solution 2

I assume you are messing with LINQ and strings sample[^] or a similar one in there.
Anyway, change following

C#

string[] wordsToMatch = { "this", "is", "a", "test" };
for (int i = 0; i < wordsToMatch.Count(); i++)
{
    matches = (from word in words
             where word.ToLowerInvariant() == wordsToMatch[i].ToLowerInvariant()
             select word).ToList();
 }
foreach (string s in matches)
{
    listBox1.DataSource = matches;
}

to

C#

List<string> wordsToMatch = new List<string>()
wordsToMatch.AddRange(new string[]{ "this", "is", "a", "test" });
string[] matches = (from word in words
         where wordsToMatch.Contains(word)
         select word).ToArray();
listBox1.DataSource = matches;
</string></string>

Posted 2-May-11 18:03pm

yesotaso

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)

pankajupadhyay29 · Accepted Answer · 2011-05-02T18:01:00

Change your code as following--

C#

List <string> matches = new List<string>();
 
string sourceText = System.IO.File.ReadAllText(fileName);
 
string[] words = sourceText.Split(new char[] { '.', '?', '!', ',', ' ',
    ';', '\'', ':', '"', '-' }, StringSplitOptions.RemoveEmptyEntries);
 
List<string> wordsToMatch = new List<string>(new string[] { "this", "is", "a", "test" });
 
 matches = (from word in words
             where wordsToMatch.Contains(word.ToLowerInvariant())
             select word).ToList();
 
    foreach (string s in matches)
    {
      listBox1.Items.Add(s);
    }

//or now you can use
//listBox1.DataSource=matches;
// but loop is not required in this case.

Member 7891073 · Accepted Answer · 2011-05-03T14:19:00

Solution 3

Thank you two very much for your help. :)

Posted 3-May-11 14:19pm