views:

80

answers:

5

I have a strongly typed list of custom objects, MyObject, which has a property Id along with some other properties.
Let's say that the Id of a MyObject defines it as unique and I want to check if my collection doesn't already have a MyObject object that has an Id of 1 before I add my new MyObject to the collection.
I want to use if(!List.Contains(myObj)) but how do I enforce the fact that only one or two properties of MyObject define it as unique?
I can use IComparable? Or do I only have to override an Equals method but I'd need to inherit something first is that right?



Thanks

+1  A: 

You can use LINQ to do this pretty easily.

var result = MyCollection.Where(p=>p.myId == Id).Count();
if(result > 0)
{
     //something
}
Robaticus
oh right on. that seems alright.
towps
Slightly more concise would be: MyCollection.Any(x => x.myId == Id)
Kirk Woll
Not only more concise, but this would have to iterate over the entire collection. `Any` would short circut upon the first match.
Anthony Pegram
Sweet. Good point! +1 to both of you. Spend it however you please.
Robaticus
I'm going to blow it all on hats.
Anthony Pegram
A: 

You can use IEquatable<T>. Implement this in your class, and then check to see if the T passed to the Equals has the same Id as this.Id. I'm sure this works for checking a key in a dictionary, but I've not used it for a collection.

davisoa
+1  A: 

You can override Equals and GetHashCode, implement an IEqualityComparer<MyObject> and use that in the Contains call, or use an extension method like Any

if (!myList.Any(obj => obj.Property == obj2.Property && obj.Property2 == obj2.Property2))
   myList.Add(obj2);
Anthony Pegram
+1, this is a good high-performance solution, especially if you use sets or dictionaries. (speaking to the Equals/GetHashCode point)
Kirk Woll
+6  A: 

List<T> uses the comparer returned by EqualityComparer<T>.Default and according to the documentation for that:

The Default property checks whether type T implements the System.IEquatable(Of T) interface and, if so, returns an EqualityComparer(Of T) that uses that implementation. Otherwise, it returns an EqualityComparer(Of T) that uses the overrides of Object.Equals and Object.GetHashCode provided by T.

So you can either implement IEquatable<T> on your custom class, or override the Equals (and GetHashCode) methods to do the comparison by the properties you require. Alternatively you could use linq:

bool contains = list.Any(i => i.Id == obj.Id);
Lee
I would go with the Linq statement. If you wanted, you could implement an extension method Contains<T>(this List<T> list, T item, Func<T,T,bool> comparer) that would look like List.Contains but add the ability to provide a comparison delegate. It would wrap a Linq statement similar to what's provided here.
KeithS
+1  A: 

List<T>.Contains uses EqualityComparer<T>.Default and which in term uses IEquatable<T> if the type implements it, and object.Equals otherwise.

You could just implement IEquatable<T> but it's a good idea to override object.Equals if you do so, and a very good idea to override GetHashCode() if you do that:

public class SomeIDdClass : IEquatable<SomeIDdClass>
{
    private readonly int _id;
    public SomeIDdClass(int id)
    {
        _id = id;
    }
    public int Id
    {
        get { return _id; }
    }
    public bool Equals(SomeIDdClass other)
    {
        return null != other && _id == other._id;
    }
    public override bool Equals(object obj)
    {
        return Equals(obj as SomeIDdClass);
    }
    public override int GetHashCode()
    {
        return _id;
    }
}

Note that the hash code relates to the criteria for equality. This is vital.

This also makes it applicable for any other case where equality, as defined by having the same ID, is useful. If you have a one-of requirement to check if a list has such an object, then I'd probably suggest just doing:

return someList.Any(item => item.Id == cmpItem.Id);
Jon Hanna