java - 实现封装的正确方法

Question

我有以下类方案。

National 有一个 Zone 的 ArrayList，Zone ArrayList Region 和 Region 有一个 ArrayList 的 Person。

所以我有下一个问题:

1) 我可以通过国家和地区“插入”一个人将其添加到地区吗？

例如:

National national = new National();
....
national.addPerson(person); // Every level has his own addPerson method

或

national.getZone(i).getRegion(i).addPerson(person);

在 OOP 中执行此操作的正确方法是什么？

2) 可以制作一个方法让我返回关卡中的每个人吗？

我的意思是例如:

Zone zone = new Zone();
...
zone.getPersons(); //Return a ArrayList with the persons of every Region in Zone.

这不利于封装？

3) 每个级别的 next()、hasNext()、first() 方法，因此我可以在该特定级别进行迭代。

就是这样。我总体上了解封装和 OOP，但我无法弄清楚什么是正确的，什么是错误的。

谢谢。

Answer 1

给定您的类图，Person 必须始终是 Region 的成员，即 National 或 Zone 不能直接包含未包含在其下的 Region 中的 Person 对象。

在这种情况下，匹配签名 addPerson(Person p) 的方法只能出现在 Region 中。正如您所指出的，那么您将不得不做

national.getZone(i).getRegion(j).addPerson(person);

没有合乎逻辑的理由，比如说，

National#addPerson(Person p)

因为 National 没有关于此人属于哪个 Zone 和 Region 的信息。

但是，您可以向 National 和 Zone 添加方便的方法，正如您所说，“推”一个人，但前提是您提供必要的数据.

National#addPerson(ZoneId zid, RegionId rid, person) { ...

对于 ZoneId 和 RegionId，替换您用来在 National 中查找区域和区域的任何键类型。

Zone#AddPerson(RegionId rid, person) { ...

如上RegionId。

这只是一个开始，您必须仔细考虑诸如当 ZoneId 和/或 RegionId 在 addPerson 上找不到时会发生什么的问题(...) 调用。在此设计中，唯一可行的选择是抛出异常，因为此时您没有足够的信息来实例化 Zone 或 Region。

编辑:

如果正如您在评论中指出的那样，Person 对象包含足够的信息来识别 Zone 和 Region，那么您的便捷方法National 和 Zone 可以执行所需的查找并委托(delegate)给链中的下一个 addPerson(Person p)。