Question

问题陈述： 我正在尝试使用Linq查询将数据库中的多表数据绑定到视图，这需要花费更多时间。我在db.Someone中有大约10000条记录建议使用IQueryable而不是IEnumerable，但它是否会影响我当前的代码（在视图和控制器中）??或者没有使用我可以完成这个？

我应该做什么才能提高加载结果的性能？ 我做错了什么？ 请建议我采取更好的方法......

控制器：

public ActionResult Index()
    {
        var result = (from pr in db.Prod.AsEnumerable()
                      join s in db.Shift.AsEnumerable() on pr.Shift equals s.ShiftID
                      join m in db.Module.AsEnumerable() on pr.Module equals m.ModuleID
                      select new GlobalModel()
                      {
                          prodModelIndex = pr,
                          prodModel = prodModel,
                          shiftModel = s,
                          moduleModel = m,
                          ddlShift = objTransactionGeneralController.GetAllShift(),
                          ddlModule = objTransactionGeneralController.GetAllModule()

                      }).ToList();
    return PartialView(result);

    }





   public TransGeneralModel GetAllModule()
    {

        objTransGeneralModel.ddlModule = (from m in db.Module.AsEnumerable()
                                           select new SelectListItem
                                           {
                                               Value = m.ModuleID.ToString(),
                                               Text = m.ModuleName,
                                           }).ToList();


        return objTransGeneralModel;
    }

    public TransGeneralModel GetAllShift()
    {

        objTransGeneralModel.ddlShift = (from s in db.Shift.AsEnumerable()
                                          select new SelectListItem
                                          {
                                              Value = s.ShiftID.ToString(),
                                              Text = s.ShiftName,
                                          }).ToList();


        return objTranGeneralModel;
    }

查看：

@model IEnumerable<SIA.Models.Trans.GlobalModel>

@using GridMvc.Html

@{
    Layout = "~/Views/Shared/_Layout.cshtml";
    ViewBag.Title = "Index";

    }

    <link rel="stylesheet" href="@Url.Content("~/Content/jquery.dataTables.min.css")">
    <script src="@Url.Content("~/Scripts/jquery-2.1.1.min.js")"></script>


    <h2>Details</h2>
    <hr />

    <div style="width: 1000px; padding-left: 70px">
        @Html.Partial("Create")
        <br />
    </div>

    <h5 class="pull-right">
        <b class="fa fa-keyboard-o" style="color: blue"></b>
        @Ajax.ActionLink("Edit", "ProdEdit", "Prod", new { }, new AjaxOptions
    {
        InsertionMode = InsertionMode.Replace,
        UpdateTargetId = "prod-details",
        HttpMethod = "GET",
    }, new { style = "color:blue" })
    </h5>

    <br />
    @using (Html.BeginForm())
    {
        @Html.AntiForgeryToken()
        @Html.ValidationSummary(true)
        if (Model.FirstOrDefault().prodModelIndex != null)
        { 

        <div id="prod-details">
            <table class="table table-striped" id="tblProdDetails">
                <thead>
                    <tr>

                        <th>
                            @Html.DisplayNameFor(model => model.FirstOrDefault().prodModelIndex.ProdID)
                        </th>
                        <th>
                            @Html.DisplayNameFor(model => model.FirstOrDefault().prodModelIndex.Date)
                        </th>
                        <th>
                            @Html.DisplayNameFor(model => model.FirstOrDefault().prodModelIndex.Module)
                        </th>
                        <th>
                            @Html.DisplayNameFor(model => model.FirstOrDefault().productionModelIndex.Shift)
                        </th>

                        <th>
                            @Html.DisplayNameFor(model => model.FirstOrDefault().prodModelIndex.Hour)
                        </th>
                        <th>
                            @Html.DisplayNameFor(model => model.FirstOrDefault().prodModelIndex.Output)
                        </th>

                    </tr>
                </thead>
                <tbody>
                    @foreach (var item in Model)
                    {
                        <tr id="customer-row-@item.prodModelIndex.ProdID">


                            <td>
                                @Html.DisplayFor(modelItem => item.prodModelIndex.ProdID)
                            </td>
                            <td>
                                @Html.DisplayFor(modelItem => item.prodModelIndex.Date)
                            </td>
                            <td>
                                @Html.DisplayFor(modelItem => item.moduleModel.ModuleName)
                            </td>
                            <td>
                                @Html.DisplayFor(modelItem => item.shiftModel.ShiftName)
                                @Html.HiddenFor(modelItem => item.prodModelIndex.Shift)
                            </td>

                            <td>
                                @Html.DisplayFor(modelItem => item.prodModelIndex.Hour)
                            </td>
                            <td>
                                @Html.DisplayFor(modelItem => item.prodModelIndex.Output)
                            </td>
                        </tr>
                    }
                </tbody>
            </table>

        </div>
        }
    }

    <script>

        $(document).ready(function () {

            $('#tblProdDetails').dataTable({
                "order": [[1, "desc"], [3, "asc"]]

            });

        });
    </script>

    @section Scripts {
        @Scripts.Render("~/bundles/jqueryval")
        @Scripts.Render("~/Scripts/jquery.dataTables.min.js")
        <script type='text/javascript'>
            $(function () {
                $('.datepicker').datepicker({
                    format: "dd M yyyy",
                }).on('changeDate', function (e) {
                    $(this).datepicker('hide');
                });
            })

        </script>
    }

