Attualmente sto lavorando con una base di codice "interessante" e vedo il seguente tipo di cosa nel codice.
public Object doSomething()
{
Object obj = new Object();
// Do some stuff to the object
obj = doSomthingElse(obj);
return obj;
}
Quando guardo questo tipo di codice, sento sempre che non è corretto passare un oggetto in un metodo come parametro, ma anche impostare l'oggetto restituito dal metodo sulla stessa variabile che faceva riferimento all'oggetto passato. Di solito faccio qualcosa di simile al seguente, dove creo una nuova variabile per evitare qualsiasi confusione.
public Object doSomething()
{
Object obj = new Object();
// Do some stuff to the object
Object anotherObj = doSomthingElse(obj);
return anotherObj;
}
Ho sbagliato a pensare che il secondo snippet di codice sia più leggibile / corretto?