Метод расширения StringBuilder для добавления коллекции в C # - PullRequest
Новая
       18

Метод расширения StringBuilder для добавления коллекции в C #

5 голосов
/ 09 декабря 2008

В C # я пытаюсь создать метод расширения для StringBuilder с именем AppendCollection (), который позволил бы мне сделать это: 

var sb1 = new StringBuilder();
var sb2 = new StringBuilder();
var people = new List<Person>() { ...init people here... };
var orders = new List<Orders>() { ...init orders here... };

sb1.AppendCollection(people, p => p.ToString());
sb2.AppendCollection(orders, o => o.ToString());

string stringPeople = sb1.ToString();
string stringOrders = sb2.ToString();

stringPeople получит строку для каждого человека в списке. Каждая строка будет результатом p.ToString (). Аналогично для stringOrders. Я не совсем уверен, как написать код, чтобы лямбды работали с дженериками.

Ответы [ 7 ]

10 голосов
/ 09 декабря 2008

Используйте делегат Func<T,string>. 

public static void AppendCollection<T>(this StringBuilder sb, 
                                       IEnumerable<T> collection, Func<T, string> method) {
   foreach(T x in collection) 
       sb.AppendLine(method(x));
}
5 голосов
/ 09 декабря 2008 
 public static void AppendCollection<T>(this StringBuilder builder, IEnumerable<T> list, Func<T,string> func)
        {
            foreach (var item in list)
            {
                builder.AppendLine(func(item));
            }
        }

Я бы не вернул строку, я бы просто добавил ее к исходному Stringbuilder, который был передан.

3 голосов
/ 09 декабря 2008

Моя версия: 

    public static string AppendCollection<T>(this StringBuilder sb, IEnumerable<T> enumerable, Func<T, string> method)
    {
        List<T> l = new List<T>(enumerable);
        l.ForEach(item => sb.AppendLine(method(item)));
        return sb.ToString();
    }

но вы не должны возвращать строку в этом случае. Я бы предпочел следующее: 

    public static void AppendCollection<T>(this StringBuilder sb, IEnumerable<T> enumerable, Func<T, string> method)
    {
        List<T> l = new List<T>(enumerable);
        l.ForEach(item => sb.AppendLine(method(item)));
    }

будет использоваться как: 

        sb.AppendCollection(people, p => p.ToString());
        sb.AppendCollection(orders, o => o.ToString());
        Console.WriteLine(sb.ToString());
3 голосов
/ 09 декабря 2008

Что-то вроде: 

  public static void AppendCollection<TItem>(this StringBuilder builder, IEnumerable<TItem> items, Func<TItem, string> valueSelector)
  {
       foreach(TItem item in items)
       {  
            builder.Append(valueSelector(item));
       }
  }

Я бы добавил полезное значение по умолчанию, чтобы сохранить определение лямбды в 90% случаев ... 

   public static void AppendCollection<TItem>(this StringBuilder builder, IEnumerable<TItem> items)
  {
      AppendCollection(builder, items, x=>x.ToString());
   }
3 голосов
/ 09 декабря 2008

Я не уверен, что вам нужно так усердно работать: 

 public static void AppendCollection( this StringBuilder builder,
                                      ICollection collection )
 {
     foreach (var item in collection)
     {
        builder.AppendLine( Convert.ToString( item ) );
     }
 }

Используется как 

 List<Person> people = ...

 StringBuilder builder = new StringBuilder();
 builder.AppendCollection( people );
 var s = builder.ToString();

Конечно, Person необходимо переопределить ToString () для получения правильного вывода для объекта Person.

2 голосов
/ 09 декабря 2008 
static class SBExtention
{
  static string AppendCollection<T>(this StringBuilder sb, 
                                    IEnumerable<T> coll, 
                                    Func<T,string> action)
  {
       foreach(T t in coll)
       {
          sb.Append(action(t));
          sb.Append("\n");
       }
       return sb.ToString();

  }
}

Однако, я думаю, вам будет лучше, если он вернет StringBuilder. Таким образом, вы можете связать это: 

  static StringBuilder AppendCollection<T>(this StringBuilder sb, 
                                    IEnumerable<T> coll, 
                                    Func<T,string> action)
  {
       // same
       return sb;

  }

string peopleAndOrders = sb.AppendCollection (people, p => p.ToString ()) .AppendCollection (orders, o => o.ToString ()). ToString ();

И я согласен с Дженнифер по поводу случая по умолчанию: 

   public static StringBuilder AppendCollection<TItem>(
                  this StringBuilder builder, 
                  IEnumerable<TItem> items)
  {
      return AppendCollection(builder, items, x=>x.ToString());
   }

string peopleAndOrders = sb.AppendCollection (люди) .AppendCollection (приказы) .ToString ();

2 голосов
/ 09 декабря 2008

Что этот метод должен вернуть? Я вижу строку, но почему, если вы добавляете StringBuilder?

То, что вы пытаетесь сделать, довольно легко, но вам нужно точно объяснить, что вы хотите.

Обновление:

Вот мой дубль. Использование метода расширения для этого глупо и бессмысленно, если вы просто собираетесь передать новый StringBuilder и вернуть строку.

Обновление 2:

Теперь, когда я вижу, что использование, то, что вы делаете, это плохая практика. В идеале вы должны делать что-то вроде: 

public static string Print<T>(this IEnumerable<T> col, Func<T,string> printer)
{
  var sb = new StringBuilder();
  foreach (T t in col)
  {
    sb.AppendLine(printer(t));
  }
  return sb.ToString();
}

string[] col = { "Foo" , "Bar" };
string lines = col.Print( s => s);

Обновление 3:

После уточнения: 

public static void AppendCollection<T>(this StringBuilder sb, 
   List<T> col, Func<T,string> printer)
{
  col.ForEach( o => sb.AppendLine(printer(o)));
}

(то же самое, что сказал Бруно Конде)

А теперь тебе это больше не нужно:)

...