How to find whether a given text is avilable from comma separated string.

Question

3.50/5 (2 votes)

See more:

Hi
I have a string which has all the system names in a comma separated form.
Example:

C#

string systemNames="system1,system2,system3,system4";

From this string I need to find a system name whether the system name exists or not.

For example

I give "system1 " I should get as "TRUE"<br />
I give "system10" I should get as "False"<br />

Kindly Help me to solve this , The code should have any Loop.

Posted 5-Apr-11 1:00am

sacraj

Updated 5-Apr-11 1:02am

Tarun.K.S

v2

Add a Solution

5 solutions

Solution 4

If the pattern of the string remains the same, you cna use this:

string inputValue = originalInput + ",";
bool result = systemNames.ToUpper().Contains(inputValue.ToUpper());

I am not sure if there is any IgnoreCase overload for the Contains method. If there is, you can use that. You can also do trimming wherever needed.

Posted 5-Apr-11 1:18am

dan!sh

Comments

arindamrudra 5-Apr-11 7:25am

Danish, your solution will not work for "system1" and "system10" because "system1" contains in both cases. So, Mahen's answer is more appropriate and Tarun's answer is the best. Please update it.

dan!sh 5-Apr-11 7:33am

You probably missed the line where I have added the comma after the input.

arindamrudra 5-Apr-11 7:42am

Yes, I missed that one. Sorry for my comment. Your answer is correct.

dan!sh 5-Apr-11 7:46am

No problems. :)

lukeer 5-Apr-11 8:36am

But then, what if you should happen to search for the last system in systemNames? That one is not followed by ','.

Manfred Rudolf Bihy 5-Apr-11 8:33am

You'll have to make a minor adjustment or the search for system4 (at the end of the list) will not work. systemNames will have to be fixed so that it'll always end with a comma.
Still my 5+, since it is the easiest way to do it.

Ciumac Sergiu 5-Apr-11 8:38am

The solution won't work if the originalInput is the last item in the systemNames, thus it will be looking for a value which has a "," at the end.

#realJSOP 5-Apr-11 9:16am

You can't assume that the comma comes right after the system name. You really should split the string on commas, and compare the two strings after they're trimmed of white space.

Solution 3

Use split function like this :

C#

string systemNames = "system1,system2,system3,system4";
String[] arr = systemNames.Split(',');

String mySystemName="System1";

foreach(String system in arr)
{
    if(system.Equals(mySystemName))
    {
        //You have found the system
    }
}

Hope it helped! :)

Posted 5-Apr-11 1:10am

Tarun.K.S

Updated 5-Apr-11 1:13am

v2

Solution 5

Hi,
this should be enough,

systemNames.Split(new[] { ',' }).Contains(inputValue);

or if you have a harsh code reviewer :)

systemNames.Split(new[] { ',' }, StringSplitOptions.RemoveEmptyEntries).Select((item)=>item.ToLower(CultureInfo.InvariantCulture).Trim()).Contains(inputValue.ToLower(CultureInfo.InvariantCulture).Trim());

Regards

Posted 5-Apr-11 2:30am

Ciumac Sergiu

Updated 5-Apr-11 3:44am

v3

Comments

Manfred Rudolf Bihy 5-Apr-11 8:34am

Good one! 5+

Tarun.K.S 5-Apr-11 9:01am

I don't think so. System1 and System11 will be same in this case. You should use "Equals" instead of "Contains".

Ciumac Sergiu 5-Apr-11 9:07am

Wrong. Array.Contains() is not the same as string.Contains().

Tarun.K.S 5-Apr-11 9:17am

Oops you are right. Sorry buddy it was my mistake.

#realJSOP 5-Apr-11 9:17am

You have to trim and then normalize the case before performing the comparison. You're making to many assumptions.

Ciumac Sergiu 5-Apr-11 9:27am

Yes, but that's a matter of decision. In an ideal case it would be:
systemNames.Split(new[] { ',' }, StringSplitOptions.RemoveEmptyEntries).Select((item)=>item.ToLower(CultureInfo.InvariantCulture)).Contains(inputValue.ToLower().Trim());

Anyway the person who asked, should decide by himself what cases to handle.

Solution 2

C#

char[] separator = new char[] { ',' };
                string[] strProductId = YourString.Split(separator);
                for (int i = 0; i < strProductId.Length - 1; i++)
                {
                   string  stringId = Convert.Tostring(strProductId[i]);
                    if( string stringId  =="Your string")
                        {
                          flag =  true
                         }
                    else
                       {
                         flag = false
                        }
                     }

Posted 5-Apr-11 1:09am

Mahendra.p25

Updated 5-Apr-11 2:10am

v2

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)

#realJSOP · Accepted Answer · 2011-04-05T02:33:00

C#

private bool SystemExists(string systemName)
{
    text = text.Trim().ToLower();
    string[] parts = systemNames.Split(',');
    int count = (from part in parts
                 where part.Trim().ToLower() == systemName
                 select part).Count();
    return (count > 0);
}

If you want it to be case-sensitive, just remove the calls to string.ToLower().

EDIT ===========

Why was this voted a 1? It's correct, and fulfills the stated needs of the OP.