C# Best practice: utilizzo di un delegato o un'interfaccia come dipendenza di classe

Question

Ho una classe che richiede un modo per recuperare un valore intero casuale con un massimo. Non voglio che questa classe dipenda da un modo specifico per recuperare quel valore casuale (come system.random). Sarebbe meglio:

(A) Utilizzare un pubblico delegato (o func)

public delegate int NextRandomInt(int maxValue); 

public class MyClass 
{ 
    public NextRandomInt NextRandomInt { get; set; } 

    public MyClass(NextRandomInt nextRandomInt) 
    { 
     NextRandomInt = nextRandomInt; 
    } 
} 

(B) Utilizzare un pubblico un'interfaccia

public interface IRandomIntProvider 
{ 
    int NextInt(int maxValue); 
} 

public class MyClass 
{ 
    public IRandomIntProvider RandomIntProvider { get; set; } 

    public MyClass(IRandomIntProvider randomIntProvider) 
    { 
     RandomIntProvider = randomIntProvider; 
    } 
} 

(C) Qualcosa altrimenti tutti insieme.

Entrambe le modalità funzionano. Mi sento come se l'utilizzo di un delegato fosse più semplice e più veloce da implementare, ma l'interfaccia è più leggibile e potrebbe essere più semplice quando l'iniezione della dipendenza arriva.

Answer 1

Secondo il mio punto di vista (ne sono sicuro) la seconda opzione con interfaccia è la migliore. Questa situazione è un tipico esempio del pattern Strategy.

Answer 2

Dipende dalla quantità di lavoro che si desidera implementare utilizzando un delegato o un'interfaccia.

Se l'interfaccia avrà solo uno o due metodi, è possibile utilizzare uno Func per forzare lo stesso comportamento. Altrimenti userei interface s.

Mark Seemann spiegato proprio questo abbastanza bene qui: link

Insomma egli afferma questo:

Quando possibile, preferisco di modellare le mie API con i delegati invece di interfacce di un metodo, dal momento che mi dà maggiore flessibilità e meno codice di infrastruttura.

Ovviamente, questa tecnica funziona solo finché è necessario solo astrarre un singolo metodo. Non appena la tua astrazione necessita di un secondo metodo, dovrai introdurre un'interfaccia corretta o, preferibilmente, una classe di base astratta.

Answer 3

1. Usa interfaccia se si pensa che ci possa essere un'altra implementazione di NextRandomInt() può esistere nel sistema.
2. Scegliere il delegato se si ritiene che debba essere eseguito come causa di un evento o richiamata.
   Altrimenti, basta andare con un metodo come membro della classe. Spetta a te se desideri che il metodo sia disponibile come metodo instance o static.