避免在视图中添加if/else逻辑

Question

我被要求避免在视图中添加像if/else一样的逻辑.当我正在开发页面的登录/注册功能时,如果用户在角色中我必须显示一组链接而如果他是在另一个角色.

这是我到目前为止所做的:

<ul id="menu">
    <li>@Html.ActionLink("Products", "Books", "Home")</li>
    @if (User.Identity.IsAuthenticated)
    { 
         <li>@Html.ActionLink("Log Out" ,"LogOut","Account")</li>
    }
    else
    {
         <li>@Html.ActionLink("Log In" ,"LogIn","Account")</li>
    }
    @if(User.IsInRole("administrator"))
    {
       <li>@Html.ActionLink("Product Manager", "Books", "ProductManager")</li>
    }
</ul>

此代码存储在_Layout.cshtml文件中.我希望避免将逻辑添加到视图中.

有什么方法可以做到吗？

Answer 1

如果您不想在视图中执行此操作,我认为您已在控制器中执行此操作.这样的东西可以用于从单个视图中删除条件语句:

// controller
ActionResult MyAction()
{
    if (!User.Identity.IsAuthenticated)
    {
        ViewBag.MenuControl = "Menu/NotLoggedIn"
    } 
    else if (User.IsInRole("Administrator"))
    {
        ViewBag.MenuControl = "Menu/Administrator"
    } 
    else
    {
        ViewBag.MenuControl = "Menu/LoggedIn"
    }

    ...
}

// view
@Html.Partial(ViewBag.MenuControl);

或者为了在许多视图中共享这个逻辑,我建议你创建一个特定的MenuController来容纳这个逻辑.

ActionResult RenderMenu()
{
    string template;
    if (!User.Identity.IsAuthenticated)
    {
        template = "Menu/NotLoggedIn"
    } 
    else if (User.IsInRole("Administrator"))
    {
        template = "Menu/Administrator"
    } 
    else
    {
        template = "Menu/LoggedIn"
    }

    return View(template);
}

// view
@Html.Action("RenderMenu", "MenuController")

但是 ......"视图逻辑"和"控制器逻辑"之间存在很大差异.毕竟,这是我们希望将视图与MVC架构中的控制器分开的主要原因之一.像"避免视图中的所有条件语句"这样的简单规则确实忽略了MVC的设计方式.

我更喜欢在视图中执行此操作,因为实际上它与如何格式化视图有关,而不是控制器应该如何运行.我坚持你现在的代码.

Answer 2

我认为if/else你的意见中有陈述没有错.问题在于实际的条件语句本身.

以此为例:

@if(User.IsInRole("administrator"))
{
   <li>@Html.ActionLink("Product Manager", "Books", "ProductManager")</li>
}

在这里,您将业务逻辑与视图逻辑混合.您正在引用角色的名称.这与视图无关.要解决这个问题,请尝试以下方法:

型号代码

public class MyModel
{
    public bool IsAdministrator { get; }
}

控制器代码

myModel.IsAdministrator = User.IsInRole("administrator");

查看代码

@if(this.Model.IsAdministrator)
{
   <li>@Html.ActionLink("Product Manager", "Books", "ProductManager")</li>
}