DisplayTemplate 代码需要修复/反馈

Question

我有一个名为 _FooterMenu 的 DisplayTemplate，其中对下面的动态页脚菜单进行了编码。

没什么要问的

有什么办法可以简化 DisplayTemplate 代码吗？

我应该使用 HtmlHelper 来隐藏逻辑吗？

布局页面中的代码

@Html.DisplayForModel("_FooterMenu")

//DisplayTemplate 代码。

@model HomeViewModel

@{var distMenu = Model.FooterMenu.Where(c => c.Title != null).Select(m => new {Title = m.Title}).Distinct() ;}
@foreach (var i in distMenu)
{
    <div class="linksarea">
    <h5>@i.Title</h5>
        <ul>
                @foreach (var item in Model.FooterMenu.FindAll(x => x.Title == i.Title))
                {
                    <li>
                        @Html.ActionLink(@item.DisplayName, "Page", "Home", new { name = @item.UrlName }, null)
                    </li>   
                }                                                   
        </ul>
    </div>
}   

Answer 1

我认为优化代码的关键是为页脚菜单项使用“GroupBy”，这将减少对 2 个 ForEach 循环的需求。我模拟了您的 MVC 应用程序，这是我想出的代码片段：

   @{
        var distMenu = Model.FooterMenu.GroupBy(q => q.Title);
    }

    @foreach (var grp in distMenu)
    { 
        <div class="linksArea">
            <h5>@grp.Key</h5>
            <ul>
            @foreach(var item in grp)
            {
                <li>
                    @Html.ActionLink(@item.DisplayName, "Page", "Home", new { name = @item.Url }, null)
                </li>   
            }
            </ul>
        </div>
    }

希望这会有所帮助。