Problemi di findbug con la mutabilità dell'oggetto Date in Java

Question

Questo è più di un seguito alle domande 1 & 2.

come detto nelle questioni il codice qui sotto

public Date getSomeDate() { 
    return someDate; 
} 

vi darà l'errore findbug issue.

La soluzione suggerita è stato quello di duplicare l'oggetto Date in entrambi i getter e setter come

public Date getSomeDate() { 
    return new Date(someDate.getTime()); 
} 

'questo un buon approccio o ci sono modi alternativi per questo?

Esiste una libreria di Date immutable disponibile in Java che possa risolvere questo problema?

Answer 1

JodaTime ha date immutabili.

Certo, va bene usare un costruttore Date in un getter, perché non dovrebbe essere?

Detto questo, solo perché lo stato mutabile di FindBugs peg come un potenziale errore, non significa che sia intrinsecamente degno di attenzione su – dipende da come viene utilizzata la classe. L'immutabilità elimina un tipo di bug, di cui potresti o non dovresti preoccuparti molto.

Answer 2

A seconda del caso di utilizzo, è possibile restituire someDate.getTime() senza avvolgerlo in un Date.

Answer 3

Aspetta un minuto ... copiando l'oggetto entro i getSomeDate e setSomeDate metodi non stiamo eliminando il rischio per la sicurezza in quanto l'oggetto modificato ritorna attraverso il setSomeDate e copie preservare i valori modificati. Sarebbe necessario rimuovere setSomeDate per risolvere questo tipo di problema di sicurezza, o non preoccuparti affatto.

Answer 4

Attenzione Folks ...

oltre adattare sia il getter e setter è necessario prendere a cuore i valori nulli:

public Date getSomeDate() { 
    if (this.someDate == null) { 
    return null; 
    } 
    return new Date(this.someDate.getTime()); 
} 

public void setSomeDate(final Date someDate) { 
    if (someDate == null) { 
    this.someDate = null; 
    } else{ 
    this.someDate = new Date(someDate.getTime()); 
    } 
}