Java Оборонительное копирование и непреодолимое

Question

Почему мы должны делать Оборонительное копирование, чтобы достичь Непрерывного класса? Посмотрите на этом коде:

public final class EmailMessage { 
    private final String from; 
    private final String to; 
    private final String message; 
    private final Date date; 

    public EmailMessage(String from, String to, String msg, Date date) 
    { 
     this.to = to; 
     this.from = from; 
     this.message = msg; 
     this.date = new Date(date.getTime());// instead of date; 
    } 

    public String getFrom() 
    { 
     return(from); 
    } 

    public Date getDate() { 
     return(new Date(date.getTime()); // instead of Date 
    } 
} 

Почему это не будет Неизменным, если мы не делали оборонительное копирование?

Answer 1

Для достижения неизменности вы должны сделать копии всех изменяемых объектов, переданных в ваш конструктор, а также вернуть копии для всех изменяемых объектов, хранящихся внутри вашего класса, если таковые имеются.

• Если вы не сделали копию даты, переданной вам, вызывающий может изменить дату после того, как объект был создан, эффективно его мутируя.
• Если вы не вернете копию из получателя изменчивого объекта, вызывающие могут мутировать объект, который они получают от вас, эффективно изменяя ваш объект.

В вашем конкретном примере класс Date изменен. Если йо пропустить копирование в конструкторе, вредоносный код может это сделать:

Date d = new ... 
EmailMessage msg = new EmailMessage("lazy dog", "quick brown fox", "Jump!", d); 
d.setTime(d.getTime()+12345); // Changes the date inside msg 

Если вы пропустите вторую копию, абоненты могут сделать это:

EmailMessage msg = ... 
Date d = msg.getDate(); 
d.setTime(d.getTime()+12345); // Changes the date inside msg 

Answer 2

Потому что в противном случае можно изменить объект государство. Давайте представим, что ваш метод getDate является следующее:

public Date getDate() { 
    return date; // instead of Date 
} 

И мы используем это следующим образом:

EmailMessage msg = new EmailMessage(...); // initialization 
Date date = msg.getDate(); 
date.setTime(...); //ooops, our msg object has another date now