Quando utilizzare le istruzioni a riga singola se? [chiuso]

L'unica volta che si usa una dichiarazione a riga singola è quando ce ne sono molte e si può formattare il codice per chiarire cosa sta succedendo. Qualsiasi altra cosa non è chiara e porterà ad errori.

ad es.

if (x) do_stuff();

è terribile Quando è combinato con altro codice, non mostra chiaramente l'istruzione condizionale, e alcune persone confondono la seguente riga come un condizionale con un rientro difficile. Questo vale per tutte le affermazioni, specialmente se sono die (), return o throw. Non ha nulla a che fare con l'esecuzione del codice, è tutto sulla leggibilità.

if (a) do_a();
if (b) do_b();
if (c) do_c();

questo è molto meglio che mettere i condizionali sulla riga successiva poiché è più chiaro ciò che è inteso, non c'è spazio per la confusione.

Preferisco sempre la versione con parentesi graffe.

1. Se non utilizzo sempre le parentesi graffe, allora potrei dimenticarti di metterle quando c'è più di una dichiarazione nel blocco.
2. Gli IDE utilizzati da altri membri del team possono riformattare automaticamente il codice per spostare l'istruzione nella nuova riga, rendendo in tal modo molto più soggetta a errori in futuro.
3. L'unione del codice tra i rami è più soggetta a errori se l'istruzione if è in conflitto. La probabilità di errore è ancora maggiore se la persona che esegue l'unione non conosce completamente le modifiche unite.
4. Una dichiarazione che non appartiene al blocco può essere inclusa in seguito per errore (aggiungo nuove istruzioni al blocco, quindi ho bisogno di aggiungere delle parentesi, ma a causa di indentazione o effetti visivi simili includo anche una dichiarazione che dovrebbe essere lasciato fuori dal blocco).

Proporrei una risposta alternativa. Preferisco i single liner quando la condizione interna è in realtà una singola riga ed è relativamente isolata dal resto delle condizioni.

Un grande esempio è:

public void DoSomething(int something) {
     // Notice how easily we can state in one line that we should exit the method if our int is 0.
     if(something == 0) return;

     CallAnotherMethodHere(something);
}

Un cattivo esempio (secondo me) sarebbe quello che @gbjbaanb ha menzionato:

if (a) do_a();
if (b) do_b();
if (c) do_c();

Per me, questa è un'istruzione switch e dovrebbe essere scritta come tale:

switch(something) {
    case "a":
        do_something_to_a();
        break;
    case "b":
        do_something_to_b();
        break;
    case "c":
        do_something_to_c();
        break;
    default:
        break;
}

IMHO molto più leggibile e più efficiente in quanto la variabile deve essere valutata solo una volta che VS esegue tutte le istruzioni if sebbene i guadagni siano trascurabili.