Question

我有以下C＃方法用于预加载库存数据表。虽然它工作得很好，但现在我在表中有很多行，加载速度非常慢。

请有人推荐一种更好，更快的方法吗？（理想情况下，删除“foreach”代码，因为这是缓慢的位！）。

public static DataTable GetProducts()
{
    DataTable table = new DataTable();

    using (DataClassesDataContext data = new DataClassesDataContext(cDbConnection.GetConnectionString()))
    {
        var query = (from p in data.Products
                     where p.Deleted == false
                     join s in data.ProductStocks on p.ProductID equals s.ProductID
                     group s by p into g
                     select new { g });


        table.Columns.Add("Barcode", typeof(string));
        table.Columns.Add("Stock Code", typeof(string));
        table.Columns.Add("Description", typeof(string));
        table.Columns.Add("Price", typeof(string));
        table.Columns.Add("Tax", typeof(string));
        table.Columns.Add("Stock", typeof(string));
        table.Columns.Add("Service Item", typeof(bool));
        table.Columns.Add("Deduct Stock", typeof(bool));

        if (query != null)
        {
            foreach (var item in query)
            {
                try
                {
                    decimal? Tax = 0;
                    if (item.g.Key.ProductTax != null)
                    {
                        Tax = Common.Utilities.IsValueValidDecimal(item.g.Key.ProductTax.TaxRate, 0);   // Tax
                    }
                    else
                    {
                        Tax = 0;
                    }

                    bool DeductStock = !Convert.ToBoolean(item.g.Key.ServiceItem);

                    string[] row = new string[] 
                    {
                        item.g.Key.EANCode.ToString(),       // Barcode
                        item.g.Key.OurStockCode.ToString(),  // Product Code
                        item.g.Key.Description.ToString(),   // desc
                        GetGUIDisplayPrice(item.g.Key.RetailPrice, item.g.Key.RetailPriceExVAT),  // cost
                        Tax.ToString(),                         // Tax   
                        item.g.Sum(s => s.QtyOnHand).ToString(), // Stock
                        item.g.Key.ServiceItem.ToString(),   // Service Item (non-stock)
                        DeductStock.ToString()                  // if not a service item, the its a stocked item so deduct!
                    };

                    table.Rows.Add(row);
                }
                catch (Exception ex)
                {
                }
            }
        }//ENDIF NULL
    }//END USING

    return table;
}

Answer 1

from p in data.Products
                 where p.Deleted == false
                 join s in data.ProductStocks on p.ProductID equals s.ProductID
                 group s by p into g
                 select new { g }

Products和ProductStocks表的架构是什么？你有什么指数？首先阅读How to analyse SQL Server performance。

有些事情立即脱颖而出：

您从客户端的服务器获取所有数据。别。处理后端。
使用Deleted位字段是（性能）灾难的一个方法。你可以添加一个聚集索引最左边的键，最好的结果可疑。分区可以帮助，但不是很多。没有银弹。尽量消除这一要求。删除已删除的行。

没有太多优化空间。停止提取所有数据。

Answer 2

最后，此函数变为存储过程，存储过程返回在服务器上创建的表，而不是客户端。这几乎是即时和大规模的性能提升！

优化并加速非常慢的Linq / SQL代码

2 个答案: