Chiamare un metodo e passare un riferimento a un oggetto e rilevare il ritorno con lo stesso riferimento

Question

Ho una domanda per principianti. Ho cercato molto per la risposta ma non riesco a trovare la risposta esatta, quindi forse qualcuno con sviluppatori più esperti può aiutarmi con questo.

Quindi, diciamo che avete una seguente situazione nel codice (questo è semplificato versione della situazione):

SomeObject a1 = new SomeObject(); 
a1 = someMethod(a1); 

public SomeObject someMethod(SomeObject a1) { 
a1.changeVariable(); 
return a1; 
} 

ora ho sentito da diverse persone dire che il superamento di un riferimento a un oggetto a un metodo e catturare il valore restituito è una cattiva pratica di codifica. Sfortunatamente nessuno può spiegarmi esattamente perché si tratta di una cattiva pratica di programmazione, e la mia ricerca della ragione non ha portato a nulla. Alcuni possono spiegare perché è una cattiva pratica fare questo? Non riesco a pensare a una situazione in cui qualcosa per questo possa andare storto.

Grazie in anticipo, voi bella gente mi hanno già aiutato con le risposte alle altre persone innumerevoli volte, si tratta di una prima volta ho davvero bisogno di inviare una domanda :)

Answer 1

Nello scenario avete dimostrato, ci non è un beneficio molto evidente. Tuttavia, come modello, può essere utile nella mia esperienza, in particolare se il metodo esegue la convalida.

Il metodo di Guava Preconditions.checkNotNull è un ottimo esempio di questo. Significa che posso scrivere codice come questo:

public class Player { 
    private final String name; 

    public Player(String name) { 
     this.name = Preconditions.checkNotNull(name); 
    } 
} 

... invece di dover dividere l'assegnazione e la convalida. Inoltre, consente la convalida di esistere come parte di un'altra chiamata, ad es. ad un altro costruttore:

super(Preconditions.checkNotNull(name));