ArrayList.Contains() isn't bool?

[Ankush Bansal]

Just saw this legacy gem

if(myArrayList.Contains(something)==true || myArrayList.Count>0)
{
if(myArrayList.Contains(something)==false)
{
default=something;
}
else
{
default=myArrayList[0];
}
}

this is for C#.net.

[OriginalGriff]

If you look at MSDN[^] you will see it returns a virtual bool, so obviously you can't trust it! :laugh:

Ideological Purity is no substitute for being able to stick your thumb down a pipe to stop the water

[Ankush Bansal]

"virtual bool" so anyone can override to make it float. :-D :thumbsup:

[OriginalGriff]

Well, you wouldn't want it to sink, would you? :laugh:

Ideological Purity is no substitute for being able to stick your thumb down a pipe to stop the water

[Member 2053006]

I love how the first Contains is redundant and the default does not get set if myArrayList is empty.

[Bernhard Hiller]

Still worse: when the array list does not contain "something", the default is set to something. When the array list contains something, the default is set to the first element of the array list, which may or may not be "something"...