Can this be 1 LINQ statement

Question

1.00/5 (1 vote)

See more:

Can this be done in 1 statement for example using OR ||

_scheduleList.Where(c => c.TeamA == "as roma")
           .Select(c =>
           {
               c.TeamA = "roma";
               return c;
           })
           .ToList();

       _scheduleList.Where(c => c.TeamB == "as roma")
           .Select(c =>
           {
               c.TeamB = "roma";
               return c;
           })
           .ToList();

What I have tried:

tried several method but this is the best I can get to.

Posted 18-Aug-22 1:28am

Joe Doe234

Updated 18-Aug-22 11:03am

Add a Solution

Comments

Richard Deeming 19-Aug-22 4:15am

The Select method is supposed to project the source. Adding side-effects to the projection is a really bad idea.

2 solutions

Solution 2

If _scheduleList is a Generic.List you can simply use the ForEach extension to update the list.

C#

_scheduleList.ForEach(c =>
     {
       c.TeamA =  c.TeamA=="as roma" ? "roma" : c.TeamA;
       c.TeamB =  c.TeamB=="as roma" ? "roma" : c.TeamB;
     });

If you wish just to select the items that have been updated, you can do something like this.

C#

var updatedItemsList = _scheduleList.Where(c =>  c.TeamA=="as roma"  || c.TeamB=="as roma")
                                    .Select(c => c).ToList();
 updatedItemsList.ForEach(c =>
 {

   c.TeamA =  c.TeamA=="as roma" ? "roma" : c.TeamA;
   c.TeamB =  c.TeamB=="as roma" ? "roma" : c.TeamB;
 });

Posted 18-Aug-22 11:03am

George Swan

Comments

Richard Deeming 19-Aug-22 4:16am

And if it's not a List<T>, you can just use a foreach loop instead. :)

George Swan 19-Aug-22 4:48am

Yes indeed but it is not as neat

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)

OriginalGriff · Accepted Answer · 2022-08-18T01:39:00

Solution 1

Not really - because you are setting a different field / property of the class when you find what you are looking for, you can't really combine the two tests.
You could combine the two result sets with a Union though:

C#

result = _scheduleList.Where(...).Union(_scheduleList.Where(...));

Posted 18-Aug-22 1:39am

OriginalGriff