Un metodo dovrebbe fare una cosa e essere bravo in questo? [duplicare]

Hai estratto troppo quando il tuo codice stesso è più chiaro del nome del metodo .

Ricorda che, quando non sono sicuro, quasi tutti i programmatori che ho visto hanno creato metodi molto più grandi e monolitici che ottimali, piuttosto che il contrario.

Un metodo con poche righe di codice è abbastanza tipico di app / libreria ben progettata.

Per citare Pulisci codice , in la pagina 36, dice:

So, another way to know that a function is doing more than "one thing" is if you can extract another function from it with a name that is not merely a restatement of its implementation.

L'idea generale è che hai finito di estrarre quando non c'è nient'altro che puoi estrarre e ottenere ancora un grumo logico di codice. Se c'è una parte di una funzione che può essere eliminata, se fa qualcosa che puoi descrivere e dare un nome, e se puoi immaginare che potresti doverla chiamare da qualche altra parte del tuo codice, allora dovresti estrarre esso.

Anche se non hai mai bisogno di chiamarlo da un'altra parte del tuo codice, il fatto che si tratti di una funzione separata significa che puoi scrivere test unitari solo per questo, il che migliora il rilevamento dei bug e significa che puoi profilarlo indipendentemente, il che migliora la capacità di valutare le perdite di prestazioni nel codice.

Estrai il codice fino a quando non c'è alcuna funzione che fa più di una cosa: se puoi estrarre il codice da una funzione in un'altra funzione, se la nuova funzione esegue qualche compito e se puoi descrivere quale è quella attività , quindi estrai il codice. Se sei preoccupato per le prestazioni, puoi sempre aggiungerlo in un secondo momento.