C# StringReader Class

Question

I have this problem, I'm using StringReader to find specific words from a textbox, so far is working great, however I need to find a way how to check specific words in every line against a string array.

The following code works:

string txt = txtInput.Text;
string user1 = "adam";
int users = 0;
int systems = 0;

using (StringReader reader = new StringReader(txt))
{
    while ((txt = reader.ReadLine()) != null)
    {
        if (txt.Contains(user1))
        {
            users++;
        }
    }
}

Now, I have created a String Array to store more than one string, but the method Contains seems to only accept a string.

string[] systemsarray = new string[] { "as400", "x500", "mainframe" };

if(txt.Contains(systemsarray))
{
    systems++;
}
// error message: cannot convert from string[] to string

Does anyone have an idea how to do this, or a way to improve it?

Thanks in advance.

Reed Copsey · Accepted Answer · 2013-08-05 19:06:42Z

9

If you are looking for the existence of any of those words in the line, try:

if(systemsarray.Any(word => txt.Contains(word)))
{
    users++;
}

answered Aug 5, 2013 at 19:06

Reed Copsey

567k80 gold badges1.2k silver badges1.4k bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

I4V Over a year ago

Or simply if (systemsarray.Any(txt.Contains))

user7116 Over a year ago

Hard to tell for certain what the OP wants, but if they need a count, replace Any with Count and store that in systems.

Reed Copsey Over a year ago

@Moo-Juice The answer HocsanMoya selected is effectively the same as this one, only unrolled in a loop instead of as a one liner using LINQ.

Hocsan Moya Over a year ago

Hi, I just did, I'm sorry about that.

Anders Arpi · Accepted Answer · 2013-08-05 19:10:38Z

5

Why not write yourself an extension method to do this?

public static class StringExtensionMethods
{
    public static bool ContainsAny(this string self, params string[] toFind)
    {
        bool found = false;
        foreach(var criteria in toFind)
            {
                if (self.Contains(criteria))
                {
                    found = true;
                    break;
                }
            };

        return found;
    }   // eo ContainsAny    
}

Usage:

string[] systemsarray = new string[] { "as400", "x500", "mainframe" };

if(txt.ContainsAny(systemsarray))
{
    systems++;
}
// error message: cannot convert from string[] to string

edited Aug 5, 2013 at 19:10

Anders Arpi

8,4373 gold badges35 silver badges49 bronze badges

answered Aug 5, 2013 at 19:07

Moo-Juice

38.9k11 gold badges83 silver badges132 bronze badges

Comments

LjCode · Accepted Answer · 2013-08-05 19:56:56Z

5

you need to extract each item from your array:

foreach (string item in systemsarray)
{
 if(txt.Contains(item))
 {
    systems++;
 }
}

edited Aug 5, 2013 at 19:56

LjCode

2922 silver badges12 bronze badges

answered Aug 5, 2013 at 19:08

celerno

1,38512 silver badges31 bronze badges

1 Comment

Hocsan Moya Over a year ago

@LuameLudik, sorry to bother again, I was wondering do you know how to get rid of the duplicates? I have tried several things and still not working, I created a new String Array and tried to remove the duplicates... String[] systems2 = new string[] {item}; IEnumerable<String> distinctsystem = systems2.Distinct(); foreach (String theString in distinctsystem) { systems++; }

keyboardP · Accepted Answer · 2013-08-05 19:39:18Z

2

If you want a case-insensitive search (as400 will match AS400), you can do this

if (systemsarray.Any(x => x.Equals(txt, StringComparison.OrdinalIgnoreCase)))
{
    //systemsarray contains txt or TXT or TxT etc...
}

You can remove StringComparison.OrdinalIgnoreCase if you want to take the case into account (or choose a different enum value).

answered Aug 5, 2013 at 19:39

keyboardP

69.5k13 gold badges162 silver badges209 bronze badges

3 Comments

Hocsan Moya Over a year ago

Hi, thanks for your help, something I did was to convert all the text to lower case; like this; string txt = txtInput.Text.ToLower(); but I will take this in consideration, thanks again!

keyboardP Over a year ago

@HocsanMoya - You're welcome. As as side note, converting and checking the lowercase is not recommended. You should usually use the comparison check above or convert it to uppercase instead more info.

Hocsan Moya Over a year ago

Hi! Thanks, in that case, I will follow your advice.

Collectives™ on Stack Overflow

C# StringReader Class

4 Answers 4

4 Comments

Comments

1 Comment

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

4 Comments

Comments

1 Comment

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related