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>
}

Answer 3

MVC的核心点是分离责任区域。模型必须完全独立，即只知道自己。除非将它与业务逻辑混合，否则在模型中使用与表示相关的逻辑是很正常的。在您的示例中，明显的改进可能是将所有授权信息移动到模型对象中，并将if / else逻辑保留在视图中。

<强>控制器：

model.IsAuthenticated = User.Identity.IsAuthenticated;
model.IsAdministrator = User.IsInRole("administrator");

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

3 个答案: