Communities

Writing
Writing
Codidact Meta
Codidact Meta
The Great Outdoors
The Great Outdoors
Photography & Video
Photography & Video
Scientific Speculation
Scientific Speculation
Cooking
Cooking
Electrical Engineering
Electrical Engineering
Judaism
Judaism
Languages & Linguistics
Languages & Linguistics
Software Development
Software Development
Mathematics
Mathematics
Christianity
Christianity
Code Golf
Code Golf
Music
Music
Physics
Physics
Linux Systems
Linux Systems
Power Users
Power Users
Tabletop RPGs
Tabletop RPGs
Community Proposals
Community Proposals
tag:snake search within a tag
answers:0 unanswered questions
user:xxxx search by author id
score:0.5 posts with 0.5+ score
"snake oil" exact phrase
votes:4 posts with 4+ votes
created:<1w created < 1 week ago
post_type:xxxx type of post
Search help
Notifications
Mark all as read See all your notifications »
Q&A

Welcome to Software Development on Codidact!

Will you help us build our independent community of developers helping developers? We're small and trying to grow. We welcome questions about all aspects of software development, from design to code to QA and more. Got questions? Got answers? Got code you'd like someone to review? Please join us.

Update list based on presence of identifier in a second list

+8
−0

In C#, I have two lists and need to mark records in the first based on the second. Here's a sample:

public class Vehicle
{
    public string Make { get; set; }
    public string VIN { get; set; }
    public string Color { get; set; }
    public bool HasRegistration { get; set; } = false;
}

The second list is simply a set of VINs (List<string>) that have been registered. I need to mark all the vehicles in the List<Vehicle> which have their VIN somewhere in the List<string>. This is what I have, and it works, but I am looking for something more LINQ.

foreach(var v in vehicles)
{
    if (vinList.Contains(v.VIN))
    {
       v.HasRegistration = true;
    }
}

I could also do v.HasRegistration = (vinList.Contains(v.VIN));

Can this be done in a single, LINQ statement?

History
Why does this post require attention from curators or moderators?
You might want to add some details to your flag.
Why should this post be closed?

1 comment thread

General comments (3 comments)

3 answers

You are accessing this answer with a direct link, so it's being shown above all other answers regardless of its score. You can return to the normal view.

+7
−0

This can't be done with the standard library. It's quite common for people who want to do this to define their own extension method

public static void Foreach<T>(this IEnumerable<T> items, Action<T> action)
{
    foreach (var item in items) action(item);
}

which would allow the one-liner

vehicles.Foreach(v => v.HasRegistration |= vinList.Contains(v.VIN));

There doesn't appear to be a consensus on whether this is an elegant and useful extension method or an abomination, so check your local style guide first.


Although it's not strictly an answer to the question, I must also point out that Contains on a list is a linear time operation, and unless you can guarantee that the list will never hold more than about three VINs you should first convert it to a data structure with a fast lookup. E.g.

var vinLookup = vinList.ToHashSet();
vehicles.Foreach(v => v.HasRegistration |= vinLookup.Contains(v.VIN));

If you don't have ToHashSet (in the standard library it's newish) then the implementation is just new HashSet<string>(vinList).

History
Why does this post require attention from curators or moderators?
You might want to add some details to your flag.

1 comment thread

Specifically for the OP's question about List there is already a ForEach method defined. It's been th... (1 comment)
+6
−0

If you want to be more Linq-like without creating a ForEach method:

vehicles = vehicles
   .Select(
       v =>
          new Vehicle
          {
             Make = v.Make,
             VIN = v.VIN,
             Color = v.Color,
             HsRegistration = vinList.Contains(v.VIN)
          }
   ).ToList();

Here we're treating Vehicle and the list like immutable objects, and creating modified copies rather than mutating them.

Can you make Vehicle a record rather than a class?

record VehicleRecord
{
    internal string Make;
    internal string VIN;
    internal string Color;
    internal bool HasRegistration;
}

If so you can simplify this to:

vehicles = vehicles
    .Select(
         v => v with { HasRegistration = vinList.Contains(v.VIN) }
    ).ToList();
History
Why does this post require attention from curators or moderators?
You might want to add some details to your flag.

0 comment threads

+4
−0

Adding to Peter Taylor's reply:

The OP requested something "more LINQ". I guess it can't get any "more linq" than... MoreLINQ!

That library also has a ForEach().

If you only need that, maybe it's a bit much to use that lib, but if you like the other stuff in there, too, which I suspect... Was not the question, but hint anyway: their MaxBy, MinBy functions are neat.

History
Why does this post require attention from curators or moderators?
You might want to add some details to your flag.

0 comment threads

Sign up to answer this question »