简单的场景，如何融入“告诉不要问”？

Question

我正在尝试对涉及人员和座位的基本场景进行建模。 一个人有一个状态属性：坐着或站立。 座位有一个 Seated 属性，用于指定当前坐在座位上的人。 此外，座位的特殊之处在于它只“接受”某些人坐在其中。 我知道席位“接受”某人听起来很奇怪，但想象一下它比其他人更喜欢某些人。

遵循“告诉，不要问”，我应该如何设计人员和Seat 对象使得只有当 Seat“接受”某人并且他的状态更改为 Sitting 时，该人才能坐在该 Seat 上。 我的第一个想法是，一个人应该有一个 SitDown 方法，如下所示：

Person.SitDown(Seat seat);

但这似乎需要 Person 类在坐下之前检查座位的状态，并且必须更新座位的 Seated 属性（而不是Seat 本身更新属性）：

// inside the Person class
void SitDown(Seat seat) {
    if (seat.AcceptsPlayer(this)) {
        seat.Seated = this;
        this.Status = Sitting;
    }
}

让 Seat 类处理一个人的座位似乎更好：

Seat.SeatPerson(Person person);

// inside Seat class
void SeatPerson(Person person) {
    if (IsAccepted(person)) {
        this.Seated = person;
        person.Status = Sitting;
    }
}

但这仍然需要 Seat 来更改该人的状态。 这是应该更新此人的状态的方式吗？ 难道只有一个人才能改变他的地位吗？ 您将如何模拟这个简单的场景？

Answer 1

引入第三个模型……既涉及座位又涉及人员的座位。 然后，您可以在每次有人坐下时创建该模型的实例，添加一些验证以防止两个人坐在同一个座位上，甚至可能添加一些超时（如果您坐在座位上的时间太长，您就会失去它） 。

Answer 2

闻起来像是你需要座位服务。 它可以容纳一个座位和一个人。 然后决定是否可以进行手术。

这样，该人只需负责将自己标记为就座和位置。
该席位只负责将自己标记为“已占用”。

检查人员和座位是否符合标准是座位服务部门的责任。

Answer 3

问题是您的模型是用循环依赖定义的。 有两种方法可以避免这种情况。

第一个并没有明确遵循“告诉，不要问”，但它更接近要点。 我们试着看看是否可以坐下，然后告诉椅子我们坐在上面。

void Person.SitDown(Seat seat) {
    if (seat.AcceptsPlayer(this)) {
        seat.SeatPerson(this);
        this.Status = Status.Sitting;
    }
}

void Seat.SeatPerson(Person person) {
    this.Seated = person;
}

更好的方法（更明确地遵循“告诉，不要问”）可能如下。 我们尝试坐在椅子上。 如果主席拒绝我们，我们知道。

void Person.SitDown(Seat seat) {
    if (seat.SeatPerson(this)) {
        this.Status = Status.Sitting;
    }
    else
    {
        //Couldn't sit down!
    }
}

bool Seat.SeatPerson(Person person) {
    if (this.IsAccepted(person) && this.Seated == null) {
        this.Seated = person;
        return true;
    }
    else
    {
        return false;
    }
}

Answer 4

使用回调，以便每个类都可以维护它负责的状态。

public class Seat
{
  public void SeatPerson(Person person, Action successAction)
  {
    if (IsAccepted(person))
    {
      this.Seated = person;
      successAction();
    }
  }
}


public class Person
{
  public void Sit(Seat seat)
  {
    seat.SeatPerson(this, this.SitComplete);
  }

  public void SitComplete()
  {
    this.Status = Sitting;
  }
}

这里仍然存在周期性依赖。

座位有责任检查尝试就座的人是否有权这样做。
一旦该人就座，座位就会指向该人。
人只知道尝试坐在座位上的方法。

按照惯例， successAction 的保留时间不应长于 SeatPerson 调用的时间。 这保证了 Seat 不会损害 Person 的状态。

Answer 5

您不需要 Seat 类。
Seat 类跟踪坐着的人。
相反，您可以删除 Seat 类并在 Person 类中添加一个名为 isSitting() { return this.Status == Sitting; 的新方法； }

Answer 6

让该人尝试坐在座位上，并根据操作的成功更新其状态：

只需调用 myPerson.TrySeat(targetseat)，如果就座过程成功，则返回 true。

//inside Person class
        public bool TrySeat(Seat seat)
        {
            if (seat.TrySeat(this))
            {
                Status = Sitting;
                return true;
            }
            else
            {
                return false;
            }
        }

//inside Seat class
        internal bool TrySeat(Person person)
        {
            if (CanSeat(person))
            {
                Seated = person;
                return true;
            }
            else
            {
                return false;
            }
        }