實施封裝的正確方法

Question

我有以下Class方案。

National有ArrayList的Zone，Zone一個的Region和RegionArrayList有Person的ArrayList。

所以我要下一個問題：

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不能直接包含不包含在其下一個RegionPerson對象。

在這種情況下，匹配簽名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和你使用任何密鑰類型來查找一個National內的區域和區域RegionId替代品。

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

如上述RegionId。

這只是一個開始，您必須考慮問題，例如在addPerson(...)調用中找不到ZoneId和/或RegionId時會發生什麼情況。在這種設計中，唯一可行的選擇是拋出異常，因爲您沒有足夠的信息來實例化Zone或Region。

編輯：

如果你在一個評論指出，Person對象包含足夠的信息來識別Zone和Region，然後在National和Zone您方便的方法可以做必要的查詢和委託下一個addPerson(Person p)在鏈中。