String.Format metodę rozszerzenia

Question

mam:

public static string Format(this string text, params object[] args) 
{ 
    return string.Format(text, args); 
} 

Więc mogę zrobić:

"blablabla {0}".Format(variable1); 

Czy to dobry/zły? Czy może stać się jeszcze krótszy? Chcę mieć ciągi budować bezproblemowo, jak pisanie tekstu bez obawy przed lub po parametrów i rzeczy:

// bad 
return "date: " + DateTime.Now.ToString("dd.MM.yyyy") + "\ntime: " + DateTime.Now.ToString("mm:HH:ss") + "\nuser: " + _user + " (" + _status + ")"; 

// better, but you have to deal with order of {0}...{n} and order of parameters 
return string.Format("date: {0}\ntime: {1}\user: {2} ({3})", ...); 

// ideal 
return "date: {DateTime.Now{dd:MM:yyyy}}\ntime: {...}\nuser: {_user} ({_status})"; 

Answer 1

Cóż, jeden złe rzeczą jest, że mając tylko jedną metodę params object[] wymusić dodatkowy przydzielanie tablicy dla połączenia.

Możesz zauważyć, że string.Format ma zakres przeciążeń do przyjmowania małej liczby argumentów (są one bardzo często używane) - Sugerowałbym ich powielenie.

swój „idealny” scenariusz może być wykonywane poprzez ponowne pisanie metodą string.Format, ale trzeba by przejść w wartościach, tj

return "date: {date}\ntime: {...}\nuser: {_user} ({_status})" 
    .Format(new { date = DateTime.Now, _user, _status }); 

(i stosując własną metodę zwyczaj Format lub one like this) - ale pamiętaj, że wymusza to nowe wystąpienie obiektu na każde połączenie.

W pewnym momencie kompilator mono miał flagę eksperymentalną, aby włączyć to bezpośrednio. Nie wiem, czy to się utrzymuje.

Answer 2

To zależy, czy kodujesz sam, czy masz zespół. W zespole jest to bardzo zły pomysł, ponieważ wszyscy będą musieli nauczyć się tej metody.

Innym problemem jest Format z argumentami dotyczącymi łańcuchów, które przypadkowo zawierają nawiasy o nieprawidłowym indeksie, takim jak {1} zamiast {2}. W ten sposób tylko zły ciąg spowoduje awarię całej aplikacji. Użyłem czegoś podobnego do mojego logowania i musiałem użyć try-catch dla FormatExceptions.

Answer 3

Używam również podobnej metody przedłużania i podoba mi się, podaję też informacje o kulturze w metodzie rozszerzenia, która jest dla systemu ustawiona w moim przypadku.

public static string Format(this string formatTemplate, params object[] args) 
{ 
    return string.Format(SysSettings.CultureInfo, formatTemplate, args); 
} 

Usages:

return "date: {0:dd.MM.yyyy}\ntime: {1:mm:HH:ss}\nuser: {2} ({3})".Format(DateTime.Now, DateTime.Now, _user, _status); 

Answer 4

To nie całkiem pasuje do Twojego ideału, ale coś takiego może pracować dla Ciebie:

public static class Extensions 
{ 
    public static string Format(this object data, string format) 
    { 
     var values = new List<object>(); 
     var type = data.GetType(); 
     format = Regex.Replace(format, @"(^|[^{])\{([^{}]+)\}([^}]|$)", x => 
     { 
      var keyValues = Regex.Split(x.Groups[2].Value, 
             "^([^:]+):?(.*)$") 
            .Where(y => !string.IsNullOrEmpty(y)); 

      var key = keyValues.ElementAt(0); 
      var valueFormat = keyValues.Count() > 1 ? 
           ":" + keyValues.ElementAt(1) : 
           string.Empty; 


      var value = GetValue(key, data, type); 

      values.Add(value); 
      return string.Format("{0}{{{1}{2}}}{3}", 
            x.Groups[1].Value, 
            values.Count - 1, 
            valueFormat, 
            x.Groups[3].Value); 
     }); 


     return string.Format(format, values.ToArray()); 
    } 

    private static object GetValue(string name, object data, Type type) 
    { 
     var info = type.GetProperty(name); 
     return info.GetValue(data, new object[0]); 
    } 
} 

ten powinien pozwolić, aby zrobić ten rodzaj formatowanie na dowolny obiekt:

new {Person = "Me", Location = "On holiday"} 
    .Format("{Person} is currently {Location}"); 

Umożliwia również dodanie formatowania:

new {Person = "Me", Until = new DateTime(2013,8,1)} 
    .Format("{Person} is away until {Until:yyyy-MM-dd}); 

Co to jest dla Ciebie? Jestem pewien, że kod mógłby zostać poprawiony pod względem wydajności, ale działa!