È una funzione interna giustificata in questa situazione

La tua implementazione mi sembra ragionevole.

Tuttavia, se la maggior parte delle altre persone che sviluppano il software ha un'esperienza di programmazione avanzata limitata, è possibile che si desideri utilizzare una funzione regolare anziché anonima, poiché sarà più ovvio che cosa sta succedendo.

Quando sei lo sviluppatore senior e il resto della tua squadra ha un livello di abilità significativamente più basso; assicurandosi che capiscano come funziona il tuo codice e può facilmente modificarlo se necessario è una considerazione importante. Poiché la tua squadra è composta principalmente da non programmatori, questa è una considerazione più ampia, poiché per le tecniche di programmazione probabilmente si avvicinano alla Usa quello che so della scala rispetto a Impara cose nuove ogni giorno fine.

Rompere la logica ripetuta e rimuovere la copia / incolla dovrebbe essere semplice per spiegare i vantaggi di. Tecniche più avanzate come le funzioni anonime o linq sarebbero una vendita più dura dal momento che non offrono miglioramenti importanti e ovvi di funzionalità / manutenzione. Gli ingegneri hardware con cui ho lavorato in passato in genere hanno imparato solo C all'università dal momento che era la lingua di scelta per i microcontrollori nei gadget che stavano costruendo. Con solo una manciata di classi, le loro abilità erano concentrate solo sulla programmazione di base pratica e non su nulla di avanzato.

Questa non è una funzione di ordine superiore, quindi dovrebbe essere facilmente comprensibile a qualsiasi programmatore C #.

Sebbene invece di utilizzare un tipo di delegato personalizzato, potresti aver utilizzato Action<string, string, string, CuttingParameters> .

Ma penso che un metodo normale sarebbe andato bene anche qui, ed è probabilmente più idiomatico. Anche se non lo userai da nessun'altra parte.

La domanda chiave è se utilizzare una funzione "normale" private o una funzione strettamente locale e la risposta dovrebbe derivare dallo stesso processo decisionale da utilizzare con un campo private rispetto a una variabile locale: Questa cosa è utile ovunque ma questa funzione? In questo caso, non sembrerebbe, quindi il tuo approccio sembra corretto.

Lì è un problema con l'utilizzo di tecniche funzionali avanzate in C #, ma C # si sta evolvendo per avere caratteristiche sempre più funzionali, quindi non dovresti evitare tutte tecniche funzionali < em> solo perché potrebbero non essere ancora conosciuti. Direi che questo sarebbe in realtà il tipo di utilizzo che sarebbe buono in una base di codice in cui le persone erano relativamente poco familiari con la programmazione funzionale, dal momento che è molto semplice sia in termini di utilità che di implementazione.