Java: accesso ai campi privati ​​direttamente da un'altra istanza della stessa classe

Question

Sto scrivendo una funzione equals(Object obj) per una classe. Vedo che è possibile accedere ai campi privati ​​di obj dal chiamante. Così, invece di utilizzare un getter:

Odp other = (Odp) obj; 
if (! other.getCollection().contains(ftw)) { 

} 

posso solo accedere al campo direttamente:

Odp other = (Odp) obj; 
if (! other.collection.contains(ftw)) { 

} 

è questa cattiva pratica?

Answer 1

No, non lo è. Il motivo per cui variabili private e metodi non sono accessibili da altre classi è quello di consentire di modificare gli interni della classe senza dover modificare tutto il codice che utilizza la classe (per impedire all'utente della classe, ad esempio, di impostare una variabile ad un valore che non dovrebbe mai avere).

Se si utilizzano variabili private di altri oggetti che non danneggiano nulla, perché se si ristrutturano gli interni della classe, si dovrà comunque modificare il codice all'interno della classe.

Answer 2

Tendo ad usare sempre getter, perché a volte un getter non è solo "return (foo)". A volte inizializzano le cose se sono nulle, o hanno un po 'di debug che vi loggano, o convalidano lo stato corrente in qualche modo. È più coerente.

Answer 3

Questo va bene e completamente normale. È un po 'strano pensare che this possa giocare con i campi privati ​​di other, ma va bene perché non c'è niente di male che possa accadere fino a quando alcuni terzi sono in grado di distruggere gli interni di un oggetto Odp. Qualsiasi metodo della classe Odp può modificare qualsiasi membro privato di qualsiasi oggetto Odp, anche non this, ma ciò va bene dal momento che qualsiasi metodo di questo tipo può ovviamente essere considerato attendibile!

Answer 4

Non penso che questa sia una cattiva pratica, ma una caratteristica della lingua. Non solo consente di testare equals come si fa, ma è anche utile in un modello di Prototipo per la creazione di oggetti.

Answer 5

L'utilizzo di un membro privato per la classe di entità può causare il funzionamento errato della classe proxy. Immaginate di dover ibernare la classe creando una query lazy. Se si controlla la variabile membro, restituisce null. Ma se chiamate get(), recupererà i dati dal database e inizializzerà il campo.

Answer 6

Si tratta di una duplicazione di

Why can I access my private variables of the "other" object directly, in my equals(Object o) method

dati privata è accessibile da qualsiasi istanza di quella classe, anche se un'istanza di classe A è l'accesso ai membri privati ​​di un altro esempio di A. E 'importante ricorda che i modificatori di accesso (privati, protetti, pubblici) controllano l'accesso alla classe, non l'accesso all'istanza