Dichiarazione delle variabili di ritorno nei metodi c # rispetto alla restituzione diretta del valore

In un dibattito sulle variabili di ritorno, alcuni membri del team preferiscono un metodo per restituire il risultato direttamente al chiamante, mentre altri preferiscono dichiarare una variabile di ritorno che viene quindi restituita al chiamante (vedere esempi di codice di seguito)

L'argomento di quest'ultimo è che consente a uno sviluppatore che sta eseguendo il debug del codice di trovare il valore di ritorno del metodo prima che ritorni al chiamante, facilitando così la comprensione del codice: ciò è particolarmente vero quando le chiamate al metodo sono concatenate.

Ci sono delle linee guida su quale sia il più efficiente e / o ci sono altri motivi per cui dovremmo adottare uno stile piuttosto che un altro?

Grazie

    private bool Is2(int a)
    {
        return a == 2;
    }

    private bool Is3(int a)
    {
        var result = a == 3;
        return result;
    }

Poiché utilizzo Resharper con Visual Studio, Ctrl-RV (o Ctrl-Alt-V, se si utilizzano le associazioni di tasti Resharper / IntelliJ) trasforma il primo esempio nel secondo esempio. Quindi, quando voglio eseguire il debug, posso farlo abbastanza facilmente. E se dimentico di rimetterlo, non mi sentirò male perché Ctrl-RI lo riporterà di nuovo per rendere più facile la lettura.

Seriamente, perdi tempo a discutere di cose più importanti. Ad esempio dove posizionare le parentesi graffe o gli spazi principali rispetto alle schede.

Personalmente trovo il primo esempio più facile da leggere. È ancora possibile eseguire il debug impostando un punto di interruzione nell'istruzione return e aggiungendolo a == 2alla finestra di controllo o utilizzando la visualizzazione rapida.

Ma questa è davvero una questione di preferenze personali. Entrambe le versioni sono OK.

Quando il codice è facilmente leggibile come il tuo esempio, non c'è nulla di sbagliato nel restituire il risultato di un'operazione logica come return a == 2. Tuttavia, se il valore restituito è un'istruzione più complessa o simile

return a > 2? doOptionA().getResult() > makeDecision("greaterThan2") : doOptionB().getResult() == makeDecision("lessThan2");

quindi ti consigliamo di utilizzare le variabili per memorizzare prima le parti e semplificare l'istruzione return, per motivi di leggibilità.

In un semplice esempio come quello, uno dei due è OK.

Per esempi più complicati, preferisco il secondo modo. Questo è solo perché è più leggibile e altri probabilmente dovranno mantenere il codice.