convert foreach loop to linq code

Question

the below code is working as expected but I'm looking to convert the below code to use Linq? any suggestions?

       string[] selections = "Men,Women,Boys".Split(',');           

        int _chkboxId = 0;  
        int _chkboxTextId = 1;
        try
        {
            string id = "lstchk_" + _chkboxId;
            while (!driver.FindElement(By.Id(id)).Equals(null))
            {
                string checkboxId = String.Format("lstchk{0}", _chkboxTextId);
                string checkboxName = driver.FindElement(By.Id(checkboxId)).Text;

                foreach (string match in selections)
                {
                    if (checkboxName == match.Trim())
                    {
                        //matched... do more work here...
                    }
                }
           }

The //matched... do more work here... piece is actually important, according to what you want to do there you could express your Linq in different ways — Wasp
– Wasp, Commented Sep 25, 2012 at 9:40

Ofer Zelig · Accepted Answer · 2012-09-25 02:08:38Z

5

foreach (string match in selections.Where(match => checkboxName == match.Trim()))
{
    //matched... do more work here...
}

answered Sep 25, 2012 at 2:08

Ofer Zelig

17.5k9 gold badges63 silver badges94 bronze badges

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

Comments

horgh · Accepted Answer · 2012-09-25 02:28:32Z

1

If your selections list contains only distinct values, than you could use

if(selections.Any(match=>match.Trim().Equals(checkboxName)))
{
    //Do work
}

instead of your loop. Same if your list may contain non-distinct values, but the work should be done only once for each checkboxName

answered Sep 25, 2012 at 2:28

horgh

18.6k22 gold badges70 silver badges125 bronze badges

Comments

nneonneo · Accepted Answer · 2012-09-25 02:08:40Z

1

switch(checkboxName) {
    case "Men":
    case "Women":
    case "Boys":
        // do more work here
    default:
        // no match, bail
}

Sometimes, there's just a different way to write the code.

answered Sep 25, 2012 at 2:08

nneonneo

181k37 gold badges331 silver badges412 bronze badges

Comments

SwordPhish23 · Accepted Answer · 2013-10-23 12:21:21Z

0

How would you convert this simple foreach to a linq statement?

    public List<string> GetItems()
        {
        var items = new List<string>();

        foreach (Ranorex.ListItem item in ranorexComboBox.Items)
            {
            items.Add(item.Text);
            }

        return items;
        }

answered Oct 23, 2013 at 12:21

SwordPhish23

112 bronze badges

Collectives™ on Stack Overflow

convert foreach loop to linq code

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related