C# Double Foreach Optimierung

Question

Ich habe ein Problem hier, vor kurzem fand ich dieses Problem mit einem Web-Projekt (erstellt in Sitefinity), wenn ich versuche, eine ganze Liste von Artikeln, die ich im Cache für einen schnelleren Zugriff hinzufügen müssen .

Ich habe festgestellt, dass einige Genie Entwickler XD verwendet, um die gesamte Liste der Artikel, die zu bestimmten Kategorie, die in der Taxon (einige Eigenschaft von Sitefinity, die diese Art von Kategorie ex erkennt erkennt: Schwangere - Babys - Kinder ... Sie haben die Idee), und müssen einige Suche in einer Artikelsammlung mit dynamischen Inhalten, wo hat die Informationen über die Artikel (Titel, Bilder, Stile, Texte).

Nun, mein Problem ist, dass sie es mit einer doppelten Schleife Forschung mit ForEach getan haben, und mein Chef möchte diese Schleife "optimieren", denn natürlich dauert es zu lange, um diese Schleife zu machen (manchmal kann es sein mehr als 3 oder 4 Minuten, und ja, mein Chef möchte diesen Prozess sofort haben :)).

hier ist der Code:

foreach (HierarchicalTaxon subt in listWeek) 
    { 
     foreach (DynamicContent item in myCollection) 
     { 
      if (item.Organizer.TaxonExists("Category", subt.Id)) 
      { 
       item.Author = subt.Name; 
       //if (articles.Where(art => art.Author.Equals(item.Author)).Count() == 0) 
       articles.Add(item); 
      } 
     } 
    } 

Also muss ich mit dieser Doppel-Loop befassen, die diese 2^n Implementierung hat, meine Frage ist, wie kann ich es tun?!?!?!

Ich dachte über das Hinzufügen einiger LINQ, wie ich in anderen Threads sah, aber es scheint viel zu langsam als dieser tatsächliche Code.

Es sieht so aus, dass myCollection nicht viele Elemente enthält, aber ich bin ziemlich sicher, dass ich diese Art der Optimierung in anderen Sätzen mit derselben Logik in anderen Projekten verwenden muss.

bitte Jungs, ein wenig Unterstützung würde geschätzt werden, ist es möglich, es schneller zu tun ??

---

Also hier gehen wir, was denkst du Leute, ist das ok?

var data = listWeek.ToDictionary(x => x.Id, x => x.Name); 
var iDdata = data.Keys; 
var nombresData = data.Values; 

foreach (DynamicContent item in myCollection) 
{ 
    if (item.Organizer.TaxonExists("Category", iDdata.First())) 
    { 
     item.Author = nombresData.First(); 
     articlesPrueba1.Add(item); 
    } 
} 

Edit: 16/08/2016 -> Nicht gerade arbeiten, zunächst einmal, ich Probleme habe, wenn ich versuche, die Werte des Wörterbuch zugreifen, so lege ich in separaten Variablen für Id und Name, ich denke, das sollte es tun, aber da bin ich mir nicht sicher.

Aber dann hatte ich dieses Problem, macht nur eine Runde in den Inhalt der Wörterbuchdaten, es scheint offensichtlich, aber ich dachte, dass würde funktionieren, aber leider, tat es nicht.

---

Ich habe versucht, LINQ zu verwenden:

foreach (HierarchicalTaxon subt in listWeek) 
{ 
    foreach (DynamicContent item in myCollection.Where(item => item.Organizer.TaxonExists("Category", subt.Id))) 
    { 
     item.Author = subt.Name; 
     articlesPrueba2.Add(item); 
     } 
    } 
} 

aber ich habe diese Ausnahme um die Abfrage auf die Datenbank zugreift, verdammt Sitefinity, ich hasse dich ... Aber macht nichts, ich don Es interessiert mich nicht wirklich an LINQ.

immer noch in ernsthaften Optimierungsproblem.

Answer 1

Sie können den Code optimieren, indem Sie die letzte Woche in eine Dictionary konvertieren, in Form von KeyValue-Paaren aus ID und Name.

var data = listWeek.ToDictionary(x => x.Id, x => x.Name); 

Als nächstes durchlaufen myCollection wie Sie sind jetzt mit einem Verfahren zu tun Kategorie von Wörterbuch zu finden.

Answer 2

Dieser Code ist sehr ineffizient, das ist sicher.

Von dem, was ich aus dem Code bekommen kann, möchten Sie im Grunde überprüfen, ob ein dynamisches Content-Element eine Kategorie hat und wenn ja, den Namen dieser Kategorie und weisen Sie es der Author-Eigenschaft zu.

Ein besserer Ansatz wäre, die Sammlung der dynamischen Inhaltselemente (nur einmal) zu wiederholen und zu überprüfen, ob das Element eine Kategorie hat. Wenn dies der Fall ist, suchen Sie das Taxon mit dem Taxonomie-Manager und erhalten Sie seinen Namen/Titel und weisen Sie ihm den dynamischen Inhalt zu.

Auf diese Weise müssen Sie die Sammlung einmal durchlaufen und dann die Taxonomien nur bei Bedarf abfragen.