C'è qualche ragione per evitare i getter privati?

Vedo spesso questo codice e non mi piace.

I miei colleghi lo usano spesso per iniettare alcune dipendenze in un'istanza. Ma così facendo, aprono la proprietà per cambiarne un po 'quando più tardi, mentre la proprietà dovrebbe essere impostata solo durante la creazione dell'oggetto.

E questo è il motivo per cui non mi piace: non puoi essere sicuro che il setter pubblico venga chiamato più tardi e cambi lo stato dell'istanza che pensavi fosse stabile.

Cioè, quando è inteso per l'iniezione di dipendenza, usa la modalità di un'iniezione di dipendenze appropriata alla creazione dell'oggetto. Se è pensato per cambiare lo stato dell'istanza, meglio usare un metodo ("comando") per dedurre lo scopo.

Il motivo per cui non l'hai mai visto prima non è perché un tale codice causerebbe problemi immediati. Il motivo è probabilmente: questa combinazione non è particolarmente utile .

Il codice che rende pubblici setter consente di chiamare il codice per modificare lo stato dell'oggetto, quindi il codice chiamante "conosce" già il contenuto di ciò che è stato trasmesso attraverso il setter e semplicemente non ha senso "nascondere" questo contenuto dal fuori rendendo privato il getter. Quindi non c'è molto vantaggio da questo, al contrario, testare lo stato dell'oggetto o riutilizzare l'oggetto diventa tipicamente più difficile.

Confronta questo, per esempio, con la combinazione di un setter privato con un getter pubblico. Questo è spesso molto utile per limitare o prevenire effetti collaterali indesiderati ed è necessario quando si progettano oggetti immutabili.