Buenas Prácticas para Volviendo referencias a objetos

Considérese este fragmento de código:Buenas Prácticas para Volviendo referencias a objetos

class MyClass{ 
    private List myList; 
    //... 
    public List getList(){ 
     return myList; 
    } 
}

como Java pasa referencias de objetos por valor, mi opinión es que cualquier objeto llamando getList() obtendrá una referencia a myList, lo que le permite modificar myList a pesar siendo private. ¿Es eso correcto?

Y, si es correcto, debería estar utilizando

return new LinkedList(myList);

para crear una copia y pasar de nuevo una referencia a la copia, en lugar del original, con el fin de evitar el acceso no autorizado a la lista referenciada por myList?

Fuente

2010-07-11 chrisbunney

técnicamente, ningún cliente externo podrá cambiar el valor del campo myList en su clase. Siempre será un control/referencia a List, haciendo referencia a la lista a la que algún código dentro de MyClass lo configuró.El cliente externo podrá modificar el contenido de esa lista –

Sí, eso es lo que quise decir, pero se volvió un poco descuidado con mi explicación. Lo editaré – chrisbunney

+1 Miro A para una aclaración correcta. Y sí, su solución al crear una copia de la lista es correcta (por lo que quiere lograr). – xagyg

Hago eso. Mejor aún, algunas veces devuelvo una copia no modificable usando la API de Colecciones.

Si no lo hace, su referencia no es privada. Cualquiera que tenga una referencia puede alterar su estado privado. Lo mismo ocurre con cualquier referencia mutable (por ejemplo, Fecha).

Fuente

2010-07-11 21:18:35 duffymo

Si proporciona una vista no modificable de la colección original, tenga en cuenta que cualquier cambio en la lista original se reflejará en la vista. La vida sería mucho más simple con colecciones inmutables ... –

Depende de lo que desee.

¿Deseas exponer la lista y hacerlo para que la gente pueda editarla?

¿O desea dejar que la gente lo mire, pero no lo modifique?

No hay una manera correcta o incorrecta en este caso. Solo depende de tus necesidades de diseño.

Fuente

2010-07-11 21:18:59 jjnguy

Por lo tanto, si su diseño dicta que un atributo tiene visibilidad privada, debe devolver una copia del objeto en lugar de una referencia simple para asegurarse de que permanece encapsulado – chrisbunney

@chris Si diseña requiere que las personas no puedan modificar su lista, luego pasen una copia. De lo contrario, pase la referencia original. – jjnguy

Sí, lo siento, fue un comentario retórico, entendí su respuesta;) – chrisbunney

Puede haber algunos casos en los que uno quiera devolver la lista "sin formato" a la persona que llama. Pero en general, creo que es una mala práctica, ya que rompe la encapsulación y, por lo tanto, está en contra de OO. Si debe devolver la lista "en bruto" y no una copia, debe ser explícitamente claro para los usuarios de MyClass.

Fuente

2010-07-11 21:34:07 Cratylus

Sí, y tiene un nombre .. "Copia defensiva". También se recomienda copiar en el extremo receptor. Como Tom has noted, el comportamiento del programa es mucho más fácil de predecir si la colección es inmutable. Entonces, a menos que tenga una muy buena razón, debe usar una colección inmutable.

Cuando Google Guava se convierte en parte de la biblioteca estándar de Java (estoy totalmente de pensar que debería), esto probablemente se convertirá en el idioma preferido:

return ImmutableList.copyOf(someList);

void (List someList){ 
    someList = ImmutableList.copyOf(someList);

Esto tiene la ventaja adicional de rendimiento, porque el método copyOf() comprueba si la colección ya es una instancia de colección inmutable (instanceof ImmutableList) y, si es así, omite la copia.

Fuente

2010-07-11 23:37:18

Creo que el patrón de hacer que los campos sean privados y proporcionar acceso es simplemente para el encapsulamiento de datos. Si quieres que algo sea realmente privado, ¡no le des métodos de acceso! A continuación, puede escribir otros métodos que devuelvan versiones inmutables de sus datos privados o copias de los mismos.

Fuente

2010-07-12 01:09:26 Jesse

Buenas Prácticas para Volviendo referencias a objetos

Respuesta

Cuestiones relacionadas