I was interested in the proper way to deal with null objects. 
I have a Contact Class that contains an Email Collection class.
In certain circumstances the email collection will contain no emails.
When i do something like:
lvContact.SubItems.Add(contact.Emails[0].EmailAddress);
This will throw an aurgument out of range exception since there is no objects in the collection.

I changed the email collection to look like this:

public class EmailCollection: CollectionBase
{
public Email this[int index] 
{
get 
{
if ( List.Count != 0 )
{
return (Email)List[index];
}
else
{
return new Email(0,string.Empty,true);
}
}
set {List[index] = value;}
} 
......

This works but is this the RIGHT way to do this?

Well, the right thing to do would be to check if the object is Null (or Nothing) before trying to access the collection.

If Not IsNothing(Contact.Emails) Then 'get email address

I'm not 100% sure of the C# equivelent,  but I would guess it is something like:

Not Contact.Emails Is Null

And actually, you should just be able to check if Contact.Emails.Count > 0...

So its better to do something like this than dealing with it in the class:

if (contact.Emails.Count > 0) {
lvContact.SubItems.Add(contact.Emails[0].EmailAddress);
} else {
lvContact.SubItems.Add("");
}

brohar,

The best thing to do is not only validate the count, but the value itself.

if (contact.Emails.Count > 0 &&
    contact.Emails[0].EmailAddress != null &&
    contact.Emails[0].EmailAddress.Length > 0) {

    lvContact.SubItems.Add(contact.Emails[0].EmailAddress);
}


You don't want a list full of nulls.  It's unnecessary, and empty strings are just as bad in my opinion.

Yes, in my opinion it is better to handle it directly rather than indirectly through the class.  This way the collection still behaves the way a collection is expected to behave.

The best way is to simply check the number of items in the collection before accessing them by looking at the "Count" property of your collection.

I'd instantiate the Email Collection regardless of if there are any emails or not, in the Contact constructor.  That way the client code won't fail because the Email Collection will always exist even if its count is 0.   Then you can use the foreach statement

foreach (Email in Contacts.Email)
{
//Do stuff and check for possible null field values
}