Verwenden einer TreeMap als Eigenschaft in einer Person-Objektklasse

Question

Ist es falsch, meinen Code wie folgt zu schreiben? Ich möchte in der Lage sein, eine E-Mail in meiner Personenklasse zu speichern, die auch den Typ der E-Mail enthält (Arbeit, Persönlich usw.). Ich entschied mich dafür, eine TreeMap zu verwenden. Ich weiß, dass es eine gute Übung für alle Variablen ist, privat zu sein und Getter und Setter zu benutzen, um sie zu manipulieren, aber ist es falsch, mein TreeSet zu manipulieren, indem ich direkt die TreeSet-Methoden anstelle meiner eigenen in meiner Person-Klasse verwende? Mit anderen Worten, ist dies ein zulässiger Weg dies zu tun? Der Code scheint gut zu funktionieren.

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

Es ist nicht schrecklich, aber es ermöglicht Merkmal Neid (den Code Geruch wodurch Objekte die Felder anderer Objekte direkt verwendet werden).

Das Problem ist, dass Sie möchten nur das Hinzufügen von E-Mail-Adressen zu der Person aktivieren, aber Sie tatsächlich alle Operationen der TreeMap offen legen. Methoden wie ceilingKey, tailMap und remove. Um die Operationen einzuschränken, die ausgeführt werden können, sollten Sie das Feld vollständig einkapseln und explizite Methoden bereitstellen.

Answer 2

Normalerweise ArrayList wird verwendet, aber ich denke, der Grund, warum Sie TreeMap verwenden, ist, möchten Sie jede E-Mail oder Telefonnummer (Schlüssel) einen ContactType (besser Enum verwenden).

Zum Beispiel können Sie nicht jbillingham@gmail.com für beide Personal, Schule.

Und auch, es ist besser, Sie machen phoneNum als privat und fügen addContact/ removeContact Methoden zu Person Klasse.