Qualcuno ha idea su come ottimizzare questo ciclo?
foreach (var CampaignUser in CampaignUserList)
{
if (!TotalItemList.Contains(CampaignUser))// bisogna fare l'override del contains
Campaign.Items.Add(CampaignUser);
TotalItemList.Add(CampaignUser);
}
Tove sugli item ho fatto l'override:
public override bool Equals(object obj)
{
var other = (Item)obj;
return this.Utente == other.Utente;//il campo utente è un'intero
}
Su grossi numeri per esempio: 90000 è lentissimo.

Grazie

Daniele Bochicchio ha scritto:

TOM_YOOX wrote:

Qualcuno ha idea su come ottimizzare questo ciclo?
foreach (var CampaignUser in CampaignUserList)

non usare foreach, ma for con intero. essendo una lista, farà molta differenza
.

Si effettivamente con un for su un intero è miglirato parecchio. Poi un ulteriore miglioramento l'ho avuto invertendo l'if, in questo modo:
Item CampaignUser;
for (int i = 0; i < CampaignUserList.Count; i++)
{
CampaignUser = CampaignUserList[i];
if (TotalItemList.Contains(CampaignUser))
continue;
Campaign.Items.Add(CampaignUser);
TotalItemList.Add(CampaignUser);
}

Grazie