principe de responsabilité unique dans l'API

Question

S'il vous plaît jeter un oeil à la suite morceau de code:

public interface ICultureService 
{ 
    List<Culture> GetCultures(); 
    bool IsCultureSupported(Culture culture); 
    Culture GetFallbackCulture(); 
} 

Nous avons constaté que la plupart des consommateurs premier appel IsCultureSupported pour valider si leur culture est prise en charge ou non. Et si la culture ne sont pas pris en charge, ils appellent GetFallbackCulture():

public CallingMethod() 
{ 
    if(!cultureManager.IsCultureSupported(currentCulture)) 
    { 
      currentCulture=cultureManager.GetFallbackCulture(); 
    } 
    . 
    . 
    . 
} 

Comme par simple Responsabilité Principe (et d'autres règles oops), est-il autorisé à introduire une fonction (en ICultureService et sa mise en œuvre) comme:

function GetFallbackCultureIfInvalid(Culture culture) 
{ 
    if(this.IsCultureSupported(culture) 
    { 
      return this.FallbackCulture(); 
    } 
} 

Answer 1

comme par simple responsabilité Principe (et d'autres règles oops), est-il correct d'introduire une fonction (en CultureManager) comme:

Qu'est-ce que vous a se référer à est appelé le Tell-Do't-Ask principe plutôt que le Principe de responsabilité unique. L'ajout de la fonction GetFallbackCultureIfInvalid rend le code client plus lisible. Vous devez également réduire la visibilité du IsCultureSupported afin que cette méthode ne soit plus visible pour le code client.

Cela dit, il semble que CultureManager est une implémentation de CultureService il n'a pas de sens d'ajouter une nouvelle méthode nommée GetFallbackCultureIfInvalid dans CultureManager qui ne fait pas partie de l'interface CultureService. Ce que vous devez faire est de coller à une seule méthode appelée GetFallbackCulture en CultureManager et laisser revenir une chute culture en arrière si la condition requise est remplie:

Culture GetFallbackCulture(Culture culture) { 
    Culture fallBackCulture = culture; 
    if(!this.IsCultureSupported(culture) { 
     fallBackCulture = this.FallbackCulture(); 
    } 

    return fallBackCulture; 
}