Using Linq to replace foreach statement

Question

I'm a little bit confused by the explanations of linq online so I figured I'd make a thread.

I want to replace my foreach statements with linq statements as I'm sure doing little things like this will make me program better.

Example of what I'd like to use LINQ with:

 public bool CheckForAccount(int accountID)
        {
            bool status = false;

            foreach (AccountHolders accountHolder in AccountHoldersList)
            {
                if (accountHolder.AccountNumber == accountID)
                {
                    status = true;
                    break;
                }
            }
            return status;
        }

Please can you provide an explanation of how it works too so I understand what you're doing.

Thanks in advance!

dav_i · Accepted Answer · 2014-04-14 16:26:21Z

6

So probably the most succinct statement would be:

public bool CheckForAccount(int accountId)
{
    return this.AccountHolderList.Any(x => x.AccountNumber == accountId);
}

In English this says

Check AccountHolderList for any cases where the AccountNumber property is equal to accountId. If there are any, return true, otherwise return false.

answered Apr 14, 2014 at 16:26

dav_i

28.2k17 gold badges110 silver badges138 bronze badges

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

6 Comments

Selman Genç Over a year ago

you are one second faster :) +1

Zain Over a year ago

Can you please explain what you're code is saying? thanks in advance

dav_i Over a year ago

@user3245390 Updated to include description of code. Though it should be obvious from the code :)

Ehsan Sajjad Over a year ago

well mine will also work now..:P but lots of downvotes got..:(

Ehsan Sajjad Over a year ago

i was not aware of any() method and its working, today learned new thing from your answer..@dav_i

|

Community · Accepted Answer · 2017-05-23 10:28:54Z

0

both

AccountHolderList.Any(x => x.AccountNumber == accountId);

and

AccountHoldersList.Exists(p => p.AccountNumber == accountId);

work equally well. Here is a more in-depth explanation of the differences: Linq .Any VS .Exists - Whats the difference?

You could also use this:

  var match = AccountHolderList.FirstOrDefault(x => x.AccountNumber == accountId);
  return match != null ? true : false;

if you want to get a reference to the match as well. This works assuming accountId is unique

edited May 23, 2017 at 10:28

CommunityBot

11 silver badge

answered Apr 14, 2014 at 17:09

Mr Balanikas

1,6811 gold badge16 silver badges29 bronze badges

1 Comment

dav_i Over a year ago

return match != null ? true : false; can be replaced just by return match != null;. It currently reads "if match not equal to null returns true return true otherwise return false"

Collectives™ on Stack Overflow

Using Linq to replace foreach statement

2 Answers 2

6 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

6 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related