Question

Given a simple class:

public class Person 
{
    public string FirstName;
    public string LastName;

    public string GetFullName() 
    {
     return FirstName + LastName;
    }
}

The user of this class will populate a List<Person> object by reading an Xml file or some other data source. Should the logic for populating the List be in the Person class or should it just remain in the calling class? In other words, should there be a public List<Persons> GetPersons() method in the Person class or in the calling class? Or should the data accessor be in another class altogether?

I know this is a rather simplistic question but I’m just curious how others typically do it.

Answer 1

What if the data store changes at a later time? What if the Person’s wil no longer be stored in an XML file but rather in a Database? Now you need to change the Person class again. I would say have some kind of interface called “IPeopleRetriever” or something:

public interface IPeopleRetriever
{
   IEnumerable<Person> GetPeople();
}

Then, have a class called XMLPeopleRetriever:

public class XMLPeopleRetriever : IPeopleRetriever
{
   public IEnumerable<Person> GetPeople() { ... }
}

That way, the consumer of your IPeopleRetriever needs to know nothing about where the Person’s came from.