Что случилось с этим циклом ForEach?

Question

Да ... это один из тех дней.

public string TagsInput { get; set; } 

//further down 
var tagList = TagsInput.Split(Resources.GlobalResources.TagSeparator.ToCharArray()).ToList(); 
tagList.ForEach(tag => tag.Trim()); //trim each list item for spaces 
tagList.ForEach(tag => tag.Replace(" ", "_")); //replace remaining inner word spacings with _ 

Обе петли ForEach не работают. tagList - это просто список.

Спасибо!

Answer 1

Trim() и Replace() не изменить строку, они призвали. Они создают новую строку, на которую было применено действие.

Вы хотите использовать Select, а не ForEach.

tagList = tagList.Select(t => t.Trim()).Select(t => t.Replace(" ", "_")).ToList(); 

Answer 2

Если по не работают, вы имеете в виду, что они на самом деле не делают ничего, я думаю, вам нужно настроить ваш код немного:

public string TagsInput { get; set; } 

//further down 
var tagList = TagsInput.Split(Resources.GlobalResources.TagSeparator.ToCharArray()).ToList(); 
tagList.ForEach(tag => tag = tag.Trim()); //trim each list item for spaces 
tagList.ForEach(tag => tag = tag.Replace(" ", "_")); //replace remaining inner word spacings with _ 

Обрезка и замена не изменяет значение строки, они возвращают новое строковое значение.

Answer 3

ForEach (и другие методы «linq») не изменяет экземпляр списка.

tagList = tagList.Select(tag => tag.Trim().Replace(" ", "_")).ToList(); 

Answer 4

Причина в том, что строка является неизменной. Таким образом, результат каждой функции Trim() или Replac() приведет к созданию новой строки. Вам нужно переназначить исходный элемент, чтобы увидеть обновленное значение.

Answer 5

Именно поэтому Microsoft havent внедрил ForEach в IEnumerable. Что случилось с этим?

public string[] TagsInput { get; set; } 

//further down 
var adjustedTags = new List<string>(); 
foreach (var tag in TagsInput.Split(Resources.GlobalResources.TagSeparator.ToCharArray())) 
{ 
    adjustedTags.Add(tag.Trim().Replace(" ", "_")); 
} 

TagsInput = adjustedTags.ToArray();