Quale di questi stili preferiscono le persone?
string mystring = "defaultvalue";
if (condition)
mystring = "othervalue";
VS
string mystring = null;
if (condtion)
mystring = "othervalue";
else
mystring = "defaultvalue";
VS
mystring = condition? "othervalue" : "defaultvalue";
Mentre le persone avranno preferenze, non importa se confrontate con uno stile particolare seguito in modo coerente, sempre .
Direi, entrambi hanno bisogno di {} di parentesi graffe per evitare che il codice venga aggiunto in modo errato:
string mystring = "defaultvalue";
if (condition)
mystring = "othervalue";
string anotherVariable = "a value that someone could think is part of the conditional statement";
Alcune persone potrebbero vederlo come eccessivo, ma a me sembra preferibile fare un semplice passaggio che prevenga i facili difetti.
Preferirei il primo stile solo perché è più breve e comunque leggibile. A seconda della lingua, penso che la scelta 3 sia ancora migliore:
string mystring = condition ? "othervalue" : "defaultvalue"
o
string mystring = if (condition) "othervalue"
else "defaultvalue"
perché ti permette di trasformare mystring in una variabile immutabile, che ha alcuni vantaggi.
Il primo stile è di 3 righe rispetto a 5 linee e non sembra meno chiaro o leggibile; è una preferenza facile dal mio punto di vista.
Come molti altri, personalmente preferisco anche la prima versione. Una ragione non menzionata finora: rende chiaro quale sia il valore predefinito (e il percorso di esecuzione).
Ma nel complesso, la differenza non è drammatica, posso vivere anche con la seconda versione (e lo faccio, poiché ce ne sono molti nella nostra app precedente).
mystring = condition? "othervalue" : "defaultvalue";
più intuitivo per me:)
Preferisco il primo stile quando esiste un valore predefinito conosciuto che si verificherà a meno che qualcos'altro sia vero. Se non ci sono impostazioni predefinite, allora userò il secondo stile.
La prima forma è superiore, in quanto offre una maggiore forma di sicurezza delle eccezioni. Se condition o qualche altro codice prima del lancio dell'assegnazione, la stringa non avrà il valore che si presume che lo farà.
Non mi piace molto mystring = condition? "othervalue" : "defaultvalue"; ma è solo una preferenza personale e forse non sono abituato a vederlo (la mia lingua è Delphi).
Gli altri sono un po 'sballottati e spesso lo lascio dipendere dalla distanza (numero di linee) tra la dichiarazione della variabile e la condizione. Breve: l'inizio con il valore predefinito. Più righe ci sono tra la dichiarazione e la condizione, più mi proverò ad avere un valore nullo o vuoto nella dichiarazione e un'istruzione if-else completa per l'assegnazione del valore.
Leggi altre domande sui tag coding-style