about if statement and arrays C#.Net

Question

4.00/5 (1 vote)

See more:

My html

<tr <%# FormatColorRow(DataBinder.Eval(Container.DataItem,"bugstoreid").ToString()) %>  id='A1'     ><tr<%#>

I want to highlight rows containing bugstoreid which could be multiple
called my function as

protected string FormatColorRow(string bugstoreid)
    {       
       string[] names = new string[] { "170", "171"};
       //*my problem goes here.

       if (bugstoreid == names[0].ToString() || bugstoreid == names[1].ToString())
       {
          return "style='backGround-color:LightBlue'";
       }  
       
       return "style='backgroundColor='''";
    }

Names are as multiple as possible can be {"170","171",......."",...........}
I don't know how to declare if statement here to return style with all names at once

string names=new string{"170,"171",....}

value of table entity seperated by comma ...please help me.
Thanks in advance!

[edit]Code block added, "Ignore HTML..." option disabled, general tidy - OriginalGriff[/edit]

Posted 6-Mar-11 21:11pm

maya-123

Updated 6-Mar-11 21:27pm

Sergey Alexandrovich Kryukov

v3

Add a Solution

3 solutions

Solution 1

Hi Preeti,

You can set Row Color from code behind. using following method

C#

protected void GridView1_RowDataBound(object sender, GridViewRowEventArgs e)
    {
        if (e.Row.RowType == DataControlRowType.DataRow)
        {
            string bugstoreid = GridView1.DataKeys[0].Value.ToString();
            string[] names = new string[] { "170", "171" };
            if (bugstoreid == names[0].ToString() || bugstoreid == names[1].ToString())
            {
                e.Row.BackColor = "LightBlue";
            }
        }
    }

Thanks,
Imdadhusen

Posted 6-Mar-11 21:18pm

Sunasara Imdadhusen

Comments

Manfred Rudolf Bihy 7-Mar-11 3:40am

My vote of 4:
What if OP decided to add more entries to the names array? Why not use Array.IndexOf(T[], T) to detect if the element was in the array? Even though your solution works it could be rewritten to be more generic.

Solution 3

I suspect (though it is difficult to tell) that you want to replace:

C#

if (bugstoreid == names[0].ToString() || bugstoreid == names[1].ToString())
   {
   return "style='backGround-color:LightBlue'";
   }

with

C#

foreach (string name in names)
   {
   if (bugstoreid == name)
      {
      return "style='backGround-color:LightBlue'";
      }
   }

BTW: you do not need to use ToString on a string: it is one already...

Posted 6-Mar-11 21:19pm

OriginalGriff

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)

jim lahey · Accepted Answer · 2011-03-06T21:18:00

Solution 2

I'd use a generic list of strings instead of an array:

C#

protected string FormatColorRow(string bugstoreid)
    {
       List<string> names = new List<string> { "170", "171"};
       //*my problem goes here.
       if (names.Contains(bugstoreid))
       {
          return "style='backGround-color:LightBlue'";
       }
       return "style='backgroundColor='''";
    }
</string></string>

Hope it hepls.

Posted 6-Mar-11 21:18pm

jim lahey

v2

Comments

Manfred Rudolf Bihy 7-Mar-11 3:43am

Nice solution. 5+
I like it that you wrapped it into a function. I'd make one minor adjustment by making the list of names also a parameter to the function. (Reuse!)

Sunasara Imdadhusen 7-Mar-11 4:44am

My vote is 5,
good call!!