Problemas de Findbugs con la mutabilidad del objeto Date en Java

Question

Esto es más de una continuación de las preguntas 1 & 2.

como se cuenta en las preguntas que el código de abajo

public Date getSomeDate() { 
    return someDate; 
} 

le dará el error findbug issue.

La solución propuesta fue la de duplicar el objeto Date en ambos captadores y definidores como

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

Es esta una buena aproximación o ¿existen formas alternativas a esto?

¿Hay alguna biblioteca de fecha inmutable disponible en java que pueda solucionar este problema?

Answer 1

JodaTime tiene fechas inmutables.

Claro, está bien usar un constructor Date en un getter, ¿por qué no sería?

Dicho esto, el hecho de que FindBugs establezca el estado mutable como un error potencial, no significa que sea intrínsecamente valioso preocuparse por –, depende de cómo se use la clase. La inmutabilidad elimina un tipo de error, que puede o no necesitar preocuparse mucho.

Answer 2

Dependiendo de su caso de uso, puede devolver someDate.getTime() sin envolverlo en un Date.

Answer 3

Espera un momento ... copiando el objeto dentro de los métodos getSomeDate y setSomeDate no estamos eliminando el riesgo de seguridad ya que el objeto modificado regresa a través de la setSomeDate y copias a preservar los valores modificados. Sería necesario eliminar setSomeDate para resolver este tipo de problema de seguridad, o no se preocupe en absoluto.

Answer 4

Atención gente ...

además de adaptar tanto el comprador y el colocador es necesario tener cuidado acerca de los valores nulos:

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()); 
    } 
}