Pratica per restituire se stesso invece invalido in Java

Question

I miei compagni di squadra mi introducono a un nuovo metodo di scrittura pratica che non restituirà nulla.

public class Test { 
    public Test doCalculations() { 
     //code 
     return this; 
    } 
    public Test appendTitle(String test) { 
     //code 
     return this; 
    } 
} 

Invece di restituire il vuoto, suggeriscono di restituire l'oggetto stesso. Uno dei vantaggi di questo approccio che dicono, è possibile concatenare metodi.

Invece di scrivere:

while(1) { 
    test.appendTitle("aaa"); 
    test.doCalculations(); 
    map.add(test); 
} 

è possibile scrivere codice più elegante:

while(1) { 
    map.add(test.appendTitle("aaa").doCalculations()); 
} 

Quali potrebbero essere gli svantaggi di questo aproach? Suggerite di includerlo nell'uso quotidiano?

Answer 1

Direi che non è una buona pratica. Osservando la firma del metodo che restituisce un oggetto, come posso sapere se si tratta di un'istanza completamente nuova che viene restituita o di quello esistente che viene restituito. Si tenga presente che in caso di metodi immutabili modificati le classi restituiscono una nuova istanza classe.