T [] параметр в строке []. Foreach()

Question

Я просто привык к использованию лямбда и всему этому хорошему. Однако я не знаю, какой аргумент мне нужно предоставить в этом случае.

Я пытаюсь изменить строку I, чтобы заменить все экземпляры записи в словаре с ее значением, ничего необычного, я мог бы сделать это с помощью простого цикла foreach, но я пытаюсь использовать его в качестве учебного упражнения ,

Мой код до сих пор выглядит следующим образом:

 string maskString = Masks[alias]; 
     Regex regex = new Regex(@"({[\w+|\d]+})"); 
     MatchCollection matches = regex.Matches(maskString); 
     string[] constants = matches.Cast<Match>().Select(m => m.Value).ToArray(); 
     string maskedString = ""; 
     if (constants.All(constant => Constants.ContainsKey(constant))) 
     { 
      constants.ForEach(constant => maskedString = maskedString.Replace(constant, Constants[constant])); 
     } 

Однако, я получаю эту ошибку: There is no argument given that corresponds to the required formal parameter 'action' of 'Array.ForEach<T>(T[], Action<T>)'. Мне кажется, что я предоставляю действие в форме выражения lambda, но я понятия не имею, зачем ему нужен другой массив или какой массив я должен ему дать.

Я попытался предоставить его null, string[] и new [] {typeof(string)}, но это просто вызывает больше ошибок.

Возможно ли, чтобы эта операция проводилась с использованием этой структуры или было бы более практичным использовать традиционный цикл foreach?

Answer 1

В любое время вы бы лучше использовать Агрегатную функцию:

maskedString = constants.Aggregate(maskedString, (current, constant) => current.Replace(constant , Constants[constant])); 

Answer 2

Похоже, что это неправильное использование метода .ForEach(), это метод static класса Array, а не метод расширения. Есть два способа выполнить это правильно:

Во-первых, использует Array.ForEach (статический метод) и обеспечивает массив в качестве первого параметра:

Array.ForEach(constants,constant => maskedString = maskedString.Replace(constant, Constants[constant])); 

Во-вторых, отбрасывает массив к первому List<T>:

constants.ToList().ForEach(constant => maskedString = maskedString.Replace(constant, Constants[constant])); 

Второй метод не изменяет исходный массив и, следовательно, если это требуется вам придется сделать myArray = myArray.ToList().ForEach().ToArray(). Спасибо Крису за это.