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.

Comments on Update list based on presence of identifier in a second list

Parent

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)
Post
+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)
Specifically for the OP's question about List there is already a ForEach method defined. It's been th...
Dana‭ wrote over 3 years ago

Specifically for the OP's question about List there is already a ForEach method defined. It's been there in the List since .NET Framework 2.0 https://docs.microsoft.com/en-us/dotnet/api/system.collections.generic.list-1.foreach