CA1305: Szczegółowość podczas określania kultury

Question

CA1305 jest wywoływana, gdy istnieje przeciążenie, które wymaga IFormatProvider, ale to przeciążenie nie jest używane w kodzie.

Na przykład, następujący kod podnosi to ostrzeżenie:

string.Format("{0} - {1}", id, name); 

Oczywistym sposobem, aby pozbyć się tego ostrzeżenia jest określenie kultury do stosowania:

string.Format(CultureInfo.InvariantCulture, "{0} - {1}", id, name); 

Myślę jednak, to jest zbyt gadatliwe.

Jakie alternatywy istnieją?

Answer 1

Prawie jedyna alternatywa, która doprowadziłaby do zmniejszenia gadatliwości przy jednoczesnym zachowaniu jawnego przekazywania dostawcy formatu, polegałaby na zastosowaniu metod fasadowych charakterystycznych dla danej kultury. Na szczęście, jeden typowo formatuje tylko InvariantCulture i CurrentCulture, więc tylko dwie metody fasadowe byłyby wymagane dla każdej bazowej metody formatowania.

Typowa metoda fasada dla przykładowego kodu może mieć podpis takiego:

public static string FormatForInvariantCulture(this string template, params object[] substitutions) 

i nazywać następująco:

"{0} - {1}".FormatForInvariantCulture(id, name); 

Innym podejściem do organizowania metod elewacyjnych byłoby w kulturze specyficzne typy formaterów, które można wtryskiwać za pomocą technik IoC. Na przykład, interfejs jak poniżej może być zdefiniowany dla formatów:

public interface IFormatter 
{ 
    string Format(string template, params object[] substitutions); 
} 

Kultura specyficzne przykłady może być następnie wstrzykiwany do typów, które trzeba wykonać formatowane z użyciem konstruktory jak następuje:

public SomeClass(IFormatter systemFormatter, IFormatter uiFormatter) 
{ 
    // ... 
} 

Niezależnie sposobu pakowania metod fasady, należy wziąć pod uwagę, że CA2241 (ProvideCorrectArgumentsToFormattingMethods) nie będzie badać użycia metod, dlatego warto rozważyć dodanie niestandardowej reguły, aby to zrobić.