Code-Effizienz für verschachtelte Schleifen C#

Question

Hier ist meine Funktion in der ich verschachtelte Schleifen verwende wie:

public string HGSearchNew(HolidayFeedService.PackageHolidays.SearchCriteria objsearchcriteria, dynamic search) 
{ 
     XmlDocument xdoc = new XmlDocument(); 
     XmlNodeList ndepartures = xdoc.SelectNodes("Destinations/Departure"); 
     string sFinalDeparture = objsearchcriteria.DepartureAirport.ToUpper(); 

     for (int i = 0; i < objsearchcriteria.DepartureAirport.Split('|').Length; i++) 
     { 

      for (int j = 0; j < ndepartures.Count; j++) 
      { 
       if (objsearchcriteria.DepartureAirport.Split('|')[i].ToUpper() == ndepartures[j]["Name"].InnerText.ToUpper()) 
       { 
        if (!sFinalDeparture.Contains(objsearchcriteria.DepartureAirport.Split('|')[i].ToUpper())) 
         sFinalDeparture += objsearchcriteria.DepartureAirport.Split('|')[i].ToUpper() + "|"; 

        break; 
       } 
      } 
     } 
     return sFinalDeparture; 
    }  

Ich möchte wie anstelle von Schleifen dieser Code effizienter machen, die Verwendung von Inhalt oder irgendwelche Funktionen zum Vergleichen . bitte hilf mir aus?

Answer 1

Ich denke, man kann das ganze Verfahren mit diesem lesbar und effizient LINQ Ansatz ersetzen:

public string HGSearchNew(HolidayFeedService.PackageHolidays.SearchCriteria objsearchcriteria) 
{ 
    XmlDocument xdoc = new XmlDocument(); 
    XmlNodeList ndepartures = xdoc.SelectNodes("Destinations/Departure"); 
    string[] departureTokens = objsearchcriteria.DepartureAirport.Split('|'); 

    var matches = ndepartures.Cast<XmlNode>() 
     .Select(node => node.Name) 
     .Intersect(departureTokens, StringComparer.InvariantCultureIgnoreCase); 

    return string.Join("|", matches); 
}