Определенный пользователем неизменяемый класс, дающий неожиданные результаты

Question

Это касается неизменяемого класса, который я создал. Это выглядит следующим образом:

final class ImmutableClass 
{ 
    private final Date d; 

    public ImmutableClass(Date d) 
    { 
    this.d=d; 
    } 

    public Date getD() 
    { 
    return this.d; 
    } 

}

При использовании этого выше класс ниже в классе, как:

public class UserClass 
{ 
    public static void main(String args[]) 
    { 
    Date d1 = new Date(); 
    ImmutableClass im= new ImmutableClass(d1); 
    System.out.print("Date is : " + im.getD() + ", object is " + im);//printing originals 

    //trying to change the date 
    d1.setMonths("10"); 
    System.out.print("New Date is : " + im.getD() + ",New object is " + im);//printing latter ones 
    } 
} 

Результат выглядит следующим образом:

Date is : Wed Dec 10 10:38:43 IST 2014, object is com.fedex.hrt.hrmailbox.ImmutableClass@19616c7 

New Date is : Mon Feb 10 10:38:43 IST 2014,New object is com.fedex.hrt.hrmailbox.ImmutableClass@19616c7 

Объект даты в ImmutableClass изменяется и даже объект тот же, новый объект не создается.

Это противоречит определению неизменяемых классов. Просьба указать, ошибаюсь ли я в своем подходе или я пропустил что-то относительно концепции неизменности ??

С уважением!

Answer 1

У вызывающего абонента есть ссылка на Date (и может изменить их ссылку). Вы можете предотвратить это, если у вас есть clone() в конструкторе. Мол,

public ImmutableClass(Date d) 
{ 
    this.d = (Date) d.clone(); 
} 

Кроме того, абонент может изменить Date вы вернетесь в getD, так что вы должны клонировать() там же;

public Date getD() 
{ 
    return this.d != null ? (Date) this.d.clone() : null; 
} 

В качестве альтернативы, для Date, хранить количество миллисекунд он Обертывания как

final class ImmutableClass { 
    private final long d; 

    public ImmutableClass(Date d) { 
     this.d = (d != null) ? d.getTime() : 0; 
    } 

    public Date getD() { 
     return new Date(d); 
    } 
}