Reduzieren Sie mehrere foreach Schleifen in Code C#

Question

Ich habe einen wirklich chaotischen Code geschrieben, nur weil ich keine Idee hatte, wie Sie durch die mehrere Liste der Objekte durchlaufen.

Code funktioniert, was er tun soll, aber ich bin nicht davon überzeugt. Hat jemand eine Idee, wie diese Art von Code umstrukturiert werden kann, damit sie besser lesbar ist?

foreach (var outlet in merchant.Outlets) 
{ 
    if (outlet != null) 
    { 
     foreach (var terminal in merchant.Terminals) 
     { 
      if (terminal != null) 
      { 
       foreach (var agreement in terminal.AssociateAgreements) 
       { 
        var bankInfo = new Bank(); 
        if (agreement != null) 
        { 
         bankInfo.UniqueID = AgreementUniqueCode + agreement.ID; 
         bankInfo.BankBIC = agreement.BankAccountInformation.SwiftBIC; 
         bankInfo.BankName = agreement.BankAccountInformation.BankName; 
         bankInfo.Address = 
          AddressEntityToAddress(agreement.BankAccountInformation.BankAddress, 
           BankingType); 

         bankInfo.type = BankType; 
        } 
        banksAccountInformation.Add(bankInfo); 
       } 
      } 
     } 
    } 
} 

Answer 1

Sie können auch die Linq-Abfragesyntax verwenden:

var outletQuery = from outlet in merchant.Outlets 
        where outlet != null 
        select outlet; 

var agreementQuery = from terminal in merchant.Terminals 
        where terminal != null 
        from agreement in terminal.AssociateAgreements 
        select agreement; 

foreach (var outlet in outletQuery) 
{ 
    foreach (var agreement in agreementQuery) 
    { 
     ProcessAgreement(agreement); 
    } 
} 

Answer 2

Sie LINQ verwenden könnte, vor allem SelectMany und Where:

List<Bank> banksAccountInformation = merchant 
    .Where(m => m.Outlets != null && m.Terminals != null) 
    .SelectMany(m => m.Terminals 
     .SelectMany(t => t.AssociateAgreements 
      .Where(aa => aa != null) 
      .Select(aa => new Bank 
      { 
       UniqueID = AgreementUniqueCode + aa.ID 
       // ... 
      }))) 
    .ToList(); 

Answer 3

Wenn die einzige Sache, die Sie auf der Objekthierarchie bohrt möchten, dann können Sie extension methods oder Linq verwenden:

// here you get a flattened collection of Terminals 
merchants.SelectMany(m => m.Terminals) 
// here you get a flattened collection of all AssociateAgreements of all Terminals 
    .SelectMany(t => t.AssociateAgreements)