ForEach и foreach

Question

Я пытаюсь добавить несколько строк в MailAddress на C#.

Если бы я должен был использовать ForEach, мой код будет выглядеть

 foreach (var item in GetPeopleList()) 
     { 
      m.Bcc.Add(new MailAddress(item.EmailAddress)); 
     } 

Я сейчас пытаюсь сделать это с моим Еогеаспом (т.е. List.ForEach()), и я не могу.

public class Person 
    { 
     public Person(string firstName, string lastName, string emailAddress) 
     { 
      FirstName = firstName; 
      LastName = lastName; 
      EmailAddress = emailAddress; 
     } 

     public string FirstName { get; set; } 
     public string LastName { get; set; } 
     public string EmailAddress { get; set; } 
    } 

     static void Main(string[] args) 
     { 
      MailMessage m = new MailMessage(); 
      List<Person> people = GetPeopleList(); 

      m.Bcc.Add(people.ForEach(Person people => 
       { 
        //what goes here? 
       } 
      )); 
     } 

     private static List<Person> GetPeopleList() 
     { 
      List<Person> peopleList = new List<Person>(); 
      //add each person, of type Person, to the list and instantiate the class (with the use of 'new') 
      peopleList.Add(new Person("Joe", "Bloggs", "Joe.Bloggs@foo.bar")); 
      peopleList.Add(new Person("John", "Smith", "John.Smith@foo.bar")); 
      peopleList.Add(new Person("Ann", "Other", "Ann.Other@foo.bar")); 
      return peopleList; 
     } 

Я пробовал несколько вариантов/вариантов этого, но я явно делаю что-то неправильно. Я прочитал об этом Eric Lippert's page и, к сожалению, это тоже не помогло.

Answer 1

Вам потребуется что-то вроде

people.ForEach(Person p => { 
    m.Bcc.Add(new MailAddress(p.EmailAddress)); 
}); 

Вместо добавления одного диапазона выбранных элементов с ForEach, вы добавляете один пункт ForEach человека в списке.

Это сказало ... Я бы предпочел обычный цикл foreach.

Answer 2

Я не знаю, если я правильно понимаю, но попробуйте:

foreach (var item in GetPeopleList()) 
{ 
    m.Bcc.Add(item.EmailAddress)); 
} 

Вы можете создать новый адрес электронной почты в вашем коде, но это не требуется, потому что вы уже получаете адрес электронной почты, от item.

Answer 3

Прямая цитата из блога:

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

foreach (Foo foo in foos) {statement with foo; }

в этот код:

foos.ForEach ((Foo Foo) => {заявление с участием Foo;});

, который использует почти точно такие же символы в немного отличающемся заказе . И все же вторая версия сложнее понять, сложнее до debug и вводит семантику закрытия, тем самым потенциально изменяя время жизни объектов в тонких направлениях.

Эрик Липперт явно призывает не делать этого.

Answer 4

попробовать

people.ForEach(Person person => 
    { 
     m.Bcc.Add(new MailAddress(person.EmailAddress)); 
    }); 

Answer 5

Linq агрегат может дать хорошее решение.

 MailMessage m = new MailMessage(); 
     GetPeopleList().Aggregate((result, iter) => 
      { 
       m.Bcc.Add(new MailAddress(iter.EmailAddress)); 
       return result; 
      });