Почему моя попытка обрезать строки в списке <string>не работает?

Question

Я попробовал следующий код в LINQPad и получил следующие результаты:

List<string> listFromSplit = new List<string>("a, b".Split(",".ToCharArray())).Dump();
listFromSplit.ForEach(delegate(string s) 
{ 
  s.Trim(); 
});
listFromSplit.Dump();

«а» и «б»

чтобы буква b не убрала пробел, как я ожидал ...?

У кого-нибудь есть идеи

[ПРИМЕЧАНИЕ: метод .Dump () - это метод расширения в LINQPad, который печатает содержимое любого объекта в хорошем интеллектуальном формате]

Answer 1

вы просто создаете обрезанную строку, не назначая ей ничего.

var s = "  asd   ";
s.Trim();

не будет обновлять s, пока ..

var s = "   asd   ";
s = s.Trim();

будет ..

var listFromSplit = "a, b".Split(',').Select(s=>s.Trim());

было бы, я полагаю, быть таким, как я поступил бы.

Answer 2

Метод String.Trim () возвращает строку, представляющую обновленную строку. Он не обновляет сам строковый объект, а создает новый.

Вы можете сделать это:

s = s.Trim();

Однако вы не можете обновить коллекцию при перечислении через нее, поэтому вам нужно либо заполнить новый список, перечисляя существующий, либо заполнить список вручную, используя строковый массив, возвращаемый String.Split.

Заполнение нового списка:

List<string> temp = new List<string>("a, b".Split(",".ToCharArray()));
List<string> listFromSplit = new List<string>();

temp.ForEach(delegate(string s) 
{ 
    listFromSplit.Add(s.Trim()); 
});

listFromSplit.Dump();

Заполнение вручную:

string[] temp = "a, b".Split(",".ToCharArray());
List<string> listFromSplit = new List<string>();

foreach (string s in temp)
{
    listFromSplit.Add(s.Trim()); 
};

listFromSplit.Dump();

Answer 3

В дополнение к ответу, опубликованному Адриан Кун , вы можете сделать следующее:

var result = listFromSplit.Select(s => s.Trim());

Answer 4

Экземпляры строки являются неизменяемыми. Все, что может изменить его, создает новый экземпляр.

Answer 5

У меня нет запущенной IDE, но это должно сделать работу (если я не ошибаюсь):

var result = from each in listFromSplit select each.Trim();

Answer 6

Вы не назначаете усеченный результат чему-либо. Это классическая ошибка, я только что избавился от привычки делать эту ошибку с помощью string.Replace:)

Answer 7

Параметры linq, предоставленные другими, должны хорошо работать. В качестве другого варианта, вот метод расширения, использующий цикл for:

    public static void TrimCollection(this IList<string> stringCollection) {

        for (int i = 0; i <= stringCollection.Count() - 1; i++)
            stringCollection[i] = stringCollection[i].Trim();

    }

Answer 8

Разделите пробелы и запятые и удалите все пустые записи. Все красиво и аккуратно. Предполагается, что ваши строки не содержат пробелов.

List<string> listFromSplit =
     new List<string>( "a , b ".Split( new char[] { ',', ' ' }, StringSplitOptions.RemoveEmptyEntries ));