이 작은 코드의 ForEach Replace는 예상 한대로 작동하지 않습니다. 여기서 내가 뭘 잘못하고 있니?

Question

이 LINQPad 관련 아니면 내가 뭔가 잘못하고있어 경우 나도 몰라,하지만이 코드는 내가 특별히 ForEach(...)

내 목표는 교체하는 것입니다, 그것은 수행 할 작업을하지 않는다 "" 빈 문자열; 이 작업을 수행하는 더 좋은 방법이 있습니까?

var lastNames = "SMITH, JOHNSON, WILLIAMS, JONES, BROWN"; 

var listLastNames = lastNames.Split(','); 
var list = listLastNames.ToList(); //so I can use .ForEach 
list.ForEach(i=>i.Replace(" ",String.Empty)); 

list.Dump(); //show it on output 

Answer 1

다른 사람들이 지적했듯이, 문자열 불변이다. 바꾸기를 호출하면 새 문자열이 반환됩니다. 기존의 문자열을 변경하지 않습니다.

이 에 순서를 변환을 수행하고, 마지막에 목록으로 변환 :

string s = "SMITH, JOHNSON, WILLIAMS, JONES, BROWN"; 
List<string> lastNames = s.Split(',').Select(x=>x.Trim()).ToList(); 

또는 쿼리 구문에서 같은 일 : 여기 당신이 원하는 것을 할 수있는 세 가지 방법이 있습니다

string s = "SMITH, JOHNSON, WILLIAMS, JONES, BROWN"; 
var query = from lastName in s.Split(',') 
      select lastName.Trim(); 
List<string> lastNames = query.ToList(); 

또는 배열을하고 자리에 배열 변이 :

string s = "SMITH, JOHNSON, WILLIAMS, JONES, BROWN"; 
string[] lastNames = s.Split(','); 
for (int i = 0; i < lastNames.Length; ++i) 
    lastNames[i] = lastNames[i].Trim(); 

Answer 2

Replace는 새 값을 반환하지만 당신이 그것을에 전화를 원래의 문자열에 영향을주지 않습니다. 당신이 필요로하는 일을하려면 Replace의 결과로 새 컬렉션을 구축해야 할 것이다 - 당신이 Select 쉽게이 작업을 수행 할 수 있습니다

var replaced = list.Select(i=>i.Replace(" ",String.Empty)); 

또 다른 장점은, 당신은에 List<T>에 캐스트 할 필요가 없습니다 이 작업을 수행.

그리고 다른 사람이 지적한 것처럼, 당신은 Replace()보다 청소기 솔루션으로 Trim()를 사용할 수 있습니다

var collection = lastNames.Split(',').Select(i => i.Trim());