C#:例如集合的封装

我想知道其中哪一种被认为是最干净或最好使用的，以及为什么。

其中一个公开乘客列表，让用户添加和删除等。另一个隐藏列表，只让用户枚举它们并使用特殊方法添加。

示例 1

class Bus
{
    public IEnumerable<Person> Passengers { get { return passengers; } }
    private List<Passengers> passengers;

    public Bus()
    {
        passengers = new List<Passenger>();
    }

    public void AddPassenger(Passenger passenger)
    {
        passengers.Add(passenger);
    }
}

var bus = new Bus1();
bus.AddPassenger(new Passenger());
foreach(var passenger in bus.Passengers)
    Console.WriteLine(passenger);

示例 2

class Bus
{
    public List<Person> Passengers { get; private set; }

    public Bus()
    {
        Passengers = new List<Passenger>();
    }
}

var bus = new Bus();
bus.Passengers.Add(new Passenger());
foreach(var passenger in bus.Passengers)
    Console.WriteLine(passenger);

我要说的第一个类是更好的封装。在这种确切的情况下，这可能是更好的方法(因为您可能应该确保它在总线上留有空间等)。但我想可能在某些情况下第二类也可能有用？就像类(class)并不真正关心该列表会发生什么，只要它有一个。你怎么看？

最佳答案

在示例一中，可以改变您的集合。

考虑以下:

var passengers = (List<Passenger>)bus.Passengers;

// Now I have control of the list!
passengers.Add(...);
passengers.Remove(...);

为了解决这个问题，你可以考虑这样的事情:

class Bus
{
  private List<Passenger> passengers;

  // Never expose the original collection
  public IEnumerable<Passenger> Passengers
  {
     get { return passengers.Select(p => p); }
  }

  // Or expose the original collection as read only
  public ReadOnlyCollection<Passenger> ReadOnlyPassengers
  {
     get { return passengers.AsReadOnly(); }
  }

  public void AddPassenger(Passenger passenger)
  {
     passengers.Add(passenger);
  }
 }

关于C#:例如集合的封装，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/1231848/