PersonオブジェクトクラスのプロパティとしてTreeMapを使う

Question

次のようにコードを書くのは間違っていますか？私はそれがある電子メールのタイプ（仕事、個人、など）を含む私の人のクラスに電子メールを格納できるようにしたい。私はこれにTreeMapを使うことにしました。すべての変数がプライベートでgetterとsetterを使って操作するのがよい習慣であることは知っていますが、私のPersonクラスではなく、TreeSetメソッドを直接使用してTreeSetを操作するのは間違っていますか？言い換えれば、これはこれを行うための有効な許容方法ですか？コードは正常に動作するようです。

public class Person { 
    private String firstName; 
    private String lastName; 
    private String note; 
    TreeMap<String, String> phoneNum = new TreeMap<String, String>(); 

    // Assume constructor method contains firstName & lastName and there are 
    // getters and setters for both 
} 

public class MainDriver { 
    public static void main(String[] args) { 
    Person p1 = new Person("John", "Smith"); 

    p1.phoneNum.put("jsmith@gmail.com", "School"); 
    p1.phoneNum.put("jsmith19@gmail.com", "Personal"); 

    Person p2 = new Person("Sam", "Johnson"); 

    p2.phoneNum.put("samjohn@gmail.com", "Personal"); 
    p2.phoneNum.put("samjohnson", "Work"); 

    System.out.println(p1.phoneNum); 
    System.out.println(p2.phoneNum); 
    } 
} 

Output: 
{jbillingham@gmail.com=Personal, jebillingham3@gmail.com=School} 
{samsamyoussef=Work, samyou@gmail.com=Personal} 

Answer 1

それはひどいはないが、それは機能の羨望（オブジェクトが、直接、他のオブジェクトのフィールドを使用することにより、コードのにおい）を可能にします。

問題はPersonに電子メールアドレスを追加するだけで、実際にはTreeMapのすべての操作が公開されていることです。 ceilingKey,tailMapおよびremoveのような方法。実行可能な操作を制限するには、フィールドを完全にカプセル化し、明示的なメソッドを提供する必要があります。

Answer 2

通常はArrayListが使用されていますが、TreeMapを使用している理由は、各電子メールまたは電話番号（キー）に1つのContactType（enumを使用する方がよい）が必要なのかと推測しています。

たとえば、Personal、Schoolの両方でjbillingham@gmail.comを使用することはできません。

さらに、phoneNumをプライベートにして、yr PersonクラスにaddContact/ removeContactメソッドを追加することをお勧めします。