Замена вложенного foreach на LINQ; изменить и обновить свойство глубоко внутри

foreach(var item in company.internalData.Emps
                        .SelectMany(emp => emp.privateData.Addresses)
                        .SelectMany(addr => addr.Items)
                        .Where(addr => addr.Type == "StreetAddress"))
     item.Text = CleanStreetAddressLine(item.Text);

var dirtyAddresses = company.internalData.Emps.SelectMany( x => x.privateData.Addresses )
                                              .SelectMany(y => y.Items)
                                              .Where( z => z.Type == "StreetAddress");

  foreach(var addr in dirtyAddresses)
    addr.Text = CleanStreetAddressLine(addr.Text);

Грязный однострочник.

company.internalData.Emps.SelectMany(x => x.privateData.Addresses)
    .SelectMany(x => x.Items)
    .Where(x => x.Type == "StreetAddress")
    .Select(x => { x.Text = CleanStreetAddressLine(x.Text); return x; });

LINQ не позволяет иметь побочные эффекты. однако вы можете сделать это:

company.internalData.Emps.SelectMany(emp => emp.Addresses).SelectMany(addr => Addr.Items).ToList().ForEach(/*either make an anonymous method or refactor your side effect code out to a method on its own*/);

Вы можете это сделать, но на самом деле вы этого не хотите. Несколько блоггеров говорили о функциональной природе Linq, и если вы посмотрите на все методы Linq, поставляемые MS, вы обнаружите, что они не вызывают побочных эффектов. Они производят возвращаемые значения, но больше ничего не меняют. Найдите аргументы в методе Linq ForEach, и вы получите хорошее объяснение этой концепции.

Имея это в виду, то, что вы, вероятно, хотите, будет примерно таким:

var addressItems = company.internalData.Emps.SelectMany(
    emp => emp.privateData.Addresses.SelectMany(
           addr => addr.Items
    )
);
foreach (var item in addressItems)
{
   ...
}

Однако, если вы действительно хотите сделать именно то, что вы спросили, тогда вам нужно двигаться в этом направлении:

var addressItems = company.internalData.Emps.SelectMany(
    emp => emp.privateData.Addresses.SelectMany(
           addr => addr.Items.Select(item =>
           { 
              // Do the stuff
              return item;
           }) 
    )
);

Мне не нравится смешивать синтаксис «понимания запроса» и синтаксис вызова метода с точками в то же утверждение.

Мне нравится идея отделить запрос от действия . Они семантически различны, поэтому разделение их в коде часто имеет смысл.

var addrItemQuery = from emp in company.internalData.Emps
                    from addr in emp.privateData.Addresses
                    from addrItem in addr.Items
                    where addrItem.Type == "StreetAddress"
                    select addrItem;

foreach (var addrItem in addrItemQuery)
{
    addrItem.Text = CleanStreetAddressLine(addrItem.Text);
}

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

• В общем, я избегаю сокращений ( Emps , emp , addr )
• Несогласованные имена более запутаны ( адрес vs. Адреса ): выберите один и придерживайтесь его
• Слово «число» неоднозначно. Это может быть удостоверение личности («Заключенный номер 378, пожалуйста, сделайте шаг вперед». ) или счет («количество овец на этом поле - 12»). Поскольку мы часто используем обе концепции в коде, важно прояснить это. Я часто использую "index" для первого и "count" для второго.
• Наличие в поле type строки - это запах кода. Если вы можете сделать его enum , ваш код, вероятно, будет лучше.