嵌套循环优化

Question

大家好我有一个奇怪的优化问题，这里是代码我已经改变了一些名字为简单起见

CollectionObject mycollobj = new CollectionObject(); 
List<string> MyProducts = new List<string>(); 
//get collection of selected customers that were passed in 
var chckedValues = form.GetValues("assignChkBx"); 
foreach(string customer in chckedValues) 
{ 
    MyProducts.Clear(); 
    //MyProducts is then set to a data access method in my data access class 
    MyProducts = DataLayerClass.GetProductsFromCustomer(customer); 
    foreach(string product in MyProducts) 
    { 
     string item1 = DataLayerClass.GetItem1(product); 
     string item2 = DataLayerClass.GetItem2(product); 
     mycollobj.loaditems(item1, item2); 
    } 
} 

本质mycollobj是用于一些相当复杂的分析，一个黑盒子（即我无法控制）。有没有更好的方法来运行这个嵌套算法？任何建议都很重要，请问你是否需要澄清任何事情。谢谢！

Answer 1

是的，这条线：MyProducts = DataLayerClass.GetProductsFromCustomer(customer);会减慢速度（每个客户都需要一个数据库），嵌套的DataLayerClass.GetItem1()/GetItem2也会让事情变得更糟。而是发送了所有的checkedValues到数据库并与客户和含有item1和item2一个Tuple返回查找：

ILookup<Customer, Tuple<string, string>> customerProducts = 
DataLayerClass.GetCustomersWithProducts(chckedValues); 

总之而言，移动逻辑到一个单一的数据库查询。