Devo usare un codice facilmente leggibile come questo
if (var.isServicePGM() || var.isStandardPGM())
{
//Much code
if (var.isServicePGM())
{
//Some code
}
else if (var.isStandardPGM())
{
//Some code
}
}
o dovrei usare un codice più veloce come questo
if (var.isServicePGM() || var.isStandardPGM())
{
//Much code
if (var.isServicePGM())
{
//Some code
}
else
{
//Some code
}
}
L'ottimizzazione prematura è cattiva.
Se non sei assolutamente sicuro che questo metodo stia causando un collo di bottiglia e questo sta già causando problemi, devi sempre concentrarti sulla leggibilità (e sulla scrittura di codice senza errori). Il codice altamente ottimizzato tende ad essere più difficile da estendere ed eseguire il debug in un futuro.
Quindi la risposta è - sì, dovresti usare più funzioni per aumentare la leggibilità.
Dai un'occhiata a questa domanda e alle risposte StackOverflow: link
Non sei sicuro di come il secondo codice possa essere più veloce, solo in alcuni casi e solo se isStandardPGM () esegue alcuni calcoli pesanti. Probabilmente dovresti assegnare i risultati a una variabile e usarli.
bool isService = var.isServicePGM();
bool isStandard = var.isStandardPGM();
if (isService || isStandard)
{
//Much code
if (isService)
{
//Some code
}
else if (isStandard)
{
//Some code
}
}
Non preoccuparti dell'efficienza finché non sai che si tratta di un problema e puoi dire solo che quando il codice è in esecuzione, non prima.
Preoccupati della manutenibilità, che è molto diversa dalla leggibilità. La manutenibilità consiste nel minimizzare l'opportunità per le persone di commettere errori quando apportano modifiche al codice (per il quale DRY è un buon principio). La leggibilità pone la domanda: chi sta leggendo? Gli scimpanzé sono intelligenti, ma non li renderete mai leggibili. E puoi fare in modo che il codice appaia leggibile semplicemente spostandone la maggior parte dallo schermo con spazi vuoti.
Leggi altre domande sui tag design readability performance