Answer 1

首先，当您调用ToList()，AsEnumerable()或FirstOrDefault()等方法时，它将对数据库执行查询。在您的情况下，最好删除它们以使用joins来查看单个查询。

var result = (from pr in db.Prod
                      join s in db.Shift on pr.Shift equals s.ShiftID
                      join m in db.Module on pr.Module equals m.ModuleID
                      select new GlobalModel()
                      {
                          prodModelIndex = pr,
                          prodModel = prodModel,
                          shiftModel = s,
                          moduleModel = m

                      }).ToList();

Answer 2

如果表由外键引用，则无需加入。你可以像这样直接访问推荐的记录

var result = (from pr in db.Prod
                      select new GlobalModel()
                      {
                          prodModelIndex = pr,
                          prodModel = prodModel,
                          shiftModel = pr.Shift,
                          moduleModel = pr.Module     
                      }).ToList();

Answer 3

<强> TL; DR

使用.Where过滤主表（Prod）或至少使用.Take()将行数限制为在屏幕上显示的内容
删除.AsEnumerable() - 您正在将整个表格实现到内存中
向表中添加外键，重新生成DBML，并使用导航而不是显式连接
请注意您在Select投影中添加的内容 - 将为结果集中的每一行调用ddlShift = objTransactionGeneralController.GetAllShift()。

详情

将.AsEnumerable()应用于您的收藏集，例如：

var result = (from pr in db.Prods.AsEnumerable()
      join s in db.Shifts.AsEnumerable() on pr.ShiftID equals s.ShiftId
      join m in db.Modules.AsEnumerable() on pr.ModuleID equals m.ModuleId
      select new ...

您当前的代码会导致对Sql Server的3个显式查询，每个查询都会将整个表加载到内存中:(例如，使用Sql Profiler或LinqPad等）

SELECT [t0].[ModuleId], ... other columns
FROM [dbo].[Module] AS [t0];

SELECT [t0].[ShiftId], ... other columns
FROM [dbo].[Shift] AS [t0];

SELECT [t0].[ProdID], [t0].[ShiftID], [t0].[ModuleID], ... other columns
FROM [dbo].[Prod] AS [t0];

鉴于您根本没有WHERE谓词，这可能不会比加入数据库慢得多。但是，总的来说，这样做根本不是一个好主意，因为：

通过应用.AsEnumerable()，您将剥夺Linq2Sql将IQueryable表达式树解析为本机Sql的能力。通常，在数据库中进行连接和过滤会更快，并且比在内存中执行此操作所需的内存更少。假设Prod，Shift和Module是Linq.Table<> s，此处的解决方案是简单地删除.AsEnumerable() - 这将允许Linq使用IQueryable扩展加入，过滤，聚合等方法
检索表中的所有行并在单个屏幕中一次显示所有行是不常见的，除非表的大小保证有少量行。通常，您会对表格应用某种过滤器。
根据Bhaarat的评论，如果您正确地在表之间设置了外键（因为示例代码暗示了它，看起来有设计的连接键），当您将表导入Linq2Sql DBML时，您还将获得实体之间的导航，因此不需要明确地加入表。
对于结果集中的每一行，都会调用此Select投影 - ddlShift = objTransactionGeneralController.GetAllShift()。这看起来很贵。执行一次，将结果存储在局部变量中，如果需要，在每个投影中引用它。或者更改您的ViewModel，以便它不会在每一行重复引用，如果需要一次。
延迟加载可能是一个性能问题（1到N问题） - 通过db.DeferredLoadingEnabled = false在DataContext上关闭它，而是显式指定要用适当的{{加载的图表的深度1}}陈述

您的代码现在看起来像这样：

LoadWith<>

生成的SQL将是单个查询，具有更准确的行限制：

using (var db = new DataClasses1DataContext())
{
    // Switch off Lazy Loading in favour of eager loading
    db.DeferredLoadingEnabled = false;
    var ds = new DataLoadOptions();
    ds.LoadWith<Prod>(p => p.Shift);
    ds.LoadWith<Prod>(s => s.Module);

    // Do this once, not in a tight loop
    var ddlShift = objTransactionGeneralController.GetAllShift();
    var ddlModule = objTransactionGeneralController.GetAllModule();

    var result = db.Prods
        .Where(p => p.ProdID > 5 && p.ProdID < 10) // Apply some kind of filtering
        .Take(1000) // And / Or Limit the rows to something sane
        .Select(pr => new GlobalModel()
          {
              prodModelIndex = pr,
     // These 2 fields are actually redundant, as we now have navigation fields 
     // for these off prodModelIndex
              shiftModel = s, 
              moduleModel = m,
              ddlShift = ddlShift,
              ddlModule = ddlModule
          }).ToList();

    return PartialView(result);
}

在MVC Razor中从db获取数据时的LINQ查询性能问题

3 个答案: