C# LINQ Remove multiple objects from a list in single execution

Question

It seems quite similar, but my question is quite different. I have a list which consists of objects which are implementation of different interface. I want to delete a particular type's objects, but after passing another criteria. A sample code is given below.

 // remove the not required parameters 
foreach (EndPoint endPoint in endPoints)
{                 
    var maps = endPoint.EndPointParameters
        .Where(x => x is IEndPointParamMapCustom)
        .Cast<IEndPointParamMapCustom>()
        .Where(x => !x.IsActive)
        .ToList();

    foreach (var custom in maps)
    {
        endPoint.EndPointParameters.Remove(custom);
    }
}

I want to remove the below foreach loop and remove the objects in the above single LINQ query. Is it possible? Thanks.

In general LINQ should never be used to modify objects but to query them — Tim Schmelter
– Tim Schmelter, Commented Jun 20, 2017 at 10:34
I guess it´s possible but the question is: why? YOur code is pretty straitforward, easy to understand and thus maintainable. Introducing any further LINQ will just mess your code up. — MakePeaceGreatAgain
– MakePeaceGreatAgain, Commented Jun 20, 2017 at 10:34
Given that the Q in LINQ stands for Query, you'll never perform this "single query" mutation without abusing the intent of LINQ. — spender
– spender, Commented Jun 20, 2017 at 10:35

Dmitrii Bychenko · Accepted Answer · 2017-06-20 20:31:07Z

2

You can use the RemoveRange function endPoint.endPointParameters.RemoveRange(maps)

edited Jun 20, 2017 at 20:31

Dmitrii Bychenko

188k20 gold badges178 silver badges231 bronze badges

answered Jun 20, 2017 at 10:42

Mehang Rai

924 bronze badges

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

2 Comments

Foyzul Karim Over a year ago

thanks. but I asked to merge the both statements into one. not just to use removerange.

Foyzul Karim Over a year ago

the syntax you provided gives a build error. it seems you didn't even check the code by yourself.

Dmitrii Bychenko · Accepted Answer · 2017-06-20 10:43:18Z

1

If EndPointParameters is a List<T> ("remove multiple objects from a list") you can try RemoveAll instead of Linq:

// Remove the not required parameters 
foreach (EndPoint endPoint in endPoints)
{ 
    // Remove all items from EndPointParameters that both
    //   1. Implement IEndPointParamMapCustom 
    //   2. Not IsActive 
    endPoint
      .EndPointParameters                
      .RemoveAll(item => (item as IEndPointParamMapCustom)?.IsActive == false);
}

answered Jun 20, 2017 at 10:43

Dmitrii Bychenko

188k20 gold badges178 silver badges231 bronze badges

Collectives™ on Stack Overflow

C# LINQ Remove multiple objects from a list in single execution

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related